[clang-tools-extra] [clang] [llvm] [VPlan] Consistently use (Part, 0) for first lane scalar values (PR #80271)

via cfe-commits cfe-commits at lists.llvm.org
Sun Feb 4 15:43:35 PST 2024


================
@@ -526,12 +542,13 @@ bool VPInstruction::onlyFirstLaneUsed(const VPValue *Op) const {
   case Instruction::ICmp:
     // TODO: Cover additional opcodes.
     return vputils::onlyFirstLaneUsed(this);
+  case VPInstruction::ComputeReductionResult:
+    return true;
----------------
ayalz wrote:

nit: fallthrough to join other `true` opcodes below?

Can these two improvements of onlyFirstLaneUsed() (additional opcodes and additional operands) be pushed independently?

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


More information about the cfe-commits mailing list