Skip to content

[Driver][SYCL] Set default device code split mode to off for FPGA #4893

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
Nov 8, 2021

Conversation

qichaogu
Copy link
Contributor

@qichaogu qichaogu commented Nov 4, 2021

The Driver passes default device code split mode (-split=auto) to
sycl-post-link when -fsycl-device-code-split is not set. This should
not be applied for FPGA, because FPGA is using SYCL_EXTERNAL for
some of their library configurations but auto device code split mode
impacts this behavior.

Signed-off-by: Qichao Gu qichao.gu@intel.com

Signed-off-by: Qichao Gu <qichao.gu@intel.com>
@qichaogu qichaogu marked this pull request as ready for review November 4, 2021 06:12
@mdtoguchi
Copy link
Contributor

This looks the same as #4447 - I would assume you just want to use this one instead.

@qichaogu
Copy link
Contributor Author

qichaogu commented Nov 5, 2021

This looks the same as #4447 - I would assume you just want to use this one instead.

Yes, I closed #4447 and use this one.

@qichaogu
Copy link
Contributor Author

qichaogu commented Nov 8, 2021

@intel/llvm-gatekeepers Could you help merge?

@dm-vodopyanov dm-vodopyanov merged commit bea72e6 into intel:sycl Nov 8, 2021
@qichaogu qichaogu deleted the fpga-device-code-split branch November 9, 2021 06:26
mlychkov added a commit to mlychkov/llvm-test-suite that referenced this pull request Dec 1, 2021
Patch intel/llvm#4893 turned off device code splitting
for FPGA target by default. That in turn unblocked use of functions from device
libraries.

Signed-off-by: Mikhail Lychkov <mikhail.lychkov@intel.com>
vladimirlaz pushed a commit to intel/llvm-test-suite that referenced this pull request Dec 5, 2021
Patch intel/llvm#4893 turned off device code splitting
for FPGA target by default. That in turn unblocked use of functions from device
libraries.

Signed-off-by: Mikhail Lychkov <mikhail.lychkov@intel.com>
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
…te#592)

Patch intel#4893 turned off device code splitting
for FPGA target by default. That in turn unblocked use of functions from device
libraries.

Signed-off-by: Mikhail Lychkov <mikhail.lychkov@intel.com>
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.

4 participants