Displaying 2 results from an estimated 2 matches for "d48705".
2018 Jul 02
2
Rotates, once again
...t; I'm guessing nobody has started implementing any of the suggested
> rotate functionality since there are still open questions, but let me
> know if I'm wrong.
>
> We're still getting patches that try to work around the current
> limitations (https://reviews.llvm.org/D48705
> <https://reviews.llvm.org/D48705> ), so we should move forward since
> we've approximated/justified the cost and benefits.
>
> Let's settle on the intrinsic definition(s).
>
> 1. There was a suggestion to generalize rotate to a "valign" or
> "d...
2018 May 17
3
Rotates, once again
Thanks Sanjay!
At this point the cost/benefit tradeoff for rotate intrinsics seems
pretty good.
John
On 05/17/2018 11:14 AM, Sanjay Patel wrote:
> A rotate intrinsic should be relatively close in cost/complexity to the
> existing bswap.
>
> A grep of intrinsic::bswap says we'd probably add code in:
> InstCombine
> InstructionSimplify
> ConstantFolding
>