[PATCH] D33616: [MS] Fix _bittest* intrinsics for values bigger than 31

Hans Wennborg via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 30 15:59:46 PDT 2017


hans added a comment.

>From looking in the Intel manual (Table 3-2, in 3.1.1.9 about Bit(BitBase,BitOffset)) it does sound like the bit offset can be negative *shudder*, so I suppose this is necessary and explains why the type is signed in the first place? Hopefully most of these will be inlined into a context where BitPos is constant or unsigned.


https://reviews.llvm.org/D33616





More information about the cfe-commits mailing list