[PATCH] D70307: [CodeComplete] Constructor overload candidates report as vector(int) instead of vector<string>(int)

[Sam McCall via Phabricator via cfe-commits]

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