[PATCH] D46929: Fix a mangling failure on clang-cl C++17

[Reid Kleckner via Phabricator via cfe-commits]

rnk added a comment.

I searched around, and I noticed that `VTableContext::getMethodVTableIndex` has the same implied contract that the caller must always provide a canonical declaration or things will break. It looks like it has three callers, and they all do this. We should probably sink the canonicalization into this helper as well and clean up the now-superfluous canonicalizations at the call site.


Repository:
  rC Clang

https://reviews.llvm.org/D46929