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.

Sign up for GitHub

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

Jump to bottom

Add a CI build for verifying PRs #166

Merged
merged 43 commits into from
Jul 18, 2023
Merged

Add a CI build for verifying PRs #166

merged 43 commits into from
Jul 18, 2023

Conversation

dpaoliello
Copy link
Collaborator

  • Adds a new GitHub Action that is triggered for PRs.
  • Verifies on Linux (GCC), MacOS (GCC) and Windows (MSVC).
  • Linux and MacOS use a prebuilt LLVM, Windows builds its own (as llvm-dev is not available for Windows).
  • Added caching for the Windows build (reduces build time from ~2hrs to 20mins).
  • Fixed test failure (caused by truncating pointers to char and then sign-extending back to int).

Sample successful run: https://github.com/JuliaHubOSS/llvm-cbe/actions/runs/5590403992

Fixes #165

@dpaoliello
Add a GitHub Action
2f3963c
@dpaoliello
Build when pushing to branch
b0c19d5
@dpaoliello
Also build unit test, fix links on Linux
dc71730
@dpaoliello
Remove windows build
4026203
@dpaoliello
Fix macos build, add back windows build
4025fb9
@dpaoliello
fix yaml syntax
977f877
@dpaoliello
Add pytest to path
a38a0df
@dpaoliello
Fix setting pip path, fix syntax for llvm checkout
7eb1bf3
@dpaoliello
Install pytest to known path, fix windows build
ed70785
@dpaoliello
Fix pytest install, fix cmake configuration
56b13ee
@dpaoliello
Fix setting python path
7980024
@dpaoliello
Print env, build tablegen
905a02f
@dpaoliello
Fix table-gen target, try to use latest python for pip
7e2b2dc
@dpaoliello
Update python and pip
2f33e7c
@dpaoliello
Use GCC 11 on MacOS
fcae39e
@dpaoliello
Alias gcc just before running tests, simplify windows build
f27351d
@dpaoliello
Use a symlink instead of alias for gcc
0bbb273
@dpaoliello
Slim Windows build, use less test workers
c4bfc8a
@dpaoliello
Use a VS Shell for Windows to avoid GCC
0cd6cac
@dpaoliello
Always target x64 for windows
3e55394
@dpaoliello
Change directory back to workspace dir
ae1496e
@dpaoliello
Cache llvm build, set env for test on windows
0100a56
@dpaoliello
Capture runner image version
2bf036e
@dpaoliello
Add branch to cache key, but fallback to main
27cacd0
@dpaoliello
print env
8c2b235
@dpaoliello
Fix imagever capture
82d28bc
@dpaoliello
Use correct arg for add-content
fa73409
@dpaoliello
use correct syntax to grab the env var
557787f
@dpaoliello
Use correct ref name for main branch
838d1fd
@dpaoliello
Force llvm-cbe to be rebuilt
b2df588
@dpaoliello dpaoliello requested a review from vtjnash July 18, 2023 18:17
@vtjnash vtjnash enabled auto-merge (squash) July 18, 2023 18:32
@vtjnash vtjnash merged commit d922151 into master Jul 18, 2023
3 checks passed
@vtjnash vtjnash deleted the actions branch July 18, 2023 20:39
This was referenced Jul 18, 2023
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vtjnash vtjnash vtjnash approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

RFC: Add CI validation for PRs
2 participants
@dpaoliello @vtjnash