[PATCH] D79053: [SVE] Fix invalid use of VectorType::getNumElements() in Value Tracking

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 29 06:56:20 PDT 2020


spatel added inline comments.


================
Comment at: llvm/lib/Analysis/ValueTracking.cpp:211
+  assert(!isa<ScalableVectorType>(Ty) &&
+         "Cannot compute known bits of scalable vector");
+  auto *FVTy = dyn_cast<FixedVectorType>(Ty);
----------------
efriedma wrote:
> I'm surprised this assertion doesn't break anything else; computeKnownBits has a lot of callers.
Yes, I think that's the reason for this failure:
https://reviews.llvm.org/harbormaster/unit/view/73867/


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79053





More information about the llvm-commits mailing list