Skip to content

Revert "[SPIR-V] Reland "Translate icmp eq and icmp ne to PtrEqual an…#22615

Merged
sarnex merged 2 commits into
intel:syclfrom
Qi-Ye-079:revert_ptr_cmp
Jul 17, 2026
Merged

Revert "[SPIR-V] Reland "Translate icmp eq and icmp ne to PtrEqual an…#22615
sarnex merged 2 commits into
intel:syclfrom
Qi-Ye-079:revert_ptr_cmp

Conversation

@Qi-Ye-079

Copy link
Copy Markdown
Contributor

Revert "[SPIR-V] Reland "Translate icmp eq and icmp ne to PtrEqual and PtrNotEqual" (#22117)"

This reverts commit cd9cb00.

@Qi-Ye-079
Qi-Ye-079 marked this pull request as ready for review July 14, 2026 16:57
@Qi-Ye-079
Qi-Ye-079 requested a review from a team as a code owner July 14, 2026 16:57
@Qi-Ye-079

Copy link
Copy Markdown
Contributor Author

ping @YuriPlyakhin

@YuriPlyakhin YuriPlyakhin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, when this lands, please, also update p.6 of #7592 "Following patches are missing" section accordingly.

@YuriPlyakhin

Copy link
Copy Markdown
Contributor

and I think you need to keep // clang-format off/on for .cl file.

@YuriPlyakhin

Copy link
Copy Markdown
Contributor

[ FAILED ] MultipleDeviceCacheInstance/MultipleDeviceCacheTest.ProgramRetain/0, where GetParam() = { 0, 1, 2 } is not related to this PR. Issue: #21553

FAIL: SYCL :: Adapters/level_zero/device_scope_events.cpp (1857 of 2548) does not look related and there is an issue: #21766

@intel/llvm-gatekeepers , please, merge.

@sarnex
sarnex merged commit 99d946f into intel:sycl Jul 17, 2026
28 of 30 checks passed
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.

3 participants