[llvm-dev] a proposed script to help with test-suite programs that output _lots_ of FP numbers
Renato Golin via llvm-dev
llvm-dev at lists.llvm.org
Thu Sep 29 12:05:20 PDT 2016
On 29 September 2016 at 19:21, Sebastian Pop <sebpop.llvm at gmail.com> wrote:
> Cumulating errors is a bad idea.
> As others have suggested, please prepare a patch that disables
> fp-contract on those testcases.
No, please, let's not disable things just because they fail.
If the test is not meaningful or if the results are not good, let's
just change the test in a meaningful way that can work with any FP
optimisation without changing meaning.
If it does change meaning, it's a bug and we *want* to catch.
cheers,
--renato
More information about the llvm-dev
mailing list