[PATCH] D102241: [clang] p1099 4/5: using enum EnumTag

[Bruno Cardoso Lopes via Phabricator via cfe-commits]

bruno accepted this revision.
bruno added a reviewer: rsmith.
bruno added a comment.
This revision is now accepted and ready to land.

Thanks for adding the tests. LGTM after some remaining nitpick.



================
Comment at: clang/lib/Sema/SemaDeclCXX.cpp:11632
+  // As with enum-decls, we ignore attributes for now.
+
+  auto *Enum = cast<EnumDecl>(DS.getRepAsDecl());
----------------
Remove this empty line.


================
Comment at: clang/lib/Sema/SemaDeclCXX.cpp:12339
+
+    if (!CheckUsingShadowDecl(UD, EC, Previous, PrevDecl)) {
+      BuildUsingShadowDecl(S, UD, EC, PrevDecl);
----------------
No need for curly braces here.


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

https://reviews.llvm.org/D102241