Skip to content
This repository was archived by the owner on Mar 28, 2023. It is now read-only.

[ESIMD] Add a test for imul() intrinsic #1566

Merged
merged 1 commit into from
Feb 7, 2023

Conversation

v-klochkov
Copy link

@v-klochkov v-klochkov commented Feb 2, 2023

The intrinsic multiplies 32-bit scalar/vector by 32-bit scalar/vector and returns the 64-bit result as 2 32-bit scalars/vectors (with low halves and high halves).

This PR tests the changes in compiler: intel/llvm#8176

Signed-off-by: Vyacheslav N Klochkov [email protected]

The intrinsic multiplies 32-bit scalar/vector by 32-bit scalar/vector
and returns the 64-bit result as 2 32-bit scalars/vectors
(with low halves and high halves).

Signed-off-by: Vyacheslav N Klochkov <[email protected]>
@v-klochkov
Copy link
Author

/verify with intel/llvm#8176

@v-klochkov v-klochkov requested a review from fineg74 February 7, 2023 16:57
v-klochkov added a commit to intel/llvm that referenced this pull request Feb 7, 2023
Previously the operation supported only vector-by-scalar,
scalar-by-vector and scalar-by-scalar multiplication. This patch enables
vector-by-vector multiplication as well.

The corresponding LIT test PR:
intel/llvm-test-suite#1566

Signed-off-by: Vyacheslav N Klochkov <[email protected]>

Signed-off-by: Vyacheslav N Klochkov <[email protected]>
@v-klochkov
Copy link
Author

This fail is unrelated to this PR:
SYCL :: SeparateCompile/same-kernel.cpp

This fail is expected for pre-ci-linux and pre-ci-windows. This test passes with Jenkins/llvm-test-suite bot:
SYCL :: ESIMD/imulh_umulh.cpp

@v-klochkov v-klochkov merged commit f56e260 into intel:intel Feb 7, 2023
@v-klochkov v-klochkov deleted the esimd_imulh_lit branch February 7, 2023 22:58
v-klochkov added a commit to v-klochkov/llvm-test-suite that referenced this pull request Feb 14, 2023
The test was disabled due to false-alarm. The cases/PRs reporting
this test as 'unrelated fail' used _old_ compiler with _new_ test.

All such PRs had CI runs with the test failed at compilation.

Even though both the PR and LIT tests were merged at same hour,
the compiled used by CI was still old, but the test was new.
PR: intel/llvm#8176
PR-LIT: intel#1566

Signed-off-by: Vyacheslav N Klochkov <[email protected]>
v-klochkov added a commit that referenced this pull request Feb 16, 2023
The test was disabled due to false-alarm. The cases/PRs reporting
this test as 'unrelated fail' used _old_ compiler with _new_ test.

All such PRs had CI runs with the test failed at compilation.

Even though both the PR and LIT tests were merged at same hour,
the compiled used by CI was still old, but the test was new.
PR: intel/llvm#8176
PR-LIT: #1566

Signed-off-by: Vyacheslav N Klochkov <[email protected]>
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Mar 22, 2023
[ESIMD] Add a test for imul intrinsic

The intrinsic multiplies 32-bit scalar/vector by 32-bit scalar/vector
and returns the 64-bit result as 2 32-bit scalars/vectors
(with low halves and high halves).

Signed-off-by: Vyacheslav N Klochkov <[email protected]>
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
[ESIMD] Add a test for imul intrinsic

The intrinsic multiplies 32-bit scalar/vector by 32-bit scalar/vector
and returns the 64-bit result as 2 32-bit scalars/vectors
(with low halves and high halves).

Signed-off-by: Vyacheslav N Klochkov <[email protected]>
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
The test was disabled due to false-alarm. The cases/PRs reporting
this test as 'unrelated fail' used _old_ compiler with _new_ test.

All such PRs had CI runs with the test failed at compilation.

Even though both the PR and LIT tests were merged at same hour,
the compiled used by CI was still old, but the test was new.
PR: intel#8176
PR-LIT: intel/llvm-test-suite#1566

Signed-off-by: Vyacheslav N Klochkov <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants