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

Richard Smith via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 10 12:16:13 PDT 2016


On Mon, Oct 10, 2016 at 10:45 AM, Hal Finkel via cfe-commits <
cfe-commits at lists.llvm.org> wrote:

> hfinkel added a comment.
>
> In https://reviews.llvm.org/D25308#566176, @arphaman wrote:
>
> > The updated patch now makes clang warn every time it encounters this
> attribute in C++ mode. Would that be the desired behaviour?
>
>
> As I understand it, transparent_union was designed for use in system
> headers, and these might certainly be included into C++ source files. Does
> that not work correctly, or does the feature just not work correctly when
> combined with C++-specific features (templates, function overloading, etc.)?


Neither Clang nor GCC supports this attribute in C++ mode in any way, as
far as I can see. All uses of this attribute within the glibc headers are
behind #ifndef __cplusplus.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20161010/4f450dba/attachment-0001.html>


More information about the cfe-commits mailing list