Support compilation of Fortran source in SPEC2017 CMake macro. Add SPEC2017 CPU Exchange2 integer rate and speed tests.
Details
Details
Diff Detail
Diff Detail
- Repository
- rT test-suite
Event Timeline
Comment Actions
I am not sure why
arc diff --update D{id}
included the first commit after master branch got renamed to main, i.e. this one
https://github.com/llvm/llvm-test-suite/commit/41bb38d9cabf9d711c513787fb07f058ee513e40
It was not present in my original patch. Once the patch gets approved, I will make sure it is not there when I do the commit.
Comment Actions
Looks OK to me, @Meinersbur can accept at some point.
CMakeLists.txt | ||
---|---|---|
237 ↗ | (On Diff #317999) | This seems unrelated. |
Comment Actions
LGTM
CMakeLists.txt | ||
---|---|---|
237 ↗ | (On Diff #317999) | Looks like this is because the patch https://github.com/llvm/llvm-test-suite/commit/41bb38d9cabf9d711c513787fb07f058ee513e40 is included in this diff. Unfortunately, the master branch is not kept in sync with main. |
This comment was removed by naromero77.
[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)