[cfe-commits] r138969 - in /cfe/trunk: include/clang/Sema/Sema.h lib/Sema/SemaDecl.cpp lib/Sema/SemaInit.cpp test/SemaCXX/uninitialized.cpp

[Richard Trieu]

On Tue, Sep 6, 2011 at 4:36 PM, Richard Trieu <rtrieu at google.com> wrote:

> On Tue, Sep 6, 2011 at 11:19 AM, Joerg Sonnenberger
> <joerg at britannica.bec.de> wrote:
> > On Thu, Sep 01, 2011 at 09:44:13PM -0000, Richard Trieu wrote:
> >> Author: rtrieu
> >> Date: Thu Sep  1 16:44:13 2011
> >> New Revision: 138969
> >>
> >> URL: http://llvm.org/viewvc/llvm-project?rev=138969&view=rev
> >> Log:
> >> Extend the self-reference warning to catch when a constructor references
> itself upon initialization, such as using itself within its own copy
> constructor.
> >
> > This produces a regression for the NetBSD code base. Test case is:
> >
> > struct foo { char a[100], *e; } bar = { .e = bar.a };
> >
> > This is well-defined, since bar.a is an address, not a value depending
> > on the initialisation.
> >
> > Joerg
> > _______________________________________________
> > cfe-commits mailing list
> > cfe-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
> >
>
> I am taking a look at this now.
>

Joerg,

Revision 139213 has a fix to no longer warn on your example.  Let me know if
there's any more problems.

Richard.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110906/c4137639/attachment.html>