[PATCH] Sema: Instantiating variable definitions weren't local enough

Richard Smith richard at metafoo.co.uk
Wed Nov 27 14:46:47 PST 2013


  LGTM, thanks!


================
Comment at: include/clang/Sema/Sema.h:6417
@@ +6416,3 @@
+    ~SavePendingLocalImplicitInstantiationsRAII() {
+      SavedPendingLocalImplicitInstantiations.swap(
+          S.PendingLocalImplicitInstantiations);
----------------
Assert that `S.PendingLocalImplicitInstantiations` is empty here?


http://llvm-reviews.chandlerc.com/D2238

BRANCH
  PR14374

ARCANIST PROJECT
  clang



More information about the cfe-commits mailing list