[clang] [llvm] [Inliner] Propagate more attributes to params when inlining (PR #91101)

Nikita Popov via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 20 02:40:46 PDT 2024


nikic wrote:

It looks like this change causes regressions in sinking/hoisting, because we currently can't handle identical calls with different call site attributes. Basic example: https://llvm.godbolt.org/z/5dTcTfs1x

We need to extend the attribute intersection support to call attributes. It's a bit trickier in this case because we can only intersect non-ABI attributes. If there are differences in ABI attributes, we can't merge the calls.

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


More information about the cfe-commits mailing list