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

via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 1 16:39:31 PDT 2024


goldsteinn 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.

Think this should be fixed :)

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


More information about the cfe-commits mailing list