On Mon, Sep 19, 2016 at 03:22:39PM +0200, Christophe Lyon wrote: > Hi, > > > On 19 September 2016 at 10:39, Tamar Christina <tamar.christ...@arm.com> > wrote: > > > > > > On 17/09/16 07:20, James Greenhalgh wrote: > > > > > > Hi James, > > > >> The convention when fixing a bugzilla is to place a line in your ChangeLog > >> which looks like: > >> > >> PR component/xxxxx > > > > Oh, I wasn't aware of this, thanks! I'll add it. > >> > >> FAIL: gcc.target/aarch64/advsimd-intrinsics/vdup-vmov.c -O0 (test > >> for excess errors) > >> Excess errors: > >> > >> .../gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vdup-vmov.c:171:5: > >> warning: implicit declaration of function 'vmov_n_p64'; did you mean > >> 'vmov_n_u64'? [-Wimplicit-function > > > > > > It looks like ARM is missing some of the poly64 functions. I can add these > > since they don't seem to be too many. > > > I had opened https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71233 to track > missing > poly64 intrinsics for the ARM target. Not sure why vmov_n_p64 isn't in the > list?
Now you mention it, I don't see vmov_n_p64 in the list of intrinsics at: http://infocenter.arm.com/help/topic/com.arm.doc.ihi0073a/IHI0073A_arm_neon_intrinsics_ref.pdf So I wonder whether this patch would actually add some extra intrinsics that we don't require? Thanks, James