[PATCH] D45710: Fast Math Flag mapping into SDNode

[Michael Berg via Phabricator via llvm-commits]

mcberg2017 added inline comments.


================
Comment at: lib/Target/AMDGPU/SIISelLowering.cpp:6700
   if ((Options.AllowFPOpFusion == FPOpFusion::Fast || Options.UnsafeFPMath ||
-       (N0->getFlags().hasUnsafeAlgebra() &&
-        N1->getFlags().hasUnsafeAlgebra())) &&
+       (N0->getFlags().hasAllowReassociation() &&
+        N1->getFlags().hasAllowReassociation())) &&
----------------
rampitec wrote:
> Production of a fused fma opcode is not tied to reassociation. The question here is legality of intermediate result rounding.
Stanislav, what change if any do you request then for this code?


https://reviews.llvm.org/D45710