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

Check that program is built once for "fused" case #697

Closed
wants to merge 1 commit into from

Conversation

bader
Copy link

@bader bader commented Dec 29, 2021

If context is shared among a root device and its subdevices, we expect
runtime library to build the program once and re-use the result to run
the program on all devices.

If context is shared among a root device and its subdevices, we expect
runtime library to build the program once and re-use the result to run
the program on all devices.
@bader bader requested a review from a team as a code owner December 29, 2021 14:38
@bader bader requested a review from steffenlarsen December 29, 2021 14:38
@bader bader marked this pull request as draft December 29, 2021 15:56
@bader
Copy link
Author

bader commented Dec 29, 2021

This should go after intel/llvm#5240.

Copy link

@steffenlarsen steffenlarsen left a comment

Choose a reason for hiding this comment

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

LGTM!

@bader bader marked this pull request as ready for review February 8, 2022 16:49
@vladimirlaz
Copy link

@bader it looks like the updated test is failing in CI. They were restated yesterday.

@bader
Copy link
Author

bader commented Feb 11, 2022

@vladimirlaz, did you compile tests with the compiler from intel/llvm#5240?

@vladimirlaz
Copy link

No, I didn't. Sorry I missed your comment.

@bader bader closed this Feb 25, 2022
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.

3 participants