[PATCH] D142551: [InstCombine] Teach isDeadPHICycle to look through multiple uses
Luke Lau via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
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
More information about the llvm-commits
mailing list