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

Wed Jun 28 23:20:03 PDT 2023

Hahnfeld added a comment.

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

> In D153003#4456595 <https://reviews.llvm.org/D153003#4456595>, @rsmith wrote:
>
>> I think the behavior change for the testcase here is correct, though I'm not sure that the patch is getting that behaviour change in the right way. Per [temp.type]/1.4 (http://eel.is/c++draft/temp.type#1.4),
>>
>>> Two template-ids are the same if [...] their corresponding template template-arguments refer to the same template.
>>
>> so `B<A>` and `B<NS::A>` are the same type. The stricter "same sequence of tokens" rule doesn't apply here, because using-declarations are not definitions.
>
> Got it. Thanks for your commenting. I can't reopen this page. So I file an issue here https://github.com/llvm/llvm-project/issues/63595. @Hahnfeld you can still work on this by following the @rsmith 's suggestion  if you're still interested. Or I'd like to take it.

You mean re-open this review? Which suggestions exactly?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153003