[PATCH] D24925: [testsuite] turn fp-contract off for ARM because output checking is not flexible enough

Sanjay Patel via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 26 09:48:22 PDT 2016


spatel created this revision.
spatel added reviewers: Abe, rengolin, hfinkel, scanon, spop, yaxunl, MatzeB.
spatel added a subscriber: llvm-commits.
Herald added subscribers: samparker, mcrosier, rengolin, aemerson.

AFAIK, I don't have any way to test this patch locally. If someone wants to try that first to make sure I haven't botched this, that would be great. :)

Motivation: we want to reinstate D24481 / rL282259 without breaking AArch64 testsuite bots, so let's use the same hack that PowerPC is currently using to overcome the too-strict reference output checks. Ideally, we would do something like rL282027 to customize the checking per test, but let's not gate FMA codegen on that because it might require some serious investigation to get right.

https://reviews.llvm.org/D24925

Files:
  Makefile.config.in

Index: Makefile.config.in
===================================================================
--- Makefile.config.in
+++ Makefile.config.in
@@ -159,6 +159,12 @@
 endif
 endif
 
+# ARM needs -ffp-contract=off so that:
+#     The outputs match the reference outputs.
+ifeq ($(ARCH),ARM)
+TEST_TARGET_FLAGS += -ffp-contract=off
+endif
+
 #
 # Provide variables specific to llvm-test
 #


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D24925.72506.patch
Type: text/x-patch
Size: 383 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160926/d3d63994/attachment.bin>


More information about the llvm-commits mailing list