JonChesterfield added a comment. In D74361#2218294 <https://reviews.llvm.org/D74361#2218294>, @erichkeane wrote:
> Yep! Declaring a global variable that isn't 'extern' with an incomplete type > is disallowed anyway, so if you call RequireCompleteType, you're likely just > diagnosing that early. > > You MIGHT have to do some work to allow: > > struct S; > extern S foo __attribute__((loader_uninitialized)); I'll add that to the tests. Looking OK so far, added a check to SemaDecl and the crash is gone. No fix needed for C. Will have a patch up soon Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74361/new/ https://reviews.llvm.org/D74361 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
