[PATCH] D102621: [X86] Pre-commit test for D90901

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 20 05:44:00 PDT 2021


spatel added inline comments.


================
Comment at: llvm/test/CodeGen/X86/fma-signed-zero.ll:4-5
+; RUN:   | FileCheck %s --check-prefixes=NO-NSZ-OPTION
+; RUN: llc < %s -mtriple=x86_64-unknown-linux-gnu -mattr=+fma \
+; RUN:   --enable-no-signed-zeros-fp-math | FileCheck %s --check-prefixes=NSZ-OPTION
+
----------------
We are moving away from global options that control FP state. Remove this RUN, and only use IR-level "nsz" fast-math-flags to show the differences. 

If I'm seeing it correctly, the tests are already duplicated with/without the FMF, so this will only reduce the FileCheck lines.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102621



More information about the llvm-commits mailing list