[PATCH] D110326: [LegacyPassManager] Demangling function name in ftime-trace output.
Anton Afanasyev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Oct 15 04:45:58 PDT 2021
anton-afanasyev added a comment.
In D110326#3061751 <https://reviews.llvm.org/D110326#3061751>, @Sockke wrote:
> In D110326#3036509 <https://reviews.llvm.org/D110326#3036509>, @anton-afanasyev wrote:
>
>> In D110326#3029804 <https://reviews.llvm.org/D110326#3029804>, @Sockke wrote:
>>
>>> How about adding an option to determine whether to demangle the function name or not?
>>
>> Making this a non-default option sounds good.
>
> Sorry for delayed response. Thanks for your review, what is the follow-up of this patch? Any other thoughts?
As before, I believe making this a non-default option would be a good compromise.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D110326/new/
https://reviews.llvm.org/D110326
More information about the llvm-commits
mailing list