[cfe-commits] r128208 - in /cfe/trunk: lib/Sema/IdentifierResolver.cpp test/SemaCXX/goto.cpp test/SemaCXX/goto2.cpp

[Benjamin Kramer]

On 24.03.2011, at 11:35, Douglas Gregor wrote:

> Author: dgregor
> Date: Thu Mar 24 05:35:39 2011
> New Revision: 128208
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=128208&view=rev
> Log:
> Fix the insertion of label declarations into the identifier chain in
> the case where we only have a single identifier with that name in the
> chain. Fixes PR9463 for real this time.

Hi Doug,

Looks like this commit broke selfhost. Reduced testcase:

$ cat t.cc
extern "C" { void label(); };

int bar();

void foo() { 
  goto label;
label:
  int t = bar();
}

$ clang++ t.cc
Assertion failed: (LabelAndGotoScopes.count(To) && "Jump didn't get added to scopes?"), function CheckJump, file JumpDiagnostics.cpp, line 541.