[PATCH] D146191: [MachineOutliner][MCP][RISCV] Don't run MCP on outlined functions

Wang Pengcheng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 27 23:34:34 PDT 2023


pcwang-thead added a comment.

In D146191#4225567 <https://reviews.llvm.org/D146191#4225567>, @craig.topper wrote:

> Do the T-Head specific pseudos being expanded need to expanded as late as the other pseudos? Could you have an earlier expansion pass? I was already nervous about the short forward branch instructions being affected before. I know we determined those cases were ok.

Yes and thanks, I just tried. They can be placed in `addPreEmitPass()` just like MCP. 
Apart from this, do you think it's still worthy marking outlined functions?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146191



More information about the llvm-commits mailing list