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

[Lewis Revill via Phabricator via llvm-commits]

lewis-revill added a comment.

In https://reviews.llvm.org/D54143#1304167, @jrtc27 wrote:

> In https://reviews.llvm.org/D54143#1289972, @lewis-revill wrote:
>
> > Deduplicate switch statements and error on an unsupported code model. Added a wrapper for loading the PC-relative address, which is expanded in `RISCVExpandPseudoInsts`. This approach is a compromise between splitting up into `auipc` and `addi` too early and blocking optimisations due to an additional basic block and splitting up too late and losing the chance to optimize the black box pseudo instruction.
>
>
> Why do we need a new wrapper? Can't we re-use `PseudoLLA` (with `isAsmParserOnly` modified to 0 of course), as this is precisely an `lla`?


That's a good point. I don't think `PseudoLLA` is a precise match? It fits better with `PseudoLA` (IE load global address) which isn't implemented yet. It would be an interesting approach.


Repository:
  rL LLVM

https://reviews.llvm.org/D54143