-
Notifications
You must be signed in to change notification settings - Fork 787
[SYCL][ESIMD] Add conversion ctor and operator #3028
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
Conversation
@kbobrovs, a test failed. However, after looking into the detail, the failure seems not relevant to esimd. What should I do in this case? |
@vladimirlaz, jenkins test failed. However, looking at the error details, I don't think it is related to this PR. How can we resolve this? |
Looks like infrastructure failure. I restarted the build. I think every PR submitter should be able to do this - click 'details' for the web page where rebuild can be started |
} | ||
|
||
bool test_replicate3() __attribute__((sycl_device)) { | ||
simd<int, 8> v0(0, 1); | ||
auto v0_rep = v0.replicate<2, 4, 2, 2>(1); | ||
|
||
return v0_rep[0] == v0[1] && v0_rep[1] == v0[3] && | ||
v0_rep[2] == v0[5]; | ||
return v0_rep[0] == v0[1] && v0_rep[1] == v0[3] && v0_rep[2] == v0[5]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please add a testcase for %=
are you going to create executable tests for new esimd functionality you add?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
test added. Yes, I will modify some existing examples to use the new esimd functionality.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kychendev @kbobrovs please review and merge timely
Signed-off-by: Gang Y Chen <gang.y.chen@intel.com>
c23d5dc
to
fda92f9
Compare
* sycl: (378 commits) [sycl-post-link][NFC] Extracted the code into a subroutine (intel#3042) [SYCL][NFC] Remove commented out code (intel#3029) [CODEOWNERS] Fix ownership of DPC++ tools tests (intel#3047) [SYCL][NFC] Make tests insensitive to dso_local (intel#3037) [SYCL] Fix acquiring a mutex in _pi_context::finalize (intel#3001) [SYCL] Fix various compilation warnings in plugins (intel#2979) [SYCL][ESIMD] Add simd class conversion ctor and operator (intel#3028) [sycl-post-link][NFC] Use range-based for loop. (intel#3033) [SYCL][NFC] Fix warning in self-build (intel#3023) [NFC] Fix sycl-post-link tests to avoid potential race (intel#3031) [SYCL][CUDA] Add missing barrier to collectives (intel#2990) [SYCL] Make Intel attributes consistent with clang attributes. (intel#3022) [SYCL] Bump SYCL minor version (intel#3026) [SYCL][Doc] Added requirement on reference to test PR in commit message (intel#3010) [SYCL] Put constant initializer list data in non-generic addr space. (intel#3005) [SYCL][L0] Fix memory leak in PiDeviceCache and ZeCommandList (intel#2974) [SYCL] Fix detection of free function calls (intel#3003) [SYCL][NFC] Clean up the builder_dir argument description (intel#3021) [SYCL][ESIMD] Fix LowerESIMD crash on a scalar fptoui LLVM instruction (intel#2699) [NFC] Remove redundant call to getMainExecutable() (intel#3018) ...
Signed-off-by: Gang Y Chen gang.y.chen@intel.com