sammccall marked 2 inline comments as done.
sammccall added inline comments.


================
Comment at: clang/test/CodeCompletion/templates.cpp:35
+// CHECK-CONSTRUCTOR: OVERLOAD: X(<#double#>)
+// (rather than X<type-parameter-0-0 *>(<#double#>)
----------------
hokein wrote:
> shouldn't be now `X<T *>` after your recent patch?
Sadly not, that patch doesn't fix every case (and I couldn't work out how to 
fix most of them)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70307



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

Reply via email to