[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