This revision was automatically updated to reflect the committed changes.
Closed by commit rL348586: [CTU] Add asserts to protect invariants (authored by
martong, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.org/D55132?vs=176151&id=17717
xazax.hun accepted this revision.
xazax.hun added inline comments.
Comment at: lib/CrossTU/CrossTranslationUnit.cpp:249
CrossTranslationUnitContext::importDefinition(const FunctionDecl *FD) {
+ assert(FD->hasBody() && "Functions to be imported should have body.");
+
---
a_sidorin accepted this revision.
a_sidorin added a comment.
This revision is now accepted and ready to land.
More assertions are always good.
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D55132/new/
https://reviews.llvm.org/D55132
martong created this revision.
martong added reviewers: xazax.hun, a_sidorin.
Herald added subscribers: cfe-commits, gamesh411, Szelethus, dkrupp, rnkovacs.
Repository:
rC Clang
https://reviews.llvm.org/D55132
Files:
lib/CrossTU/CrossTranslationUnit.cpp
Index: lib/CrossTU/CrossTranslationU
