[PATCH] D142093: [InstCombine] trunc (fptoui|fptosi)
Sam Parker via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 19 03:33:56 PST 2023
samparker added inline comments.
================
Comment at: llvm/lib/Transforms/InstCombine/InstCombineCasts.cpp:506
+ // simple types while evaluating truncations.
+ if (isGuaranteedNotToBeUndefOrPoison(I->getOperand(0)))
+ if (I->getOperand(0)->getType()->getScalarType()->isHalfTy())
----------------
And I'm still not sure whether this is needed? Alive seems to want it to be happy, but I pretty sure integer transforms are performed elsewhere without considering fp-to-int conversions as inputs.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D142093/new/
https://reviews.llvm.org/D142093
More information about the llvm-commits
mailing list