[PATCH] D41039: Add support for attribute "trivial_abi"
Akira Hatanaka via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jan 5 10:21:11 PST 2018
ahatanak marked an inline comment as done.
ahatanak added inline comments.
Comment at: include/clang/Basic/AttrDocs.td:2261
+- The class or its subobjects have Objective-C pointer type members and ARC is
> I think the right list of exceptions is:
> - The class directly declares a virtual base or virtual methods.
> - The class has a base class that is non-trivial for the purposes of calls.
> - The class has a non-static data member whose type is non-trivial for the purposes of calls, which includes:
> - classes that are non-trivial for the purposes of calls
> - __weak-qualified types in Objective-C++
> - arrays of any of the above
> I don't see why __strong types would be affected. We've talked about changing the C++ ABI for structs containing __strong members as part of the __strong-pointers-in-structs feature, but that's not even implicated here because there's an attribute which did not previously exist, so there's no established ABI.
I realized I hadn't taken care of arrays of non-trivial structs. I fixed it in checkIllFormedTrivialABIStruct and added a test case to test/SemaObjCXX/attr-trivial-abi.mm.
More information about the cfe-commits