On Sat, Oct 23, 2010 at 4:27 PM, Peter Collingbourne <[email protected]> wrote:
> Hi,
>
> This implements a GNU C extension: two types are compatible if they
> appear as a function argument, one of the types is a transparent
> union type and the other type is compatible with a union member.
>
> OK to commit?

+  if (!getLangOptions().CPlusPlus && getLangOptions().GNUMode) {

This check is unnecessary; we can't have a TransparentUnionAttr unless
the source uses GNU extensions, and we shouldn't ever get here in C++
mode (mergeTypes doesn't have useful semantics in C++).

Otherwise, looks fine; go ahead and commit.

-Eli
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to