Skip to content
This repository was archived by the owner on Mar 28, 2023. It is now read-only.

[SYCL] Fix gcc-style %sycl_options on Windows #1306

Merged
merged 1 commit into from
Oct 4, 2022

Conversation

aelovikov-intel
Copy link

On windows, -lsycl have to be changed to -lsycl6 for the gcc-style invocations. Note that clang-cl style had been updated sycl.lib -> sycl6.lib before.

I originally tried to get rid of explicit library name by using "-fsycl" instead but it would break some tests for explicit manual compilation.

On windows, -lsycl have to be changed to -lsycl6 for the gcc-style
invocations. Note that clang-cl style had been updated sycl.lib ->
sycl6.lib before.

I originally tried to get rid of explicit library name by using "-fsycl"
instead but it would break some tests for explicit manual compilation.
@aelovikov-intel aelovikov-intel marked this pull request as ready for review October 4, 2022 19:36
@aelovikov-intel aelovikov-intel requested a review from a team as a code owner October 4, 2022 19:36
@againull againull merged commit e440ea2 into intel:intel Oct 4, 2022
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Mar 22, 2023
On windows, -lsycl have to be changed to -lsycl6 for the gcc-style
invocations. Note that clang-cl style had been updated sycl.lib ->
sycl6.lib before.

I originally tried to get rid of explicit library name by using "-fsycl"
instead but it would break some tests for explicit manual compilation.
aelovikov-intel added a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
…1306)

On windows, -lsycl have to be changed to -lsycl6 for the gcc-style
invocations. Note that clang-cl style had been updated sycl.lib ->
sycl6.lib before.

I originally tried to get rid of explicit library name by using "-fsycl"
instead but it would break some tests for explicit manual compilation.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants