Skip to content

[NFC][SYCL] Clean up FPGA archive files after test runs #2678

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 5 commits into from
Oct 28, 2020

Conversation

AGindinson
Copy link
Contributor

@AGindinson AGindinson commented Oct 23, 2020

In some local configurations, re-launching fpga_aocx*.cpp tests
appends new objects to the archive created during preceding test
runs. Overwrite the archive files in a separate RUN line to avoid
misleading failures in local development environments.

Signed-off-by: Artem Gindinson [email protected]

Artem Gindinson added 2 commits October 23, 2020 20:01
In some local configurations, re-launching fpga_aocx*.cpp tests
appends new objects to the archive created during preceding test
runs. Overwrite the archive files in a separate `RUN` line to avoid
misleading failures in local development environments.

Signed-off-by: Artem Gindinson <[email protected]>
Signed-off-by: Artem Gindinson <[email protected]>
@AGindinson AGindinson requested a review from a team as a code owner October 23, 2020 17:44
@AGindinson AGindinson force-pushed the private/agindins/aocx-test-cleanup branch from 35ea990 to b695de7 Compare October 23, 2020 20:07
Artem Gindinson added 2 commits October 26, 2020 06:15
Signed-off-by: Artem Gindinson <[email protected]>
@AGindinson AGindinson requested a review from mdtoguchi October 26, 2020 06:21
Signed-off-by: Artem Gindinson <[email protected]>
@AGindinson AGindinson requested a review from s-kanaev October 26, 2020 10:55
@AGindinson
Copy link
Contributor Author

@intel/llvm-reviewers-runtime Ping

@romanovvlad romanovvlad merged commit 750c979 into intel:sycl Oct 28, 2020
@AGindinson AGindinson deleted the private/agindins/aocx-test-cleanup branch October 30, 2020 06:56
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.

5 participants