Skip to content

Updates to README.md regarding baremetal.patch #63

Open

Description

@colematt

Based on the status of patch D46822 -- which appears to be accepted, closed, and committed to upstream as of July 31, 2018 -- is this step under How can I build upstream LLVM+Clang... still required?

wget https://reviews.llvm.org/D46822?download=true -O baremetal.patch
patch -p0 < baremetal.patch

When I attempted to apply the patch, I got the dreaded "Reversed (or previously applied) patch detected!" error from patch. It seems the patch's contents are now all in the upstream source.

Activity

  1. liang1232018 commented on Sep 6, 2018

    @liang1232018

    I have read the upstream source code git from https://github.com/riscv/riscv-gnu-toolchain .
    ALL the patch in baremetal.patch has already been added in the source code.
    so , I think it's not required now, And I build the llvm-upstream without that patch successfully

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions