[PATCH] D54143: [WIP, RISCV] Generate address sequences suitable for mcmodel=medium

[Lewis Revill via Phabricator via llvm-commits]

lewis-revill added a comment.

So my problem is that (if we expand `PseudoLLA` in codegen in the same way as we do in `RISCVAsmParser`) then we lose some flexibility for later addressing uses. Currently I can use the wrapper to do `(WrapperPCRel %pcrel_hi(sym))`, `(WrapperPCRel %got_pcrel_hi(sym))`, `(WrapperPCRel %tls_ie_pcrel_hi(sym))` and `(WrapperPCRel %tls_gd_pcrel_hi(sym))`, whereas with `PseudoLLA` I am limited to just `%pcrel_hi` addressing. The addition of `PseudoLA` would help greatly, but only for PIC, not for TLS, meaning another wrapper would be required anyway. Maybe this is an acceptable approach to use `PseudoLLA` and `PseudoLA` where it fits then add a wrapper later?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D54143