[PATCH] D101597: [test-suite] New SPEC2017 macro and enhancements to run_specpp macro

[Michael Kruse via Phabricator via llvm-commits]

Meinersbur accepted this revision.
Meinersbur added inline comments.
This revision is now accepted and ready to land.


================
Comment at: External/SPEC/SpecCPU2017.cmake:371
 macro(speccpu2017_run_specpp)
-  cmake_parse_arguments(_arg "" "" "SPECPP_SRCS;SPECPP_DEFS" ${ARGN})
+  cmake_parse_arguments(_arg "" "SPECPP_TARGET" "SPECPP_SRCS;SPECPP_DEFS" ${ARGN})
   set(_specpp_bin ${TEST_SUITE_SPEC2017_ROOT}/bin/specpp)
----------------
[suggestion] Argument could be more self-explanatory or documented. Suggestion: `ADD_SOURCES_TO_TARGET`

(The `SPECPP_` prefixes seem unnecessary, these are function arguments that don't clash with any namespace)


Repository:
  rT test-suite

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D101597/new/

https://reviews.llvm.org/D101597