[PATCH] [AArch64] Handle vec4 sitofp and uitofp for half

James Molloy james at jamesmolloy.co.uk
Tue Apr 21 12:49:31 PDT 2015


Hi Pirama,

This looks OK to me, but could you please add a test for v8f16 and v16f16
to check oversize vectors are handled sanely? I'm trying to keep the
backend oversize-safe.

Cheers,

James
On Tue, 21 Apr 2015 at 20:44, Pirama Arumuga Nainar <pirama at google.com>
wrote:

> IIUC, this patch shouldn't affect v4i32 to v4f32 or v4f64 operations, but
> couldn't find any tests for those operations. I confirmed that locally, and
> can add tests with this patch if requested.
>
> In a test of v4f16 operations (similar to the comprehensive test in
> http://reviews.llvm.org/D8648), only sitofp and uitofp cause problems,
> and will be fixed with this patch.
>
>
> http://reviews.llvm.org/D9166
>
> 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/20150421/c7c35ba6/attachment.html>


More information about the llvm-commits mailing list