Skip to content

[SYCL][NFC] Fix warnings reported by Clang #3050

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

Merged
merged 5 commits into from
Jan 18, 2021

Conversation

alexbatashev
Copy link
Contributor

No description provided.

Alexander Batashev added 4 commits January 18, 2021 13:31
* sycl:
  [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)
@alexbatashev alexbatashev requested review from smaslov-intel and a team as code owners January 18, 2021 10:38
@bader bader merged commit 2ffafb9 into intel:sycl Jan 18, 2021
alexbatashev pushed a commit to alexbatashev/llvm that referenced this pull request Jan 19, 2021
* sycl:
  [SYCL][NFC] Fix warnings reported by Clang (intel#3050)
  [SYCL] Enhance device libraries filename check when constructing llvm-link commands (intel#2944)
  [SYCL][NFC] Add vtable ABI checks (intel#3032)
  [SYCL] Cleanup the deprecated Intel attribute logic (intel#3036)
@alexbatashev alexbatashev deleted the clang_warnings branch July 28, 2021 06:44
iclsrc pushed a commit that referenced this pull request Mar 28, 2025
…3050)

Instead of checking that the image operands are equal to the LOD mask,
check that the LOD bit is in the image operands. This way an LOD value
of zero may be ignored even when image operands contains bits other than
the LOD bit, such as for the SPIR-V 1.6 nontemporal image operand.

fixes #3049

Original commit:
KhronosGroup/SPIRV-LLVM-Translator@4b18b19832697fa
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