[PATCH] D69336: [Support] Make BitVector::BitWord size_t

Ehud Katz via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 23 14:26:05 PDT 2019


ekatz added a comment.

In D69336#1719215 <https://reviews.llvm.org/D69336#1719215>, @kparzysz wrote:

> There are custom implementations of these functions for 64-bit unsigned values, so you should make it `uint64_t`.  There is no connection with the pointer size.


Some intrinsics only available for specific architectures. For example, on Windows, _BitScanReverse64 is only available in a 64-bit build (refer to LeadingZerosCounter<T,8> in MathExtras.h).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69336





More information about the llvm-commits mailing list