[PATCH] D153003: [ODRHash] Fix ODR hashing of template names

Fri Jun 30 05:14:18 PDT 2023

v.g.vassilev added a comment.

In D153003#4462388 <https://reviews.llvm.org/D153003#4462388>, @ChuanqiXu wrote:

>> Oh, I guess we're somehow adding a semi-resolved form of the base class type of D into the ODR hash, which then includes details of the using-declaration. That seems wrong -- we should either (preferably) be including only the syntactic form of the base specifier (because local syntax is what the ODR covers), or the canonical type (which should be the same for both using-declarations).
>
> Got it. I'll try to fix it. Thanks for the suggestion.

Thanks @rsmith for the differential diagnosis!

@ChuanqiXu, could you add me and @Hahnfeld in the loop as that's critical for us.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153003