[PATCH] D25254: test-suite: Change extension used for reference outputs by Makefile-based harness so we can start improving how the CMake-based harness works without breaking the old system or the buildbots

Renato Golin via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 5 02:50:24 PDT 2016


rengolin added a comment.

Can you expand on how you plan to add the second set of reference outputs?

We already have multiple reference outputs, for example big endian vs. little endian. I was expecting you to have done a similar thing, which doesn't involve changing every single reference file. :)

One way this could be simpler is to change the Makefile on each affected directories to duplicate the tests & check differently against the reference file. It can be the same reference, but with different thresholds for different fp-contract options.


https://reviews.llvm.org/D25254





More information about the cfe-commits mailing list