Added WaveOps tests for QuadReadAcrossY

[kcloudy0717]

PR for QuadReadAcrossY tests, resolves #883.

[kcloudy0717]

Pinging @bob80905 and @farzonl for review.

[bob80905]

LGTM, I won't block on the nit, but it would be nice to update the comments on both readacross x / y

[bob80905]

Just realized there are test failures caused by read across Y. Please file the appropriate issues and add the XFAILS.

[kcloudy0717]

Just realized there are test failures caused by read across Y. Please file the appropriate issues and add the XFAILS.

Will do. Usually I delay filling any XFAILs until reported by workflows, which requires someone from the team to approve the workflow runs.

I'll go over them and file appropriate issues. Thanks for the heads up.

[kcloudy0717]

@bob80905 I updated tests with proper XFAILs. I do have one question: for the issues I reused the ones from QuadReadAcrossX because they are identical, do we want to create new issues or can we reuse the old ones?

[bob80905]

@bob80905 I updated tests with proper XFAILs. I do have one question: for the issues I reused the ones from QuadReadAcrossX because they are identical, do we want to create new issues or can we reuse the old ones?

I think we can reuse the issue, though I would recommend editing the issue to include Y as well.

[kcloudy0717]

@bob80905 Updated XFAILs and related github issues. This PR is good to go for me, whenever you're ready, feel free to merge.

[inbelic]

It looks like we might need to mark this as failing for WARP: https://github.com/llvm/offload-test-suite/actions/runs/23505742729.

Although interestingly, the pr checks seem to indicate that it was passing. grep for QuadReadAcrossY.fp64.test in the workflow logs.

It is thus not clear if the failure is intermittent or not

[kcloudy0717]

Hmmm, interesting. Either way I'll go ahead a file an issue and open a PR with XFAIL for this test. We can always close it later if it was intermittent .

[kcloudy0717]

Issue:#1013
PR: #1014

[inbelic]

Awesome, thanks!