[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
Abe Skolnik via cfe-commits
cfe-commits at lists.llvm.org
Wed Oct 5 09:45:54 PDT 2016
> Can you expand on how you plan to add the second set of reference outputs?
I plan to {either re-target or replace with normal files} the new symlinks in upcoming patches.
The patch proposal to which you referred in the above is just "stage 1" of a large clean-up.
We [Sebastian and I] have CMake code in progress that will significantly improve the harness
WRT reference outputs.
> We already have multiple reference outputs, for example big endian vs. little endian.
Yes. I have noticed that, and made appropriate use of it in the WIP CMake code. That CMake
code is not yet ready for general inspection, but if anybody reading this wants to read what`s
new in the WIP then please send me an e-mail about that.
> I was expecting you to have done a similar thing, which doesn't involve changing every single reference file. :)
1: I _am_ doing a "similar thing", but it is not yet ready for general inspection. In my WIP
check-out of "test-suite" much is currently broken, waiting for me to fix it as part of my work
[e.g. half-done fixes and improvements]. I`m reasonably sure the already-proposed patch is
free of new breakage.
2: I didn`t change _any_ reference files in the patch proposal to which you referred in the
above; I simply changed what the Makefile harness demands in terms of its reference-output
pathnames, and added symlinks to the then-current reference output pathnames so that the
Makefile harness won`t break.
The objective of the preceding, as I mentioned earlier, is to enable us all to fix/improve the
CMake harness without breaking the Makefile harness.
> 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.
A key motivation in the relevant proposed patch is to change the Makefile harness as _little_
as _possible_. We [SP & I] are not planning to improve the Makefile harness except in such a
way as to make it "get out of the way" of the modern harness [as in the proposed patch
currently under discussion].
If the Makefile harness is going to be completely-unneeded extremely soon, then a valid
alternative is to just delete all files in the repo that are only there for the benefit of the
Makefile harness and to cease assuming that we need to preserve backwards compatibility. That
would make the relevant proposed patch obsolete.
Regards,
Abe
More information about the cfe-commits
mailing list