aaron.ballman requested changes to this revision.
aaron.ballman added inline comments.
This revision now requires changes to proceed.


================
Comment at: lib/Sema/SemaDeclAttr.cpp:3011
                                        const AttributeList &Attr) {
+  if (S.getLangOpts().CPlusPlus) {
+    return;
----------------
The correct way to do this is to mark the attribute as C-only on its 
declaration in Attr.td. See the definition of `FlagEnum` for an example.


Repository:
  rL LLVM

https://reviews.llvm.org/D25308



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to