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

Enable github actions #94

Merged

Conversation

michalpasztamobica
Copy link

Checkout and build release_90 branch on every push to the branch.
This PR depends on flang-compiler/llvm#87 as it relies on pre-built llvm artifacts instead of building them on its own (this saves about 1h and avoids build duplication).
The resulting build directory is later uploaded as an artifact to be used in the process of building flang repository.
An example build action can be found here.

FYI, @RichBarton-Arm

Copy link
Collaborator

@RichBarton-Arm RichBarton-Arm left a comment

Choose a reason for hiding this comment

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

This all LGTM from a quick look. We have examples of this running well in Michal's fork so I am happy to approve on that basis.

@gklimowicz
Copy link

I don't have an environment set up right now to evaluate this. I am removing myself from the list of reviewers to keep from holding this up.

Copy link

@bryanpkc bryanpkc left a comment

Choose a reason for hiding this comment

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

LGTM.

Checkout and build release_90 branch on every push.
Rely on llvm repository artifacts, instead of buidling them.
Upload artifacts for flang repository.
@kiranchandramohan kiranchandramohan merged commit e6d8988 into flang-compiler:release_90 Nov 3, 2020
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.

6 participants