[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