[PATCH] D118249: [demangler] remove StdQualifiedName

Nathan Sidwell via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 28 05:54:45 PST 2022


urnathan added a comment.

In D118249#3278121 <https://reviews.llvm.org/D118249#3278121>, @ChuanqiXu wrote:

> In D118249#3275622 <https://reviews.llvm.org/D118249#3275622>, @urnathan wrote:
>
>> No, sorry for being unclear.
>>
>> std::name (St4Name)-> we would use StdQualifiedName
>> std::name1::name2 (NSt5Name15Name2E)-> we would use NestedName (std, ...)
>> std::name1<Inst> (NSt5Name1I4InstEE)-> we would also use NestedName (std, ...)
>>
>> this patch moves to using NestedName in all these cases.  (The other substitutions, such as 'Sa' remain unchanged)
>
> Yeah, but it looks like StdQualifiedName is deleted completely...

Yes.  That is the intent.


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

https://reviews.llvm.org/D118249



More information about the llvm-commits mailing list