Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LoongArch64: Fixed lapack test for LA264 #5124

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

XiWeiGu
Copy link
Contributor

@XiWeiGu XiWeiGu commented Feb 12, 2025

Fixed the error when running make lapack-test with TARGET=LA264

CNClareChen and others added 7 commits February 12, 2025 14:39
Fixed register zeroing operation

Signed-off-by: Hao Chen <[email protected]>
Signed-off-by: gxw <[email protected]>
Fixed incorrect register usage in instructions

Signed-off-by: gxw <[email protected]>
Fixed index retrieval issue when there are
identical maximum absolute values

Signed-off-by: Hao Chen <[email protected]>
Signed-off-by: gxw <[email protected]>
Fixed incorrect store operation

Signed-off-by: gxw <[email protected]>
Do not check whether the input parameters c and s are zero,
as this may cause errors with special values (same as scal).
Although OpenBLAS's own test suite doesn't catch this, it will
cause LAPACK test cases to fail.
Fixed the error when the stride is zero
When the data type is single-precision real or single-precision complex,
converting it to double precision does not prevent overflow (as exposed in LAPACK tests).
The only solution is to follow C's approach: find the maximum value in the
array and divide each element by that maximum to avoid this issue
@XiWeiGu XiWeiGu force-pushed the LoongArch64-LA264-lapack-fixed branch from 68f9c81 to 2c4a5cc Compare February 12, 2025 07:48
@martin-frbg martin-frbg added this to the 0.3.30 milestone Feb 12, 2025
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