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

[Michal Meloun via Phabricator via llvm-commits]

strejda added a comment.

Many thanks for constructive response. For me either from 2 or 3 is fully OK.
But may I add other variant?
Do 2 for 'adr' but 3 for rest ('ldr'...). This removes unnecessary linker complexity while holding all other useful features.
The motivation for this is:

- Relocation for 'adr' cannot be implemented in a dynamic environment at all (imho, there is no dynamic relocation suitable for 'adr' instructions)
- In static environment,usage should be limited to same compilation unit, same section. Extremely small relocation range doesn't allow other usage without significant chance of relocation overflow in final linking.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72892