Skip to content

Reduce complex GEMM pre_checkin tests#673

Merged
leekillough merged 2 commits into
ROCm:developfrom
leekillough:reduce_tests
Aug 26, 2019
Merged

Reduce complex GEMM pre_checkin tests#673
leekillough merged 2 commits into
ROCm:developfrom
leekillough:reduce_tests

Conversation

@leekillough
Copy link
Copy Markdown
Contributor

This reduces the number of pre_checkin complex GEMM tests, by reducing the number of combinations of (alpha,alphai) and (beta,betai).

 ./rocblas-test  --gtest_filter=pre_checkin/gemm_strided_batched*

The runtime of the tests on a Vega 20 are reduced from:

[----------] Global test environment tear-down
[==========] 26945 tests from 2 test cases ran. (11615163 ms total)
[  PASSED  ] 26945 tests.

real    194m15.733s
user    178m29.709s
sys     14m23.518s

To:

[----------] Global test environment tear-down
[==========] 21585 tests from 2 test cases ran. (7257974 ms total)
[  PASSED  ] 21585 tests.

real    121m32.824s
user    111m35.696s
sys     9m3.093s

Combined with the BLIS changes (#621), this should lower the time it takes to run pre_checkin CI tests.

Copy link
Copy Markdown
Contributor

@zaliu zaliu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Contributor

@amcamd amcamd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@leekillough leekillough merged commit aca403d into ROCm:develop Aug 26, 2019
@leekillough leekillough deleted the reduce_tests branch December 27, 2019 00:29
mlse-lib-jenkins pushed a commit that referenced this pull request May 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants