[PATCH] D107004: Turn off fp reassociation in IA intrinsic header files.
Simon Pilgrim via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jul 29 03:11:23 PDT 2021
RKSimon requested changes to this revision.
RKSimon added a comment.
This revision now requires changes to proceed.
I really don't support this change - -ffast-math should be allowed to perform aggressive folds and it makes no sense to make an exception for common intrinsics like these.
We should just ensure the intrinsics are suitably documented to warn that they are affected by fast math flags like any other operation.
An alternative, if you have specific (and common) use cases, would be to add an opt-in command line switch that disables fast math flags on intrinsics.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D107004/new/
https://reviews.llvm.org/D107004
More information about the cfe-commits
mailing list