[clang] [Clang] Implement P3034R1 Module Declarations Shouldn’t be Macros (PR #90574)

via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 30 08:15:15 PDT 2024


================
@@ -1671,6 +1671,8 @@ def err_unexpected_module_decl : Error<
   "module declaration can only appear at the top level">;
 def err_module_expected_ident : Error<
   "expected a module name after '%select{module|import}0'">;
+def err_module_decl_cannot_be_macros : Error<
+  "module declaration cannot be a macro">;
----------------
yronglin wrote:

Should we also print the macro spelling text in diagnostic message? And I found that the first letter in other diagnostic message is lower case, should we update `The name of a ...` to `the name of a ...`?

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


More information about the cfe-commits mailing list