D74436 changed the default fused FP contract which broke several
projects and it may take a while to address those issues. This
change allows setting the default fused FP contract via CMake, so
vendors can change their defaults rather than having to include the
-ffp-contract=off flag in every project to revert to the previous
behavior while the issues that were uncovered by D74436 are being
addressed.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
I just got back from vacation, so am probably missing some of the discussion elsewhere, but I'm a bit skeptical of this patch. It seems it would be confusing for users if clang generates code with different floating-point behaviour depending on how it was built.
Comment Actions
I agree. This isn't the right approach. The patch in question is currently reverted; if you'd like to argue to not restore it, that conversation should be part of that review.