[PATCH] D151424: [CodeGen] Make use of MachineInstr::all_defs and all_uses. NFCI.

Thu Jun 1 22:50:32 PDT 2023

foad added a comment.

In D151424#4388971 <https://reviews.llvm.org/D151424#4388971>, @nikic wrote:

> It looks like this causes a major compile-time regression: https://llvm-compile-time-tracker.com/compare.php?from=2de54b919ba5fd9ccf37038cddfc36e97eb480af&to=5022fc2ad31b5e3211e2458347c89412b8c5ec1b&stat=instructions:u
>
> The implementation of these methods looks somewhat suspect: https://github.com/llvm/llvm-project/blob/7f374b6902fad9caed41284a57d573abe9ada9d1/llvm/include/llvm/CodeGen/MachineInstr.h#L732 Not sure why this is creating an `std::function`.

Thanks/sorry. I won't be able to look at this for a few days. Feel free to revert if you want.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151424