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

[program examples] lit test on Makefile #1397

Merged
merged 31 commits into from
Apr 25, 2024
Merged

[program examples] lit test on Makefile #1397

merged 31 commits into from
Apr 25, 2024

Conversation

denolf
Copy link
Collaborator

@denolf denolf commented Apr 24, 2024

repeat of #1387

example of makefile lit testing

@denolf denolf marked this pull request as ready for review April 24, 2024 19:51
@fifield
Copy link
Collaborator

fifield commented Apr 24, 2024

  • shouldn't need to make clean first actually I'm not sure about this, since you're using make.
  • should be using %run_on_npu on the make run line

Copy link
Contributor

github-actions bot commented Apr 24, 2024

Coverage Report

Created: 2024-04-25 02:38

Click here for information about interpreting this report.

FilenameFunction CoverageLine CoverageRegion CoverageBranch Coverage
Totals- - - -
Generated by llvm-cov -- llvm version 14.0.0

denolf and others added 2 commits April 24, 2024 20:03
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@jgmelber jgmelber merged commit 0644a0a into main Apr 25, 2024
54 checks passed
@jgmelber jgmelber deleted the makefileLitTested branch April 25, 2024 03:04
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.

4 participants