[PATCH] D109883: [Analysis] Add support for vscale in computeKnownBitsFromOperator

[David Sherwood via Phabricator via llvm-commits]

david-arm marked an inline comment as done.
david-arm added inline comments.


================
Comment at: llvm/include/llvm/Support/KnownBits.h:291
+  // Same as above, but with a uint64_t argument.
+  static KnownBits makeConstant(uint64_t C) { return makeConstant(C); }
+
----------------
foad wrote:
> I'm confused. How does this work (since there is no implicit conversion from uint64_t to APInt) and what does it do (i.e. what BitWidth does it use)?
Hi @foad, oh I see. I should probably write

  return makeConstant(APInt(C));

instead. I imagine the compiler has just assumed I'm calling the version that takes an APInt!


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

https://reviews.llvm.org/D109883