diff options
| author | Dmitri Gribenko <gribozavr@gmail.com> | 2012-06-21 00:28:14 +0000 |
|---|---|---|
| committer | Dmitri Gribenko <gribozavr@gmail.com> | 2012-06-21 00:28:14 +0000 |
| commit | bd1be2c90c6bbc83fe420da51733469a4904831b (patch) | |
| tree | cc5f48e7f389360f3a9da18fc27224ef7879c394 /lib/AST/RawCommentList.cpp | |
| parent | 52a04812e5767dab68efb33ad044760b5b168941 (diff) | |
RawCommentList::addComment: fix the assertion so it actually checks that new comment is after the last one (change Comments[0] to Comments.back()), and handle the case of two consecutive comments, e.g. /** *//* */. There is already a testcase for that (but it didn't trigger the assert because the assert itself was wrong).
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@158882 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/AST/RawCommentList.cpp')
| -rw-r--r-- | lib/AST/RawCommentList.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/lib/AST/RawCommentList.cpp b/lib/AST/RawCommentList.cpp index 06510294e7..1baa9b87eb 100644 --- a/lib/AST/RawCommentList.cpp +++ b/lib/AST/RawCommentList.cpp @@ -159,8 +159,10 @@ void RawCommentList::addComment(const RawComment &RC) { return; assert((Comments.empty() || + Comments.back().getSourceRange().getEnd() == + RC.getSourceRange().getBegin() || SourceMgr.isBeforeInTranslationUnit( - Comments[0].getSourceRange().getEnd(), + Comments.back().getSourceRange().getEnd(), RC.getSourceRange().getBegin())) && "comments are not coming in source order"); |