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

Renato Golin via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 26 10:00:40 PDT 2016


rengolin requested changes to this revision.
rengolin added a comment.
This revision now requires changes to proceed.

I'm not sure what is the point of this patch. Can you describe what's broken and why this fixes it?

If this is related to the other patch reverted, than, as I said there, this needs a wider discussion and I'd be very surprised if we couldn't fix whatever the problems are on ARM instead of just completely disabling it and making ARM a different path from other architectures.

I really don't want us to start segregating the implementations per architecture, especially when nothing is broken yet.

cheers,
--renato


https://reviews.llvm.org/D24925





More information about the llvm-commits mailing list