[PATCH] D25308: [Sema] Ignore transparent_union attributes in C++

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 10 06:42:41 PDT 2016


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





More information about the cfe-commits mailing list