This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] [AArch64, NEON] Improve vpmaxX & vpminX intrinsics
- From: Marcus Shawcroft <marcus dot shawcroft at gmail dot com>
- To: "Yangfei (Felix)" <felix dot yang at huawei dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Fri, 5 Dec 2014 18:44:47 +0000
- Subject: Re: [PATCH] [AArch64, NEON] Improve vpmaxX & vpminX intrinsics
- Authentication-results: sourceware.org; auth=none
- References: <DA41BE1DDCA941489001C7FBD7A8820E837A2D0A at szxema507-mbx dot china dot huawei dot com>
On 28 November 2014 at 09:23, Yangfei (Felix) <felix.yang@huawei.com> wrote:
> Hi,
> This patch converts vpmaxX & vpminX intrinsics to use builtin functions instead of the previous inline assembly syntax.
> Regtested with aarch64-linux-gnu on QEMU. Also passed the glorious testsuite of Christophe Lyon.
> OK for the trunk?
Hi Felix, We know from experience that the advsimd intrinsics tend
to be fragile for big endian and in general it is fairly easy to break
the big endian case. For these advsimd improvements that you are
working on (that we very much appreciate) it is important to run both
little endian and big endian regressions.
Thanks
/Marcus