[llvm] [IR] Remove non-canonical matchings (PR #96763)
    Simon Pilgrim via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Sat Jun 29 09:00:12 PDT 2024
    
    
  
https://github.com/RKSimon commented:
What are you trying to accomplish with this patch? The SLSR change looks sensible, we use the m_c_Add/m_c_Or matchers to avoid duplication in commutative ops. But what are you trying to avoid with the other changes? Are you certain that we won't see constants on the LHS on the ADD/AND ops? Canonicalization isn't always done in time - especially in general code like getKnownBitsFromAndXorOr.
https://github.com/llvm/llvm-project/pull/96763
    
    
More information about the llvm-commits
mailing list