[PATCH] D129523: [Reassociate] Enable FP reassociation via 'reassoc' and 'nsz'
Nikita Popov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 12 01:04:59 PDT 2022
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
More information about the llvm-commits
mailing list