[PATCH] D124308: [MachineScheduler] exclude INLINEASM from schedule when it would increase register pressure

[Nick Desaulniers via Phabricator via llvm-commits]

nickdesaulniers abandoned this revision.
nickdesaulniers added a comment.

I'm not comfortable pursuing this approach.

Say we had an example where there were two instructions that consumed physregs, producing virt regs, then inlineasm with lots of operands, then consumers of the virt regs, and we could only reschedule one of the two virt reg produces below the asm.

This approach will pessimistically allow neither to be moved.

I will have to think more about this, but am super busy with other things ATM. So I plan to at least land https://reviews.llvm.org/D122350 for now.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124308