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

[SYCL][ESIMD] Enable invoke_simd simd_mask test #1630

Merged
merged 5 commits into from
Mar 14, 2023

Conversation

sarnex
Copy link

@sarnex sarnex commented Mar 3, 2023

Add tests for all valid simd_mask types and the boolean conversion to/from them.

Main PR: intel/llvm#8524

@sarnex
Copy link
Author

sarnex commented Mar 3, 2023

/verify with intel/llvm#8524

@sarnex
Copy link
Author

sarnex commented Mar 6, 2023

Failure is as expected as it required intel/llvm change

@sarnex sarnex marked this pull request as ready for review March 6, 2023 15:48
@sarnex sarnex requested a review from a team as a code owner March 6, 2023 15:49
@v-klochkov
Copy link

/verify with intel/llvm#8524

@v-klochkov
Copy link

@sarnex - please take a look at this fail: http://llvm-ci2.intel.com:8080/job/SYCL_CI/job/intel/job/LLVM_Test_Suite_Associate_CI/536/display/redirect
[2023-03-06T21:07:16.159Z] Failed Tests (1):
[2023-03-06T21:07:16.159Z] SYCL :: InvokeSimd/Spec/simd_mask.cpp

It fails at compile time due to tuple_for_each()

@sarnex
Copy link
Author

sarnex commented Mar 9, 2023

@v-klochkov ah ok thanks, namespace problem

use of undeclared identifier 'tuple_for_each'; did you mean 'sycl::detail::boost::mp11::tuple_for_each'?

@v-klochkov
Copy link

/verify with intel/llvm#8524

@v-klochkov
Copy link

@sarnex - Jenkins/llvm-test-suite failed catastrophically. Can you please do 'git merge' to this PR (and may-be the PR in ESIMD headers as well) to restart it?

sarnex added 4 commits March 13, 2023 16:43
Add tests for all valid simd_mask types and the boolean
conversion to/from them.

Signed-off-by: Sarnie, Nick <[email protected]>
Signed-off-by: Sarnie, Nick <[email protected]>
Signed-off-by: Sarnie, Nick <[email protected]>
Signed-off-by: Sarnie, Nick <[email protected]>
@sarnex
Copy link
Author

sarnex commented Mar 13, 2023

done, lets hope it works this time

@sarnex
Copy link
Author

sarnex commented Mar 13, 2023

/verify with intel/llvm#8524

1 similar comment
@v-klochkov
Copy link

/verify with intel/llvm#8524

@v-klochkov
Copy link

v-klochkov commented Mar 13, 2023

/verify with intel/llvm#8524

It seems sometimes (often?) such command is ignored. The first one was ignored this time.
Same command from me (1 minute ago) started the CI bot.

Signed-off-by: Sarnie, Nick <[email protected]>
@sarnex
Copy link
Author

sarnex commented Mar 14, 2023

/verify with intel/llvm#8524

It seems sometimes (often?) such command is ignored. The first one was ignored this time. Same command from me (1 minute ago) started the CI bot.

weird, thanks.

@sarnex
Copy link
Author

sarnex commented Mar 14, 2023

/verify with intel/llvm#8524

@sarnex
Copy link
Author

sarnex commented Mar 14, 2023

@v-klochkov I found with /verify with doeesn't work for me, I'm missing permissions, I'll try to get it fixed. Ping me offline if you're interested

@v-klochkov
Copy link

/verify with intel/llvm#8524

v-klochkov pushed a commit to intel/llvm that referenced this pull request Mar 14, 2023
)

As per our recent discussions, bool passed into invoke_simd should
convert to simd_mask with a user-defined type. Conversely, simd_mask
will convert back to bool from inside the invoked function.

I also had to fix a couple of minor bugs in the implementation of the
DPCPP-specific changes in std::experimental::simd.

I will make a follow-up PR to update the invoke_simd spec with the
results of our recent discussion.

Test PR: intel/llvm-test-suite#1630

---------

Signed-off-by: Sarnie, Nick <[email protected]>
@v-klochkov v-klochkov merged commit 94697c4 into intel:intel Mar 14, 2023
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Mar 22, 2023
* [SYCL][ESIMD] Enable invoke_simd simd_mask test

Add tests for all valid simd_mask types and the boolean
conversion to/from them.

Signed-off-by: Sarnie, Nick <[email protected]>
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
…e#1630)

* [SYCL][ESIMD] Enable invoke_simd simd_mask test

Add tests for all valid simd_mask types and the boolean
conversion to/from them.

Signed-off-by: Sarnie, Nick <[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