[PATCH] D17407: [Sema] PR25755 Handle out of order designated initializers

Don Hinton via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 22 12:08:55 PDT 2019


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





More information about the cfe-commits mailing list