[PATCH] D17893: Sema: Add semantic analysis for the C++ ABI stability attributes and whitelist.

Mehdi AMINI via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 16 08:30:18 PDT 2016


joker.eph added a comment.

LGTM, but I'm not a clang expert, so if Richard or someone else could double-check?


================
Comment at: lib/Sema/SemaDeclCXX.cpp:4999
@@ +4998,3 @@
+    return;
+  }
+
----------------
Isn't this correct by the loop which starts with `DC = InnermostExternalDC`?


http://reviews.llvm.org/D17893





More information about the cfe-commits mailing list