[llvm] InstCombine: Clean up SimplifyDemandedFPClass use context application (PR #176886)
Yingwei Zheng via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 20 08:28:07 PST 2026
================
@@ -2375,12 +2358,6 @@ Value *InstCombinerImpl::SimplifyDemandedUseFPClass(Instruction *I,
SimplifyDemandedFPClass(I, 0, SrcDemandedMask, KnownLHS, Depth + 1))
return I;
- // Propagate nnan-ness to sources to simplify source checks.
- if ((DemandedMask & fcNan) == fcNone) {
- KnownLHS.knownNot(fcNan);
- KnownRHS.knownNot(fcNan);
- }
-
if (FMF.noInfs()) {
// Flag implies inputs cannot be infinity.
KnownLHS.knownNot(fcInf);
----------------
dtcxzyw wrote:
Do we still need this propagation? FMF -> DemandedMask -> SrcDemandedMask -> nested call to SimplifyDemandedFPClass(DemandedMask -> Known) -> KnownLHS/RHS
https://github.com/llvm/llvm-project/pull/176886
More information about the llvm-commits
mailing list