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
[ESIMD] Doxygen update part I - fix structure, filter non-public APIs. #5383
Conversation
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.
Overall LGTM, minor comments can be resolved post-commit.
sycl/include/sycl/ext/intel/experimental/esimd/emu/detail/esimd_emulator_device_interface.hpp
Outdated
Show resolved
Hide resolved
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.
LGTM
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, address pre-commit failures.
@sndmitriev, @kychendev - please take a look. Your approval is needed for merging. |
@bader - please take a look, your comments have been addressed. |
Signed-off-by: Konstantin S Bobrovsky <konstantin.s.bobrovsky@intel.com>
…d_emulator_device_interface.hpp Co-authored-by: Alexander Batashev <alexbatashev@outlook.com>
Signed-off-by: Konstantin S Bobrovsky <konstantin.s.bobrovsky@intel.com>
Signed-off-by: Konstantin S Bobrovsky <konstantin.s.bobrovsky@intel.com>
Signed-off-by: Konstantin S Bobrovsky <konstantin.s.bobrovsky@intel.com>
78f33c9
to
d8b046a
Compare
Had to rebase to reproduce and fix test failure. |
@sndmitriev, @kychendev - please take a look. Your approval is needed for merging. |
@bader, @alexbatashev - please re-approve. Had to rebase and fix slm_init to fix test failure |
@alexbatashev, thanks for approval. Could you merge? Alexey approved this already (approval just expired due to my rebase) |
@kbobrovs I can't, but @vladimirlaz can |
@alexbatashev - thanks, pinged Vladimir |
Signed-off-by: Konstantin S Bobrovsky konstantin.s.bobrovsky@intel.com