Bump llvm -> 9.0.0 (fixes arm builds) #737

Merged
mcrosson merged 1 commits from llvm_900 into master 4 years ago
mcrosson commented 4 years ago (Migrated from github.com)

Arm builds are broken on the latest cargo/code. Needed a bump to LLVM 9.0.0 to get past some compile errors for the wasm / lld stuff needed for plume-front

Patch bumps LLVM and has been tested as buildable on arm32v7 and arm64v8

Arm builds are broken on the latest cargo/code. Needed a bump to LLVM 9.0.0 to get past some compile errors for the wasm / lld stuff needed for plume-front Patch bumps LLVM and has been tested as buildable on arm32v7 and arm64v8
elegaanz (Migrated from github.com) approved these changes 4 years ago
elegaanz (Migrated from github.com) left a comment

I trust you on that, if you say it is required.

I trust you on that, if you say it is required.
codecov[bot] commented 4 years ago (Migrated from github.com)

Codecov Report

Merging #737 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #737   +/-   ##
=======================================
  Coverage   39.16%   39.16%           
=======================================
  Files          73       73           
  Lines        9653     9653           
  Branches     2183     2183           
=======================================
  Hits         3781     3781           
  Misses       4819     4819           
  Partials     1053     1053
# [Codecov](https://codecov.io/gh/Plume-org/Plume/pull/737?src=pr&el=h1) Report > Merging [#737](https://codecov.io/gh/Plume-org/Plume/pull/737?src=pr&el=desc) into [master](https://codecov.io/gh/Plume-org/Plume/commit/2a58835f9291729076dc06febf050e47e08d73c1?src=pr&el=desc) will **not change** coverage. > The diff coverage is `n/a`. ```diff @@ Coverage Diff @@ ## master #737 +/- ## ======================================= Coverage 39.16% 39.16% ======================================= Files 73 73 Lines 9653 9653 Branches 2183 2183 ======================================= Hits 3781 3781 Misses 4819 4819 Partials 1053 1053 ```

Reviewers

The pull request has been merged as 02c528cae4.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b llvm_900 master
git pull origin llvm_900

Step 2:

Merge the changes and update on Forgejo.
git checkout master
git merge --no-ff llvm_900
git push origin master
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Plume/Plume#737
Loading…
There is no content yet.