[PATCH] D129523: [Reassociate] Enable FP reassociation via 'reassoc' and 'nsz'

[Nikita Popov via Phabricator via llvm-commits]

nikic added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/Reassociate.cpp:151
+  assert(I && I->getType()->isFPOrFPVectorTy() && "Should only check FP ops");
+  return I->isFast() || I->isAssociative();
+}
----------------
Just `I->isAssociative()` is sufficient, `isFast()` implies `isAssociative()`.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D129523/new/

https://reviews.llvm.org/D129523