[PATCH] D153260: [RISCV][MC] Implement mapping symbols

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 24 07:47:03 PDT 2023


asb added a comment.

In D153260#4484423 <https://reviews.llvm.org/D153260#4484423>, @kito-cheng wrote:

> This patch LGTM, however D137417 <https://reviews.llvm.org/D137417> has implement more complete mapping symbol support, I am OK to accept/land this part first and then let D137417 <https://reviews.llvm.org/D137417> to implement the rest part, and actually I am prefer this way since this patch implement some detail more right on creating mapping symbol, but I would like defer the final decision to @asb.
>
> cc @LiDongjin

If the consensus is this patch is more incremental it makes sense to land it I think.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153260



More information about the llvm-commits mailing list