[PATCH] D109371: [RISCV][RFC] Add LLVM support for RISC-V overlay system

Jessica Clarke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 1 04:22:37 PDT 2021


jrtc27 added a comment.

In D109371#3099949 <https://reviews.llvm.org/D109371#3099949>, @edward-jones wrote:

> Some of the previously changes made were undone in order to match the spec as-written (notably I undid the renaming of relocations).
>
> This could do with another review now because I don't expect this patch will change too much now unless the spec also changes.

Aside from the comment I just made about this attitude on the corresponding Clang patch, standard RISC-V relocations can only be specified by the psABI, so the names absolutely can still be changed if the spec authors propose bad names.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D109371/new/

https://reviews.llvm.org/D109371



More information about the llvm-commits mailing list