[PATCH] D154205: [MachineLICM] Handle subloops

Wed Jul 12 10:46:52 PDT 2023

jaykang10 added a comment.

In D154205#4494140 <https://reviews.llvm.org/D154205#4494140>, @nikic wrote:

>> we could skip to visit basic blocks which are already visited with inner loop because the loop invariant code of the inner loop has already been hoisted to inner loop's preheader.
>
> Please do implement this (and revert this patch in the meantime if non-trivial). LICM should visit each block only once, not once per parent loop.

Experimentally, I tried it but it was not simple and caused regressions... Let me try it again.
In the meantime, if possible, I would like to keep this patch in the meantime.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154205