[compiler-rt] [llvm] [Transforms][IPO] Add func suffix in ArgumentPromotion and DeadArgumentElimination (PR #109899)

via llvm-commits llvm-commits at lists.llvm.org
Sun Sep 29 20:48:06 PDT 2024


yonghong-song wrote:

> > In kernel tracing, if func name is not changed, the func signature will be assumed to be based on source code. If compiler silently changes signature, then kernel tracing could get incorrect result. So we either need to change func name to indicate func signature have changed or we need additional information in the binary which will tell signature has changed and better what is the new signature.
> 
> The signature won't change if it's externally visible.

@arsenm Indeed, you are right. We only talk about static functions whose signatures may change.

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


More information about the llvm-commits mailing list