[PATCH] D150966: [clang] Don't define predefined macros multiple times

[Sergei Barannikov via Phabricator via cfe-commits]

barannikov88 added a comment.

Is it possible/worth to add an assertion to Builder.defineMacro to enforce this 
one definition rule?



================
Comment at: clang/lib/Frontend/InitPreprocessor.cpp:1305
+    Builder.defineMacro("__ELF__");
+  }
+
----------------
Redundant braces.



Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D150966/new/

https://reviews.llvm.org/D150966

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits