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

Christopher Tetreault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 29 10:11:22 PDT 2020


ctetreau added a comment.

In D79053#2009094 <https://reviews.llvm.org/D79053#2009094>, @efriedma wrote:

> We probably should support computeKnownBits with scalable vectors; the functionality mostly just works.  The one part that doesn't is the "DemandedElts", but really it's a minor part, and we can just ignore it for scalable vectors.  (So essentially, drop the new assertion from computeKnownBits, and bail out in the few places in the implementation that actually care.)


Yeah, it looks like there are some parts of computeKnownBits that should work for scalable vectors. I'll update it to do the right thing.


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