[llvm] [TableGen] Detect invalid -D arguments and fail. (PR #102813)

Sergei Barannikov via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 11 15:58:10 PDT 2024


================
@@ -54,9 +74,17 @@ TGLexer::TGLexer(SourceMgr &SM, ArrayRef<std::string> Macros) : SrcMgr(SM) {
   PrepIncludeStack.push_back(
       std::make_unique<std::vector<PreprocessorControlDesc>>());
 
-  // Put all macros defined in the command line into the DefinedMacros set.
-  for (const std::string &MacroName : Macros)
+  // Add all macros defined on the command line to the DefinedMacros set.
+  // Check invalid macro names and print fatal error if we find one.
+  for (const std::string &MacroName : Macros) {
+    const char *Begin = MacroName.c_str();
+    const char *End = lexMacroName(Begin);
----------------
s-barannikov wrote:

This assumes MacroName is null-terminated. If it really is, this should be asserted before calling the function.


https://github.com/llvm/llvm-project/pull/102813


More information about the llvm-commits mailing list