[llvm] [SLP] Fix PoisonValue in the argument VL of setOperand. (PR #118949)

Alexey Bataev via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 6 05:19:42 PST 2024


================
@@ -3338,14 +3338,13 @@ class BoUpSLP {
       copy(OpVL, Operands[OpIdx].begin());
     }
 
-    /// Set this bundle's operand from \p VL.
-    void setOperand(ArrayRef<Value *> VL, const BoUpSLP &R,
-                    bool RequireReorder = false) {
-      VLOperands Ops(VL, R);
+    /// Set this bundle's operand from Scalars.
+    void setOperand(const BoUpSLP &R, bool RequireReorder = false) {
----------------
alexey-bataev wrote:

Better to pass first instruction as an argument rather than redo the search for it

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


More information about the llvm-commits mailing list