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

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 29 08:45:26 PDT 2023


asb added a comment.

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

> In D146191#4230864 <https://reviews.llvm.org/D146191#4230864>, @asb wrote:
>
>> Thanks Jessica - in that case, LGTM once it has the MIR test case Jessica requested.
>
> I thought the earlier conclusion was that the RISC-V part wasn’t needed if T-Head moves their pseudo expansion in their downstream.

Yes, let's drop the RISC-V part of this if there's not a good motivation for running MCP so late.


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