[PATCH] D95023: [test-suite] SPEC2017 CPU Exchange2 integer tests.

[Michael Kruse via Phabricator via llvm-commits]

Meinersbur added inline comments.


================
Comment at: External/SPEC/CINT2017rate/CMakeLists.txt:8-15
 add_subdirectory(541.leela_r)      # C++
-#add_subdirectory(548.exchange2_r) # Fortran
 add_subdirectory(557.xz_r)         # C
 
 add_subdirectory(999.specrand_ir)  # C
+
+if(TEST_SUITE_FORTRAN)
+add_subdirectory(548.exchange2_r) # Fortran
----------------
[style] Missing indention

The other tests are sorted numerically and I would prefer to keep it that way (a directories listing usually is sorted that way as well)


================
Comment at: External/SPEC/CINT2017speed/CMakeLists.txt:14
+if(TEST_SUITE_FORTRAN)
+add_subdirectory(648.exchange2_s) # Fortran
+endif()
----------------
[style] missing indention


Repository:
  rT test-suite

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

https://reviews.llvm.org/D95023