[PATCH] D142551: [InstCombine] Teach isDeadPHICycle to look through multiple uses

Wed Jan 25 09:35:36 PST 2023

luke added a comment.

In D142551#4080424 <https://reviews.llvm.org/D142551#4080424>, @luismarques wrote:

> In D142551#4080298 <https://reviews.llvm.org/D142551#4080298>, @luke wrote:
>
>> Good idea, I don't have access to a remote on it though. Would someone be able to post a branch with this patch on it?
>
> https://llvm-compile-time-tracker.com/compare.php?from=f84d3dd0fd4ec3be52488e4c29baebb0077b04d4&to=b79315a7071612621ac543cd7595cb487c437a64&stat=instructions:u

Thanks. That seems like a non-negligible increase. Would it be worthwhile then limiting the search depth for non-phi node instructions to maybe 1 or 2 at a time?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142551