[PATCH] D128261: [Fortran] Relax relative tolerance for FCVS tests

Steve Scalpone via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 27 00:37:46 PDT 2022


sscalpone added a comment.

My preference is that the reference output match the output of flang with the default settings.   Would that work with the gfortran + the provided FP_TOLERANCE?   The consensus was not to include fcvs in the llvm-project tree; we can revisit that decision if you'd like to start a thread in discourse or on of the weekly calls.

Regarding FP_TOLERANCE: We know that precision varies not only from compiler to compiler but also when using different command-line options.   Again, my preference would be to have the committed files match flang with its default options.    When testing other compilers, or other option sets, the appropriate value could be supplied through CMake.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D128261/new/

https://reviews.llvm.org/D128261



More information about the llvm-commits mailing list