[PATCH] D32244: [BitVector] Add << and >> operators

Zachary Turner via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 20 10:07:25 PDT 2017


Is there any reason BitVector could not also use uint64_t?  The fact that
it's target dependent leads to more than a few headaches when trying to
make portable operations.

On Thu, Apr 20, 2017 at 9:41 AM Craig Topper via Phabricator <
reviews at reviews.llvm.org> wrote:

> craig.topper added a comment.
>
> BitVector's word size is dependent on target as it is using "unsigned
> long" not uint32_t/uint64_t. tcShiftRight is hardcoded to uint64_t though
> we could probably make it a template.
>
>
> Repository:
>   rL LLVM
>
> https://reviews.llvm.org/D32244
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170420/f3cf9949/attachment.html>


More information about the llvm-commits mailing list