This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH, AArch64] Use llfloor and llceil for vcvtmd_s64_f64 and vcvtpd_s64_f64 in arm_neon.h

From: Marcus Shawcroft <marcus dot shawcroft at arm dot com>
To: Yufeng Zhang <Yufeng dot Zhang at arm dot com>
Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
Date: Wed, 12 Mar 2014 14:32:36 +0000
Subject: Re: [PATCH, AArch64] Use llfloor and llceil for vcvtmd_s64_f64 and vcvtpd_s64_f64 in arm_neon.h
Authentication-results: sourceware.org; auth=none
References: <52CAA1F5 dot 8090204 at arm dot com> <CAFqB+PxMjN_7eV6nrSnv_o414Qv8mSt-QhRbuD79hL-bCGFJVA at mail dot gmail dot com> <530B518E dot 8090803 at arm dot com>

On 24/02/14 14:05, Yufeng Zhang wrote:

Hi Marcus,

On 01/14/14 12:30, Marcus Shawcroft wrote:

On 6 January 2014 12:30, Yufeng Zhang<Yufeng.Zhang@arm.com>  wrote:

This patch fixes the implementation of vcvtmd_s64_f64 and vcvtpd_s64_f64 in
arm_neon.h to use llfloor and llceil instead, which are ILP32-friendly.

This patch will fix the following test failure in the ILP32 mode:

FAIL: gcc.target/aarch64/vect-vcvt.c scan-assembler fcvtms\\tx[0-9]+,
d[0-9]+

OK for the trunk?



OK, but we should wait for stage-1 now.


Although the ILP32 is an experimental feature for 4.9, I think as a bug
fix the patch shall go in for stage-4.


OK provided there is no RM objection in the next 24 hours.

/Marcus

Follow-Ups:
- Re: [PATCH, AArch64] Use llfloor and llceil for vcvtmd_s64_f64 and vcvtpd_s64_f64 in arm_neon.h
  - From: Jakub Jelinek

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]