[PATCH] D154205: [MachineLICM] Handle subloops
JinGu Kang via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
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
More information about the llvm-commits
mailing list