[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