[clang] [llvm] [Inliner] Propagate more attributes to params when inlining (PR #91101)
    Nikita Popov via llvm-commits 
    llvm-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 llvm-commits
mailing list