[PATCH] D157694: [RISCV][MC]Add support for Binary MCExpr

Job Noorman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 25 03:29:46 PDT 2023


jobnoorman added a comment.

In D157694#4587199 <https://reviews.llvm.org/D157694#4587199>, @wangpc wrote:

> I tried to fix this issue yesterday and I used the same way as this patch. Its problem is that it can't complile case in https://github.com/llvm/llvm-project/issues/64623:
>
>   <unknown>:0: error: unsupported relocation type
>
> Because there is no corresponding relocation type for fixup type `fixup_riscv_12_i`.

I think this is fine as this error is only generated when relaxation is enabled in which case it safer to not allow the symbol difference expression in LI. The error message could be improved though.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157694



More information about the llvm-commits mailing list