diff options
author | Andy Kaylor <andrew.kaylor@intel.com> | 2024-04-29 13:22:08 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-04-29 13:22:08 -0700 |
commit | 8ba880b587074ad6c8624ed45ea2b289f653667f (patch) | |
tree | 2bbfd2c5486e4bb12690a8c0311d0e343bb113e0 /clang/lib/CodeGen/CodeGenModule.cpp | |
parent | e5907c88594133e45b8d1d4e29702b96d7f45c77 (diff) | |
download | llvm-8ba880b587074ad6c8624ed45ea2b289f653667f.zip llvm-8ba880b587074ad6c8624ed45ea2b289f653667f.tar.gz llvm-8ba880b587074ad6c8624ed45ea2b289f653667f.tar.bz2 |
[Driver] Clean up denormal handling with fast-math-related options (#89477)
This change cleans up the clang driver handling of umbrella options like
-ffast-math, -funsafe-math-optimizations, and -ffp-model, and aligns the
behavior of -ffp-model=fast with -ffast-math with regard to the linking
of crtfastmath.o.
We agreed in a previous review that the fast-math options should not
attempt to change the -fdenormal-fp-math option, which is inherently
target-specific.
The clang user's manual claims that -ffp-model=fast "behaves identically
to specifying both -ffast-math and -ffp-contract=fast." Since
-ffast-math causes crtfastmath.o to be linked if it is available, that should
also happen with -ffp-model=fast.
I am going to be proposing further changes to -ffp-model=fast,
decoupling it from -ffast-math and introducing a new
-ffp-model=aggressive that matches the current behavior, but I wanted
to solidfy the current behavior before I do that.
Diffstat (limited to 'clang/lib/CodeGen/CodeGenModule.cpp')
0 files changed, 0 insertions, 0 deletions