[PATCH] D80172: Revert "Re-fix _lrotl/_lrotr to always take Long, no matter the platform."
Michael Spencer via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue May 19 12:02:59 PDT 2020
Bigcheese added a comment.
In D80172#2042946 <https://reviews.llvm.org/D80172#2042946>, @rnk wrote:
> I'd be fine going back to the behavior from before.
>
> > Without this behavior there is no intrinsic for 32 bit rotates on these platforms.
>
> Strictly speaking, there seems to be `__builtin_rotate(left|right)32`, which is portable to all clang platforms.
Sorry, I meant there's no MSVC intrinsic.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D80172/new/
https://reviews.llvm.org/D80172
More information about the cfe-commits
mailing list