[PATCH] D76050: [LoopPeel] Turn incorrect assert into a check

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 12 03:03:21 PDT 2020


lebedev.ri added a comment.

As @fedor.sergeev is pointing out, that assert exists because in the review
we thought it was an invariant that doesn't make sense to check via an `if()`,
but instead just `assert()` it. Clearly, that is wrong, so let's go back to checking it.


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

https://reviews.llvm.org/D76050





More information about the llvm-commits mailing list