[llvm] [Transforms][IPO] Add func suffix in ArgumentPromotion and DeadArgume… (PR #105742)

Eli Friedman via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 23 14:19:14 PDT 2024


https://github.com/efriedma-quic commented:

The general idea of changing the name seems reasonable; in these cases, we know the program doesn't actually depend on the name of the symbol, so the name is just for the sake of debugging.

Should we try to prevent accumulating more than one suffix on a function?

If the name is using C++ mangling, does the demangler produce something reasonable?  If not, can we modify the name in a demangler-friendly way?

If you try to set a breakpoint on a function, how will a debugger react to the changed name?

https://github.com/llvm/llvm-project/pull/105742


More information about the llvm-commits mailing list