Finally: enable the asserts that assure very conservative integrity checks.

This commit is contained in:
Vassil Vassilev 2013-08-08 11:22:29 +02:00 committed by sftnight
parent dc993fb58b
commit a571d69951
3 changed files with 8 additions and 6 deletions

View File

@ -398,12 +398,18 @@ namespace cling {
commitTransaction(deserT);
}
void IncrementalParser::transformTransactionAST(Transaction* T) const {
void IncrementalParser::transformTransactionAST(Transaction* T) {
bool success = true;
// We are sure it's safe to pipe it through the transformers
// Consume late transformers init
Transaction* initT = beginTransaction(CompilationOptions());
for (size_t i = 0; success && i < m_ASTTransformers.size(); ++i)
success = m_ASTTransformers[i]->TransformTransaction(*T);
if (endTransaction(initT))
commitTransaction(initT);
if (!success)
T->setIssuedDiags(Transaction::kErrors);
}

View File

@ -211,7 +211,7 @@ namespace cling {
///
///\param[in] T - the transaction to be transformed.
///
void transformTransactionAST(Transaction* T) const;
void transformTransactionAST(Transaction* T);
///\brief Runs IR transformers on a transaction.
///

View File

@ -103,10 +103,8 @@ namespace cling {
void Transaction::append(DelayCallInfo DCI) {
assert(!DCI.m_DGR.isNull() && "Appending null DGR?!");
#ifdef TEMPORARILY_DISABLED
assert(getState() == kCollecting
&& "Cannot append declarations in current state.");
#endif
forceAppend(DCI);
}
@ -115,7 +113,6 @@ namespace cling {
assert((getState() == kCollecting || getState() == kCompleted)
&& "Must not be");
#ifdef TEMPORARILY_DISABLED
#ifndef NDEBUG
// Check for duplicates
for (size_t i = 0, e = m_DeclQueue.size(); i < e; ++i) {
@ -127,7 +124,6 @@ namespace cling {
if (oldDCI.m_Call != kCCIHandleVTable)
assert(oldDCI != DCI && "Duplicates?!");
}
#endif
#endif
bool checkForWrapper = !m_WrapperFD;