[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