[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