diff options
author | Benjamin Kramer <benny.kra@googlemail.com> | 2011-11-22 20:39:31 +0000 |
---|---|---|
committer | Benjamin Kramer <benny.kra@googlemail.com> | 2011-11-22 20:39:31 +0000 |
commit | 6300f5b4382d5128c9346a4d4f05e7bac2e3d771 (patch) | |
tree | f32c3bb6ecc94a446abaa19cc0e6a50d847a2ddd /lib/Lex/Lexer.cpp | |
parent | 1e855657867e95c8302f55de3f63409f9b1927c3 (diff) |
Remove assert from hot code path and add a clarifying comment.
The assert wasn't adding much value but slowed down Release+Asserts builds.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@145082 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Lex/Lexer.cpp')
-rw-r--r-- | lib/Lex/Lexer.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/lib/Lex/Lexer.cpp b/lib/Lex/Lexer.cpp index a115579843..a9b11d69c9 100644 --- a/lib/Lex/Lexer.cpp +++ b/lib/Lex/Lexer.cpp @@ -1920,10 +1920,10 @@ bool Lexer::SkipBlockComment(Token &Result, const char *CurPtr) { while (CurPtr+16 <= BufferEnd) { int cmp = _mm_movemask_epi8(_mm_cmpeq_epi8(*(__m128i*)CurPtr, Slashes)); if (cmp != 0) { - // Adjust the pointer to the first '/' that was found. - CurPtr += llvm::CountTrailingZeros_32(cmp); - C = *CurPtr++; - assert(C == '/'); + // Adjust the pointer to point directly after the first slash. It's + // not necessary to set C here, it will be overwritten at the end of + // the outer loop. + CurPtr += llvm::CountTrailingZeros_32(cmp) + 1; goto FoundSlash; } CurPtr += 16; |