[PATCH] PR22931 - When cloning scopes, reset the scope in Sema instead of using the cloned scope as the current scope

Richard Trieu rtrieu at google.com
Wed Mar 18 15:04:08 PDT 2015


Committed in r232675

As Richard Smith mentioned, this is an assertion failure which wouldn't show up in release builds.  In addition, look at the test case on what is required to trigger this assertion:

2 instantiation scopes (templated class plus templated function)
2 late parsed attributes
second attribute requiring a lookup on the inner scope (The parameter decl F)

Missing any of these would avoid the assertion.


http://reviews.llvm.org/D8407

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list