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

Nichols A. Romero via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 30 09:07:05 PDT 2021


naromero77 added inline comments.


================
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)
----------------
Meinersbur wrote:
> [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)
Would this work?

```
cmake_parse_arguments(_arg "" "TARGET" "ADD_SRCS_TO_TARGET;DEFS" ${ARGN})
```



Repository:
  rT test-suite

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

https://reviews.llvm.org/D101597



More information about the llvm-commits mailing list