[PATCH] D74361: [Clang] Undef attribute for global variables

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

Aside from the diagnostic wording, I think this LG to me. However, I'd appreciate if @rjmccall would also sign off.



================
Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:5344
+def err_loader_uninitialized_extern_decl : Error<
+  "external declaration of variable cannot have the 'loader_uninitialized' attribute">;
 def err_block_extern_cant_init : Error<
----------------
How would you feel about: `"variable %0 cannot be declared both 'extern' and with the 'loader_uninitialized' attribute"` (or something along those lines) to clarify "external declaration"?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74361