[PATCH] D80172: Revert "Re-fix _lrotl/_lrotr to always take Long, no matter the platform."

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 18 17:54:25 PDT 2020


rnk added a comment.

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.


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