#612 Fix arm builds

Merged
mcrosson merged 1 commits from arm_fixup into master 1 year ago
mcrosson commented 1 year ago (Migrated from github.com)

The latest plume sources need LLVM 8.0.0 for compilation. This PR bumps LLVM and restores arm64 builds.

arm32 builds should work but nix is currently broken for builds. See #611 for more information on the arm32 breakages. Once nix is updated, this should also fix arm32 builds.

I’ve also removed a pair of comments that reference unneeded LLVM tools.

The latest plume sources need LLVM 8.0.0 for compilation. This PR bumps LLVM and restores arm64 builds. arm32 builds should work but nix is currently broken for builds. See #611 for more information on the arm32 breakages. Once nix is updated, this should also fix arm32 builds. I've also removed a pair of comments that reference unneeded LLVM tools.
codecov[bot] commented 1 year ago (Migrated from github.com)
Owner

Codecov Report

No coverage uploaded for pull request base (master@b2312d7). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             master    #612   +/-   ##
========================================
  Coverage          ?   34.6%           
========================================
  Files             ?      67           
  Lines             ?    7840           
  Branches          ?    1883           
========================================
  Hits              ?    2713           
  Misses            ?    4367           
  Partials          ?     760
# [Codecov](https://codecov.io/gh/Plume-org/Plume/pull/612?src=pr&el=h1) Report > :exclamation: No coverage uploaded for pull request base (`master@b2312d7`). [Click here to learn what that means](https://docs.codecov.io/docs/error-reference#section-missing-base-commit). > The diff coverage is `n/a`. ```diff @@ Coverage Diff @@ ## master #612 +/- ## ======================================== Coverage ? 34.6% ======================================== Files ? 67 Lines ? 7840 Branches ? 1883 ======================================== Hits ? 2713 Misses ? 4367 Partials ? 760 ```
elegaanz (Migrated from github.com) approved these changes 1 year ago
elegaanz (Migrated from github.com) left a comment

👍

Reviewers

Plume_migration_agent approved these changes 1 year ago
The pull request has been merged as e7126ae335.
Sign in to join this conversation.
No Milestone
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.