[llvm] [InstCombine] Fix the correctness of missing check reassoc attribute (PR #71277)
    Matt Arsenault via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Nov  6 22:44:29 PST 2023
    
    
  
================
@@ -607,176 +781,9 @@ Instruction *InstCombinerImpl::visitFMul(BinaryOperator &I) {
   if (Value *V = SimplifySelectsFeedingBinaryOp(I, Op0, Op1))
     return replaceInstUsesWith(I, V);
 
-  if (I.hasAllowReassoc()) {
----------------
arsenm wrote:
Oh right, I'm still getting confused by the PR interface. I would just pre-push the first commit 
https://github.com/llvm/llvm-project/pull/71277
    
    
More information about the llvm-commits
mailing list