[PATCH] D44670: [CXX] Templates specialization visibility can be wrong

[Doug Gregor via Phabricator via cfe-commits]

doug.gregor added inline comments.


================
Comment at: lib/AST/Decl.cpp:1078
+    for (const auto *RD :
+         spec->getSpecializedTemplate()->getTemplatedDecl()->redecls()) {
+      auto Vis = getVisibilityOf(RD, kind);
----------------
Do we want to look at *all* redeclarations, or only those declarations that precede the declaration that we found? The latter seems more correct, but IIRC visibility has often been able to "look forward" to declarations that come later in the translation unit.


Repository:
  rC Clang

https://reviews.llvm.org/D44670