[clang] [HLSL] Make fast math the default for HLSL (PR #119820)
Sarah Spall via cfe-commits
cfe-commits at lists.llvm.org
Wed Dec 18 10:44:44 PST 2024
spall wrote:
> This looks correct, but how much work is it to update all of these tests to have fast math enabled rather than explicitly disabling it? I worry that (1) folks looking at these tests in the future will think that disabling fast math is a meaningful part of the test and (2) people will crib these run lines and add more tests that build with non-default flags for no particular reason
It's definitely more difficult, because I was able to add -fno-fast-math with a quick search and replace. Of course I'll do it the way you suggest if you think that is better.
https://github.com/llvm/llvm-project/pull/119820
More information about the cfe-commits
mailing list