Skip to content
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

[Driver][SYCL] Update archive creation with -fsycl-link for FPGA #7325

Merged
merged 1 commit into from
Nov 9, 2022

Conversation

mdtoguchi
Copy link
Contributor

When generating archives using -fsycl-link=early|image, any associated generated objects and device binaries are added to the archive for later processing. Archives can also be used in this manner, but creating the archives with 'cr' would just add the archive to the final generated archive, which does not allow the final archive to be used for host linking.

Update the archive generation by using 'cqL' for llvm-ar, as that allows for any archives that are pulled in to be added via the contents instead of the archives themselves.

When generating archives using -fsycl-link=early|image, any associated
generated objects and device binaries are added to the archive for later
processing.  Archives can also be used in this manner, but creating the
archives with 'cr' would just add the archive to the final generated
archive, which does not allow the final archive to be used for host linking.

Update the archive generation by using 'cqL' for llvm-ar, as that allows
for any archives that are pulled in to be added via the contents instead of
the archives themselves.
@mdtoguchi mdtoguchi requested a review from a team as a code owner November 9, 2022 00:48
@steffenlarsen
Copy link
Contributor

Unexpected pass in regression/tanh_fix_test.cpp and failure in Regression/device_num.cpp are known issues.

@steffenlarsen steffenlarsen merged commit 3159db5 into intel:sycl Nov 9, 2022
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