-
Notifications
You must be signed in to change notification settings - Fork 130
[SYCL][ESIMD] Add test on simd load constructor for fp_extra types #797
Conversation
It makes variable name more self describe
Looks like 26b43da should be in a separate PR, with |
@yuriykoch yes, you is right, 26b43da has been reverted and these changes will be provided in a separate PR |
@sndmitriev Would you mind to take a look? |
@kbobrovs Would you mind to merge this PR? |
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.
@vasilytric could you please resolve the conflict?
@vladimirlaz Done. Let's wait CI result |
@sndmitriev I am sorry that I request approval from you third time for this PR. But could you approve this PR? |
@kbobrovs Would you merge this PR? |
@vladimirlaz , please approve and merge, I can't due to "1 change requested" |
Auto pulldown and update tc files for xmain branch on 20220325
add '/SYCL/Plugin/*level_zero*' back to fix intel#797 merge issue.
…ntel/llvm-test-suite#797) * [SYCL][ESIMD] Add test on simd load constructor for fp_extra types * [SYCL][ESIMD] Replace "dim" with "size" It makes variable name more self describe * Revert "[SYCL][ESIMD] Replace "dim" with "size"" This reverts commit 26b43da97c29a20423e16bf7f25b3922c2fcce59.
Extend coverage for existing test on simd load constructor by coverage extra floating point types.
There is no changes in the tests logic.