[PATCH] D129630: [SCEVExpander] Make CanonicalMode handing in isSafeToExpand() more robust (PR50506)

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 13 17:50:45 PDT 2022


reames accepted this revision.
reames added a comment.
This revision is now accepted and ready to land.

LGTM to this patch as is.  I agree with @fhahn's comments, but those are reasonable follow ups and don't need to be done in this patch.  This is already is a significant improvement.

As a follow up thought, have you considered whether the member function variant of isSafeToExpand could be implemented in terms of isSafeToExpandAt with the internal insertion point?  Not sure if that's a good idea or not...


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

https://reviews.llvm.org/D129630



More information about the llvm-commits mailing list