[PATCH] D122835: [SCEV] Removed an unnecessary assertion in SCEV.

YangguangLi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 3 13:20:13 PDT 2022


Yangguang added a comment.

In D122835#3489313 <https://reviews.llvm.org/D122835#3489313>, @reames wrote:

> My LGTM specifically applied to the prior version of the patch.  I do see value in the assert, and would prefer that version  I don't feel strongly about this, so feel free to land on nikic's LG.

Thanks for the feedback!


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

https://reviews.llvm.org/D122835



More information about the llvm-commits mailing list