[llvm] [SLP] Make getSameOpcode support different instructions if they have same semantics. (PR #112181)
    Alexey Bataev via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Oct 21 17:00:29 PDT 2024
    
    
  
================
@@ -14935,7 +15108,7 @@ Value *BoUpSLP::vectorizeTree(TreeEntry *E, bool PostponedPHIs) {
       Value *V = Builder.CreateBinOp(
           static_cast<Instruction::BinaryOps>(E->getOpcode()), LHS,
           RHS);
-      propagateIRFlags(V, E->Scalars, VL0, It == MinBWs.end());
+      propagateIRFlags(V, E->Scalars, nullptr, It == MinBWs.end());
----------------
alexey-bataev wrote:
I think this part still requires extra work. If the opcode of the instruction does not match the opcode of intersection, its flags are ignored. This is not correct and must be fixed
https://github.com/llvm/llvm-project/pull/112181
    
    
More information about the llvm-commits
mailing list