hintonda added a comment.

I took a look at C++ designator restrictions shown here aggregate 
initialization 
<https://en.cppreference.com/w/cpp/language/aggregate_initialization>, and 
believe it to be orthogonal to this change.  Thus, I'd prefer to put that in a 
separate patch.

However, based on this, I also believe I should move the tests into 
test/Sema/designated-initializers.c.

Please let me know if that sounds like the right thing to do.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D17407



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

Reply via email to