[PATCH] D64380: Add 'require_designated_init' and 'required' attribute to clang

[Saleem Abdulrasool via Phabricator via cfe-commits]

compnerd added a comment.

I don't see any cases where `[[clang::required]]` is tested, am I missing something?



================
Comment at: clang/test/SemaCXX/attr-designated-init-required.cpp:3
+
+#define ATTR [[clang::designated_init_required]]
+
----------------
Why the macro?


================
Comment at: clang/test/SemaCXX/attr-require-designated-init.cpp:3
+
+#define ATTR [[clang::require_designated_init]]
+
----------------
Why the macro?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D64380