[PATCH] D23096: [Sema] Pass CombineWithOuterScope = true to constructor of LocalInstantiationScope

Serge Pavlov via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 3 11:01:39 PDT 2016


sepavloff added inline comments.

================
Comment at: test/SemaTemplate/default-expr-arguments-3.cpp:20
@@ +19,2 @@
+  }
+}
----------------
Please add the following test to the patch:
```
template<typename T> void f1() {
  enum class foo { a, b };
  struct S {
    int g1(foo n = foo::a);
  };
}

template void f1<int>();
```
It must also compile correctly.


https://reviews.llvm.org/D23096





More information about the cfe-commits mailing list