[cfe-commits] r66896 - in /cfe/trunk: lib/Sema/Sema.h lib/Sema/SemaExpr.cpp lib/Sema/SemaStmt.cpp test/Sema/block-labels.c

[Chris Lattner]

On Mar 13, 2009, at 9:03 AM, Steve Naroff wrote:

> Author: snaroff
> Date: Fri Mar 13 11:03:38 2009
> New Revision: 66896
>
> URL: http://llvm.org/viewvc/llvm-project?rev=66896&view=rev
> Log:
> Reimplement fix for <rdar://problem/6451399> problems with labels  
> and blocks.
>
> This solution is much simpler (and doesn't add any per-scope  
> overhead, which concerned Chris).
>
> The only downside is the LabelMap is now declared in two places  
> (Sema and BlockSemaInfo). My original fix tried to unify the  
> LabelMap in "Scope" (which would support nested functions in  
> general). In any event, this fixes the bug given the

Looks great to me, thanks Steve!

-Chris