[PATCH] D79264: [ValueTracking] Use "Optional" for DemandedElts argument to ComputeKnownBits

[Eli Friedman via Phabricator via llvm-commits]

efriedma created this revision.
efriedma added reviewers: spatel, ctetreau.
Herald added a subscriber: hiraditya.
Herald added a project: LLVM.
efriedma updated this revision to Diff 261568.
efriedma added a comment.

Also fix ComputeNumSignBits.  Fix a couple bugs (I forgot to run ninja check).


This makes it more clear that it's only expected for vector inputs, as opposed to using a hardcoded "APInt(1, 1)".


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D79264

Files:
  llvm/include/llvm/Analysis/ValueTracking.h
  llvm/lib/Analysis/ValueTracking.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D79264.261568.patch
Type: text/x-patch
Size: 15174 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200502/7d6d538b/attachment.bin>