Sorry about the delay; I haven't lost track of this, just haven't had a chance to take a look yet.
-Eli On Wed, Feb 1, 2012 at 9:25 AM, Enea Zaffanella <[email protected]> wrote: > Ping. > > NOTE: fyi, the commit causing regression was detected by svn-bisect. > > Enea. > > > On 01/28/2012 06:37 PM, Enea Zaffanella wrote: >> >> On 01/21/2012 02:01 AM, Eli Friedman wrote: >>> >>> Author: efriedma >>> Date: Fri Jan 20 19:01:51 2012 >>> New Revision: 148610 >>> >>> URL: http://llvm.org/viewvc/llvm-project?rev=148610&view=rev >>> Log: >>> Make clang's AST model sizeof and typeof with potentially-evaluated >>> operands correctly, similar to what we already do with typeid. >>> >>> >>> Added: >>> cfe/trunk/test/Sema/vla-2.c >>> Modified: >>> cfe/trunk/include/clang/Sema/Sema.h >>> cfe/trunk/lib/Parse/ParseDecl.cpp >>> cfe/trunk/lib/Parse/ParseExpr.cpp >>> cfe/trunk/lib/Sema/AnalysisBasedWarnings.cpp >>> cfe/trunk/lib/Sema/SemaExpr.cpp >>> cfe/trunk/test/Sema/i-c-e.c >>> cfe/trunk/test/SemaCXX/vararg-non-pod.cpp >> >> >> >> It seems that this change is causing a spurious parse error on the >> following program fragment: >> >> $ cat bug.c >> int foo(int a) { >> return sizeof(*(int(*)[({ goto v; v: a;})]) 0); >> } >> >> >> $ clang -fsyntax-only bug.c >> bug.c:2:37: error: >> redefinition of label 'v' >> return sizeof(*(int(*)[({ goto v; v: a;})]) 0); >> ^ >> bug.c:2:37: note: previous >> definition is here >> 1 error generated. >> >> Enea. >> _______________________________________________ >> cfe-commits mailing list >> [email protected] >> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits >> > _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
