[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