[PATCH] D14569: [SDAG] Introduce a new BITREVERSE node along with a corresponding LLVM intrinsic

[James Molloy via llvm-commits]

Hi Hal,

Sure, will do. Thanks!

James

On Wed, 11 Nov 2015 at 15:41 hfinkel at anl.gov via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> hfinkel accepted this revision.
> hfinkel added a comment.
> This revision is now accepted and ready to land.
>
> We need a few more tests, especially for the generic expansion. PPC
> (because you should hit the integer promotion logic) and x86 would be good
> options, and I think it is perfectly reasonable to test the i8, or even
> i16, expansions completely. Otherwise, LGTM.
>
> In follow-up, we should make sure we optimize bitreverse(bitreverse(x)) ->
> x.
>
>
> Repository:
>   rL LLVM
>
> http://reviews.llvm.org/D14569
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151111/72a4a1c9/attachment.html>