[PATCH] D45453: [InstCombine] Enable Add/Sub simplifications with only 'reassoc' FMF
Warren Ristow via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sat Apr 14 19:24:29 PDT 2018
wristow added a comment.
> The fsub needs FMF, but not the fmul. That's the general rule we're using in FMF-enabled folds. The non-strict final value (the result of the fsub) is what we're reassociating/factoring here, so as long as that instruction has reassoc+nsz, it doesn't matter if the fmul intermediate value is strict
Got it. Thanks for explaining,
https://reviews.llvm.org/D45453
More information about the llvm-commits
mailing list