erichkeane added inline comments.

================
Comment at: clang/test/CXX/drs/dr11xx.cpp:31
+  }
+}
+
----------------
Can you also add the last example (with struct A) from the issue, and find a 
way to confirm that it is calling the right version?


================
Comment at: clang/www/cxx_dr_status.html:6476
     <td>Remove dual-scope lookup of member template names</td>
-    <td class="none" align="center">Unknown</td>
+    <td class="full" align="center">Clang 6</td>
   </tr>
----------------
Were you able to track down which patch fixed this in clang 6?

I also don't think we can mark this as 'full' support, if its still broken in 
C++98.  Unless we can find a good reason this wasn't/wont be done for C++98 as 
well, I don't know if we can mark this as 'done'.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D142315/new/

https://reviews.llvm.org/D142315

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to