[PATCH] D80357: [NFC] [PowerPC] Clean-up fast-math options in PPC tests
Jinsong Ji via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu May 21 06:59:09 PDT 2020
jsji requested changes to this revision.
jsji added a comment.
This revision now requires changes to proceed.
I don't think we should remove all the options from the tests.
Yes, they can be and should be replaced by instruction level flags, but the options are still supported.
So we shouldn't remove all of them from regression tests..
We can add or duplicate tests to include instruction level flags, but we should still keep testing options (until they are removed).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D80357/new/
https://reviews.llvm.org/D80357
More information about the llvm-commits
mailing list