[PATCH] D72892: [MC][ARM] Resolve some pcrel fixups at assembly time

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 16 17:56:27 PST 2020


MaskRay added a comment.

In D72892#1825574 <https://reviews.llvm.org/D72892#1825574>, @pcc wrote:

> Shouldn't we just implement support for the relocation instead?


I am not familiar with ARM/Thumb. From reading the tests, it looks these instructions usually work for local symbols.

When used with global symbols, I agree that we should eventually support the relocation types in MC and lld. Not sure if that is easy, so created this patch to address your immediate problem.

ARM Big Endian is not changed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72892





More information about the llvm-commits mailing list