[PATCH] D83916: Fixed warning about signed/unsigned comparison

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 17 10:32:45 PDT 2020


arsenm accepted this revision.
arsenm added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:7410
              (*SameTypeIt)->getType() == EltTy &&
-             (SameTypeIt - IncIt) < MaxNumElts) {
+             unsigned(SameTypeIt - IncIt) < MaxNumElts) {
         VisitedInstrs.insert(*SameTypeIt);
----------------
static_cast? The constructory integer casts are weird


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

https://reviews.llvm.org/D83916





More information about the llvm-commits mailing list