[PATCH] D142689: [SCEV] Remove applyLoopGuards ExprsToRewrite

Joshua Cao via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 13 08:48:56 PDT 2023


caojoshua added a comment.

In D142689#4416890 <https://reviews.llvm.org/D142689#4416890>, @fhahn wrote:

> Is there a reason for why this was abandoned? Just curious if this is still a worthwhile simplification

The function got rewritten pretty dramatically and I did not want to rebase and rewrite at the time. I do think this work should still be possible.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142689



More information about the llvm-commits mailing list