From: Benjamin Kramer <benny.kra@googlemail.com>
Date: Tue, 22 Nov 2011 20:39:31 +0000 (+0000)
Subject: Remove assert from hot code path and add a clarifying comment.
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=6300f5b4382d5128c9346a4d4f05e7bac2e3d771;p=clang

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
---

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;