Skip to content

Conversation

Naghasan
Copy link
Contributor

For non SPIR target, offload-deps ignored the offloading target arch.
This patch set the associated arch to the actions,
properly cache the input info when building the job and
properly set the -targets flag to be aligned with the bundler.

Signed-off-by: Victor Lomuller [email protected]

@Naghasan Naghasan force-pushed the victor/fix-offload-deps-target branch 2 times, most recently from c2422b0 to e0e492f Compare September 1, 2021 07:54
Copy link
Contributor

@AGindinson AGindinson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mostly LGTM:

Naghasan and others added 2 commits September 2, 2021 17:37
For non SPIR target, offload-deps ignored the offloading target arch.
This patch set the associated arch to the actions,
properly cache the input info when building the job and
properly set the `-targets` flag to be aligned with the bundler.

Signed-off-by: Victor Lomuller <[email protected]>
@Naghasan Naghasan force-pushed the victor/fix-offload-deps-target branch from d3833fe to 5687526 Compare September 2, 2021 15:39
Copy link
Contributor

@AGindinson AGindinson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Copy link
Contributor

@AGindinson AGindinson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@bader bader added cuda CUDA back-end hip Issues related to execution on HIP backend. labels Sep 3, 2021
@bader bader merged commit 3c7971d into intel:sycl Sep 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cuda CUDA back-end hip Issues related to execution on HIP backend.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants