Skip to content

Fix flipped indices in benchmark for gemv#3440

Merged
martin-frbg merged 1 commit intoOpenMathLib:developfrom
mhillenbrand:fix_gemv_indices
Nov 4, 2021
Merged

Fix flipped indices in benchmark for gemv#3440
martin-frbg merged 1 commit intoOpenMathLib:developfrom
mhillenbrand:fix_gemv_indices

Conversation

@mhillenbrand
Copy link
Copy Markdown
Contributor

@mhillenbrand mhillenbrand commented Nov 3, 2021

Fixes #3439

Tested on x86-64 and s390x with the following combinations:

./cgemv.goto
OPENBLAS_PARAM_M=200 ./cgemv.goto
OPENBLAS_PARAM_M=400 ./cgemv.goto
./zgemv.goto
OPENBLAS_PARAM_M=400 ./zgemv.goto
OPENBLAS_PARAM_M=100 ./zgemv.goto

@martin-frbg martin-frbg merged commit 3737766 into OpenMathLib:develop Nov 4, 2021
@martin-frbg martin-frbg added this to the 0.3.19 milestone Nov 4, 2021
@mhillenbrand mhillenbrand deleted the fix_gemv_indices branch November 4, 2021 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

0.3.19

Development

Successfully merging this pull request may close these issues.

benchmark/cgemv.goto and zgemv.goto hit Segmentation fault when initializing inputs (seen on x86-64 and s390x)

2 participants

@mhillenbrand @martin-frbg