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

[via llvm-commits]

yonghong-song wrote:

@teresajohnson @arsenm I tried sampling based profiling with one of bpf applications and it looks like the added suffixes are not affecting sampling based profiling. See the details in the above. I also marked the patch as RFC as you suggested.

For func suffixes (or more than one suffixes), gcc already has precedences. The below are some examples when build clang with gcc:
```
$ llvm-readelf -s clang | grep isra | grep constprop.
...
135408: 00000000061d7140  1529 FUNC    LOCAL  DEFAULT    14 _ZN4llvm4yaml7yamlizeISt6vectorIN12_GLOBAL__N_15ParamESaIS4_EENS0_12EmptyContextEEENSt9enable_ifIXsrNS0_18has_SequenceTraitsIT_EE5valueEvE4typeERNS0_2IOERSA_bRT0_.constprop.0.isra.0
135415: 00000000061de7b0  6558 FUNC    LOCAL  DEFAULT    14 _ZN4llvm4yaml7yamlizeISt6vectorIN12_GLOBAL__N_15ClassESaIS4_EENS0_12EmptyContextEEENSt9enable_ifIXsrNS0_18has_SequenceTraitsIT_EE5valueEvE4typeERNS0_2IOERSA_bRT0_.constprop.0.isra.0
135416: 00000000061e0150  1755 FUNC    LOCAL  DEFAULT    14 _ZN4llvm4yaml7yamlizeISt6vectorIN12_GLOBAL__N_18FunctionESaIS4_EENS0_12EmptyContextEEENSt9enable_ifIXsrNS0_18has_SequenceTraitsIT_EE5valueEvE4typeERNS0_2IOERSA_bRT0_.constprop.0.isra.0
135417: 00000000061e0830  5534 FUNC    LOCAL  DEFAULT    14 _ZN4llvm4yaml7yamlizeISt6vectorIN12_GLOBAL__N_13TagESaIS4_EENS0_12EmptyContextEEENSt9enable_ifIXsrNS0_18has_SequenceTraitsIT_EE5valueEvE4typeERNS0_2IOERSA_bRT0_.constprop.0.isra.0
135419: 00000000061e3ab0  1502 FUNC    LOCAL  DEFAULT    14 _ZN4llvm4yaml7yamlizeISt6vectorIN12_GLOBAL__N_19VersionedESaIS4_EENS0_12EmptyContextEEENSt9enable_ifIXsrNS0_18has_SequenceTraitsIT_EE5valueEvE4typeERNS0_2IOERSA_bRT0_.constprop.0.isra.0
```

And there are even some cases having three suffixes:
```
$ llvm-readelf -s clang | grep isra | grep constprop | grep part
  5663: 000000000147dd70  1041 FUNC    LOCAL  DEFAULT    14 _ZN4llvm10GCNTTIImpl18getVectorInstrCostEjPNS_4TypeENS_19TargetTransformInfo14TargetCostKindEjPNS_5ValueES6_.part.0.constprop.1.isra.0
  5664: 000000000147e190  1041 FUNC    LOCAL  DEFAULT    14 _ZN4llvm10GCNTTIImpl18getVectorInstrCostEjPNS_4TypeENS_19TargetTransformInfo14TargetCostKindEjPNS_5ValueES6_.part.0.constprop.0.isra.0

```
So if new suffixes in clang won't affect functionality, then it should be okay for clang as well to allow multiple suffixes.

Please let me know what you think.

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