Skip to content

[USM] Align OpenCL USM extension header with the specification #1162

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 1 commit into from
Feb 24, 2020

Conversation

bader
Copy link
Contributor

@bader bader commented Feb 22, 2020

Fix the cl_device_unified_shared_memory_capabilities_intel bitfield type
name.

Signed-off-by: Alexey Bader alexey.bader@intel.com

Fix the cl_device_unified_shared_memory_capabilities_intel bitfield type
name.

Signed-off-by: Alexey Bader <alexey.bader@intel.com>
@bader bader requested review from jbrodman and bashbaug February 22, 2020 14:01
Copy link
Contributor

@jbrodman jbrodman left a comment

Choose a reason for hiding this comment

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

LGTM - good catch.

Copy link
Contributor

@bashbaug bashbaug left a comment

Choose a reason for hiding this comment

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

Thanks, this looks fine for now.

I have a PR out to add USM to the standard OpenCL headers:

KhronosGroup/OpenCL-Headers#65

Once that is approved we should switch to the support in the standard headers vs. duplicating the USM definitions here.

@bader bader merged commit a0c0e33 into intel:sycl Feb 24, 2020
@bader bader deleted the usm-type-fix branch February 24, 2020 15:59
alexbatashev pushed a commit to alexbatashev/llvm that referenced this pull request Feb 25, 2020
…ages_docs

* origin/sycl: (1092 commits)
  [CI] Add clang-format checker to pre-commit checks (intel#1163)
  [SYCL][CUDA] Initial CUDA backend support (intel#1091)
  [USM] Align OpenCL USM extension header with the specification (intel#1162)
  [SYCL][NFC] Fix unreferenced variable warning (intel#1158)
  [SYCL] Fix __spirv_GroupBroadcast overloads (intel#1152)
  [SYCL] Add llvm/Demangle link dependency for llvm-no-spir-kernel (intel#1156)
  [SYCL] LowerWGScope pass should not be skipped when -O0 is used
  [SYCL][Doc][USM] Add refactored pointer and device queries to USM spec (intel#1118)
  [SYCL] Update the kernel parameter rule to is-trivially-copy-construc… (intel#1144)
  [SYCL] Move internal headers to source dir (intel#1136)
  [SYCL] Forbid declaration of non-const static variables inside kernels (intel#1141)
  [SYCL][NFC] Remove idle space (intel#1148)
  [SYCL] Improve the error mechanism of llvm-no-spir-kernel (intel#1068)
  [SYCL] Added CTS test config (intel#1063)
  [SYCL] Implement check-sycl-deploy target (intel#1142)
  [SYCL] Preserve original message and code of kernel/program build result (intel#1108)
  [SYCL] Fix LIT after LLVM change in community
  Translate LLVM's cmpxchg instruction to SPIR-V
  Add volatile qualifier for atom_ builtins
  Fix -Wunused-variable warnings
  ...
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