[PATCH] D77535: [GlobalISel] Add extended inline assembler support

Jessica Paquette via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 6 10:51:27 PDT 2020


paquette added a comment.

> Before moving on with target specific operand constraints, I'd like to get some feedback on the general approach.

In the past, I think we've been trying to keep target-specific stuff out of the IR Translator as much as possible. For example, CallLowering is its own thing.

Maybe it would make sense to take a similar approach to CallLowering here and split inline ASM lowering into its own thing, which targets can build upon?

(Also thanks for starting work on this!)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77535





More information about the llvm-commits mailing list