[llvm] [AMDGPU] Price the packed form of a vector of i1 (PR #217327)
Frederik Harwath via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 27 06:48:28 PDT 2026
================
@@ -980,13 +980,59 @@ InstructionCost GCNTTIImpl::getCFInstrCost(unsigned Opcode,
return BaseT::getCFInstrCost(Opcode, CostKind, I);
}
+// A vector of i1 has no packed form on this target: every element lives in its
+// own mask. Measured instruction counts per element to pack the masks into an
+// integer, and to unpack them again.
+static constexpr unsigned MaskPackCostPerElt = 4;
+static constexpr unsigned MaskUnpackCostPerElt = 3;
+
+/// Returns the number of elements when \p Ty is a fixed vector of i1 with more
+/// than one element.
+static std::optional<unsigned> getPackedMaskElts(Type *Ty) {
+ auto *FVT = dyn_cast<FixedVectorType>(Ty);
+ if (FVT && FVT->getElementType()->isIntegerTy(1) && FVT->getNumElements() > 1)
+ return FVT->getNumElements();
+ return std::nullopt;
+}
+
+InstructionCost GCNTTIImpl::getCastInstrCost(unsigned Opcode, Type *Dst,
+ Type *Src,
+ TTI::CastContextHint CCH,
+ TTI::TargetCostKind CostKind,
+ const Instruction *I) const {
+ // A bitcast between a vector of i1 and an integer packs or unpacks a mask.
+ if (Opcode == Instruction::BitCast) {
+ if (std::optional<unsigned> Elts = getPackedMaskElts(Src);
+ Elts && Dst->isIntegerTy())
----------------
frederik-h wrote:
Could check isIntegerTy(*Elts)? I think the verifier would reject bitcasts if this does not match, but could it be clearer to do this?
https://github.com/llvm/llvm-project/pull/217327
More information about the llvm-commits
mailing list