-
Notifications
You must be signed in to change notification settings - Fork 769
[SYCL][Driver]Enable multiple third-party targets for SYCL AOT. #18145
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
base: sycl
Are you sure you want to change the base?
[SYCL][Driver]Enable multiple third-party targets for SYCL AOT. #18145
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - are there plans on updating documentation to allow for multiple target support?
From the PR description:
I assume there should be no space between values in |
I added some extra space for readability purposes but I have removed the spaces now in the description. |
Signed-off-by: srividya sundaram <[email protected]>
Signed-off-by: srividya sundaram <[email protected]>
Signed-off-by: srividya sundaram <[email protected]>
Signed-off-by: srividya sundaram <[email protected]>
Signed-off-by: srividya sundaram <[email protected]>
Signed-off-by: srividya sundaram <[email protected]>
Signed-off-by: srividya sundaram <[email protected]>
Signed-off-by: srividya sundaram <[email protected]>
Signed-off-by: srividya sundaram <[email protected]>
68e7901
to
f17119d
Compare
This patch enables
-fsycl-targets
option to support a comma-separated list ofNVidia
andAMD
Processor Graphics values as target triple values for the SYCL AOT flow.Example: