Revert "[clang][Lex] Fix a crash on malformed string literals"

This reverts commit 36a2002.
llvm · Oct 5, 2022 · feea7ef · feea7ef
1 parent 2813bc5
commit feea7ef
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 9 deletions.
diff --git a/clang/lib/Lex/LiteralSupport.cpp b/clang/lib/Lex/LiteralSupport.cpp
@@ -545,6 +545,7 @@ static bool ProcessNamedUCNEscape(const char *ThisTokBegin,
            diag::err_delimited_escape_missing_brace)
           << StringRef(&ThisTokBuf[-1], 1);
     }
+    ThisTokBuf++;
     return false;
   }
   ThisTokBuf++;

diff --git a/clang/test/Lexer/char-escapes-delimited.c b/clang/test/Lexer/char-escapes-delimited.c
@@ -94,7 +94,7 @@ void named(void) {
 
   unsigned h = U'\N{LOTUS}';                      // ext-warning {{extension}} cxx2b-warning {{C++2b}}
   unsigned i = u'\N{GREEK CAPITAL LETTER DELTA}'; // ext-warning {{extension}} cxx2b-warning {{C++2b}}
-  char j = '\NN';                                 // expected-error {{expected '{' after '\N' escape sequence}} expected-warning {{multi-character character constant}}
+  char j = '\NN';                                 // expected-error {{expected '{' after '\N' escape sequence}}
   unsigned k = u'\N{LOTUS';                       // expected-error {{incomplete universal character name}}
 }
 

diff --git a/clang/unittests/Lex/LexerTest.cpp b/clang/unittests/Lex/LexerTest.cpp
@@ -18,7 +18,6 @@
 #include "clang/Basic/TokenKinds.h"
 #include "clang/Lex/HeaderSearch.h"
 #include "clang/Lex/HeaderSearchOptions.h"
-#include "clang/Lex/LiteralSupport.h"
 #include "clang/Lex/MacroArgs.h"
 #include "clang/Lex/MacroInfo.h"
 #include "clang/Lex/ModuleLoader.h"
@@ -660,11 +659,4 @@ TEST_F(LexerTest, RawAndNormalLexSameForLineComments) {
   }
   EXPECT_TRUE(ToksView.empty());
 }
-
-TEST_F(LexerTest, BrokenStringLiteral) {
-  const llvm::StringLiteral Source = R"cpp("\N")cpp";
-  // Make sure this isn't crashing.
-  StringLiteralParser P(Lex(Source), *PP);
-  EXPECT_TRUE(P.hadError);
-}
 } // anonymous namespace