[PATCH] D23492: Make function local tags visible.

Vassil Vassilev via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 22 09:54:40 PDT 2016


v.g.vassilev added inline comments.

================
Comment at: lib/Sema/SemaTemplateInstantiateDecl.cpp:3617
@@ +3616,3 @@
+    PatternDef = nullptr;
+  // FIXME: We need to track the instantiation stack in order to know which
+  // definitions should be visible within this instantiation.
----------------
akuegel wrote:
> Line 3619 to 3623 breaks the compilation of some of our targets. I am not sure if it is because of a mistake in our code or if there is something bad in this patch.
> This comment seems to indicate that there is still something missing to make the check here correct?
@akuegel, would you be able to reduce a standalone example?


https://reviews.llvm.org/D23492





More information about the cfe-commits mailing list