[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