[PATCH] [x86] @llvm.ctpop.v8i32 custom lowering

Chandler Carruth chandlerc at gmail.com
Mon Dec 22 11:56:51 PST 2014


I see you've gone ahead and committed this.

Please actually implement the significantly better algorithm I point you at
if you're going to have an x86-speciifc implementation. Also please
implement this for the other vector types. I really don't want this to be
left in a half-done state forever.

Expanding to other vector types doesn't seem unreasonable to do in a
follow-up patch, but I think it would have been better to start off with
the final algorithm. We now have a pretty substantial pile of code in the
x86 backend that will be completely replaced. =/

On Mon, Dec 15, 2014 at 9:57 AM, Bruno Cardoso Lopes <
bruno.cardoso at gmail.com> wrote:

> Ping :-)
>
>
> http://reviews.llvm.org/D6531
>
> EMAIL PREFERENCES
>   http://reviews.llvm.org/settings/panel/emailpreferences/
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20141222/68360a05/attachment.html>


More information about the llvm-commits mailing list