emmettneyman marked 2 inline comments as done. emmettneyman added inline comments.
================ Comment at: clang/test/SemaCXX/attr-designated-init-required.cpp:3 + +#define ATTR [[clang::designated_init_required]] + ---------------- compnerd wrote: > Why the macro? I modeled this file after `test/SemaCXX/attr-require-constant-initialization.cpp` where a macro is defined at the top. I assume just to keep things neat. ================ Comment at: clang/test/SemaCXX/attr-require-designated-init.cpp:3 + +#define ATTR [[clang::require_designated_init]] + ---------------- compnerd wrote: > Why the macro? See above. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64380/new/ https://reviews.llvm.org/D64380 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
