Skip to content

Conversation

@meowsbits
Copy link

@meowsbits meowsbits commented Jun 4, 2020

Drafting CI pipeline installation.

As-is, the change set includes two pipeline options; only one is necessary. We can have a discussion comparing these options (Github vs. Azure), or alternatives, and then remove or change to use only what seems optimal and necessary.

Both pipelines build the llc executable and archive it as a .zip, retaining the artifact.

Latest builds on my personal fork, for reference:

Next steps for the CI pipeline could be discussed and implemented here, or in a following one or few PRs. These would probably include, but may not be limited to:

  • Add testing to the pipeline as a predicate for building.
    • Linting?
  • Automated Github Release drafts including the manufactured archives.
  • Current builds are only provided for Ubuntu. I'm not sure what the limits of the program are, but I expect we can expand this to cover other wanted OSes as well.

Pending:

  • Minor tweaks to the configs to modify parameters using my personal fork to using the upstream repo.
    • If Azure, possibly setting up the necessary Azure entities and permissions. (Github, I think, is plug-n-play.)
  • Git hashes and/or tags are nice to have in the archive names.

meowsbits and others added 17 commits May 12, 2020 17:00
Make was killed with signal 9, meaning system
ran out of RAM most likely. This is an attempt
to fix that.
https://github.com/meowsbits/evm_llvm/runs/668739545?check_suite_focus=true#step:3:2561

Signed-off-by: meows <b5c6@protonmail.com>
Signed-off-by: meows <b5c6@protonmail.com>
Signed-off-by: meows <b5c6@protonmail.com>
Log nproc

Signed-off-by: meows <b5c6@protonmail.com>
Signed-off-by: meows <b5c6@protonmail.com>
solution?: use CMAKE_BUILD_TYPE=Release to
limit memory requirements

Signed-off-by: meows <b5c6@protonmail.com>
This reverts commit 6d6d68c.

This change turned out to be unnecessary.
Anecdotally, both Azure and Actions
have 2 procs available. This configuration
standardizes the make commands to use this
value (whatever it is), and hopefully future-
proofs it for optimzation to the CI env.

Signed-off-by: meows <b5c6@protonmail.com>
Signed-off-by: meows <b5c6@protonmail.com>
This is congruent to behavior at Github Actions
config.

Signed-off-by: meows <b5c6@protonmail.com>
This allows a consumer to cross reference the checksum
between logs and artifacts.

Signed-off-by: meows <b5c6@protonmail.com>
Signed-off-by: meows <b5c6@protonmail.com>
Simplifies and streamlines make step greatly.

Signed-off-by: meows <b5c6@protonmail.com>
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.

1 participant