Skip to content

Conversation

@sndmitriev
Copy link
Contributor

Signed-off-by: Sergey Dmitriev serguei.n.dmitriev@intel.com

Signed-off-by: Sergey Dmitriev <serguei.n.dmitriev@intel.com>
@sndmitriev sndmitriev requested a review from kbobrovs September 23, 2021 08:58
@sndmitriev sndmitriev marked this pull request as draft September 23, 2021 17:02
@sndmitriev sndmitriev marked this pull request as ready for review September 24, 2021 00:54
return esimd_rndu<RT, 1U>(src0, flags);
}

template <typename RT, int SZ>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add doxygen comments

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@kbobrovs
Copy link
Contributor

kbobrovs commented Sep 25, 2021

@sndmitriev, please also add a E2E test to https://github.com/intel/llvm-test-suite/tree/intel/SYCL/ESIMD and add a link here.

(removed my other note about LowerESIMD.cpp, as no changes are needed there)

@sndmitriev
Copy link
Contributor Author

sndmitriev commented Sep 27, 2021

@sndmitriev, please also add a E2E test to https://github.com/intel/llvm-test-suite/tree/intel/SYCL/ESIMD and add a link here.

(removed my other note about LowerESIMD.cpp, as no changes are needed there)

Link to the test suite's PR - intel/llvm-test-suite#477. I guess it won't pass testing because this PR has not been merged yet.

@kbobrovs kbobrovs merged commit 1717a14 into intel:sycl Sep 28, 2021
@sndmitriev sndmitriev deleted the public/sndmitriev/esimd_trunc branch September 28, 2021 04:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants