Skip to content

[SYCL] Refactor -fsycl-esimd and -fsycl-int-header options #2466

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
Sep 14, 2020

Conversation

bader
Copy link
Contributor

@bader bader commented Sep 12, 2020

-fsycl-int-header option initialization code moved closer to code
section processing other SYCL options.

-fsycl-esimd enables optimizations in SYCL mode, which is not necessary
after 5976ff0

-fsycl-int-header option initialization code moved closer to code
section processing other SYCL options.

-fsycl-esimd enables optimizations in SYCL mode, which is not necessary
after intel@5976ff0
@bader bader merged commit dc5e614 into intel:sycl Sep 14, 2020
alexbatashev pushed a commit to alexbatashev/llvm that referenced this pull request Sep 15, 2020
* origin/sycl:
  [SYCL] Fixes in release notes (intel#2470)
  [SYCL] Refactor -fsycl-esimd and -fsycl-int-header options (intel#2466)
jsji pushed a commit that referenced this pull request Apr 4, 2024
It seems we currently allow OCL to SPIR-V translation only for OpenCL C sources. This PR tries to extend this to OpenCL C++ and C++ for OpenCL sources as well.

Signed-off-by: Arvind Sudarsanam <[email protected]>

Original commit:
KhronosGroup/SPIRV-LLVM-Translator@80dfd864ae55636
iclsrc pushed a commit that referenced this pull request Apr 25, 2024
…)" (#2508)

This reverts commit 80dfd864ae556369747074bbde8fc1d4c48547be.
The commit break Translator in cases when user supplied function is named
with the prefix which, after demangling, is the same as OpenCL builtin.
The culprit code is placed in during SPIRWriter OCL pass in
OCLToSPIRVBase::visitCallInst(CallInst &).
Failing name would be for example atomic_fetch_and_sub_uint32_explicit.

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