[PATCH] D39506: Add a script to run various benchmarks and send the result to lnt

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 13 04:50:06 PST 2017


jhenderson added a comment.

I just tried running this locally without using cset, and I got an error from the '--' part of the command, which presumably is intended for use with cset. Removing it would cause cset to fail, if used as you described, but it looks like it is possible to do "--wrapper sudo,cset,shield,-e,--" and still get the expected result, even though the -e is in the seemingly wrong place. Note: I have not verified that in this case the run is definitely shielded, although it looks likely.

One feature request, that doesn't need doing in the first iteration, is for the ability to print the summarised results to stdout optionally instead of posting to LNT. At the moment, this script requires LNT to run, which for a casual user who just wants to quickly compare two sets of times may not be the best situation.


https://reviews.llvm.org/D39506





More information about the llvm-commits mailing list