[PATCH] D36717: [test-suite] Add SPEC CPU 2017

Michael Kruse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 17 17:55:59 PDT 2017


Meinersbur added a comment.

One more remark: The 'ref' dataset of 638.imagick_s on all the computers I tried on took between 2 and 3 hours. Submitted results to SPEC <https://www.spec.org/cpu2017/results/res2017q2/cpu2017-20161026-00017.html> are in a similar range. Unfortunately `timeit.py` has a 7200 seconds (2 hours) limit hardcoded (for `--limit-cpu` __and__ `--timeout`). I had to remove that limit to be able to have a successful run.

Can we make the limit configurable or set higher?



================
Comment at: External/SPEC/CFP2017rate/519.lbm_r/CMakeLists.txt:9
+
+link_libraries(-lm)
+
----------------
MatzeB wrote:
> Most parts of the test-suite use `list(APPEND LDFLAGS -lm)` for this (that's not better but maybe more consistent)
I assumed that CPPFLAGS, CFLAGS and LDFLAGS are there only from the automatic upgrade from the Makefiles. The SPEC2006 also mostly use include_directories/add_definitions instead of C(PP)FLAGS. I stayed consistent with that.

The handling of `LDFLAGS` in `llvm_test_executable` is mmmh... ugly. I also need the flags applied on the image_validator and ldecod from x264. `link_libraries` was practical here because it also applied to these executables.


https://reviews.llvm.org/D36717





More information about the llvm-commits mailing list