[llvm] [SDAG] Harden assumption in getMemsetStringVal (PR #126207)
Matt Arsenault via llvm-commits
llvm-commits at lists.llvm.org
Sun Feb 9 18:12:15 PST 2025
================
@@ -8015,16 +8015,8 @@ static SDValue getMemsetStringVal(EVT VT, const SDLoc &dl, SelectionDAG &DAG,
if (Slice.Array == nullptr) {
if (VT.isInteger())
return DAG.getConstant(0, dl, VT);
- if (VT == MVT::f32 || VT == MVT::f64 || VT == MVT::f128)
+ if (VT.isFloatingPoint())
return DAG.getConstantFP(0.0, dl, VT);
- if (VT.isVector()) {
- unsigned NumElts = VT.getVectorNumElements();
- MVT EltVT = (VT.getVectorElementType() == MVT::f32) ? MVT::i32 : MVT::i64;
- return DAG.getNode(ISD::BITCAST, dl, VT,
- DAG.getConstant(0, dl,
- EVT::getVectorVT(*DAG.getContext(),
- EltVT, NumElts)));
- }
----------------
arsenm wrote:
I mean instead of having this unreachable case
https://github.com/llvm/llvm-project/pull/126207
More information about the llvm-commits
mailing list