[PATCH] D118249: [demangler] remove StdQualifiedName

Chuanqi Xu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 7 05:19:52 PST 2022


ChuanqiXu accepted this revision.
ChuanqiXu added a comment.
This revision is now accepted and ready to land.

In D118249#3300683 <https://reviews.llvm.org/D118249#3300683>, @urnathan wrote:

> In D118249#3299854 <https://reviews.llvm.org/D118249#3299854>, @ChuanqiXu wrote:
>
>> 
>
>
>
>> So the key point here is: Although `::std` namespace is special in mangling, we could treat it homogeneously in the Node since we could handle it later. Do I understand right? If yes, I think this one should be good.
>
> yes, that is correct.  i hope you enjoyed your vacation.

Thanks : )


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

https://reviews.llvm.org/D118249



More information about the llvm-commits mailing list