[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