Re: [PATCH][ARM] PR target/67929 Tighten vfp3_const_double_for_bits checks

2015-11-02 Thread Richard Earnshaw
On 02/11/15 11:36, Kyrill Tkachov wrote: > > On 02/11/15 09:29, Kyrill Tkachov wrote: >> >> On 02/11/15 09:28, Yvan Roux wrote: >>> On 2 November 2015 at 10:24, Ramana Radhakrishnan >>> wrote: On 02/11/15 09:01, Christophe Lyon wrote: > On 2 November 2015 at 09:51, Yvan Roux wrote:

Re: [PATCH][ARM] PR target/67929 Tighten vfp3_const_double_for_bits checks

2015-11-02 Thread Kyrill Tkachov
On 02/11/15 09:29, Kyrill Tkachov wrote: On 02/11/15 09:28, Yvan Roux wrote: On 2 November 2015 at 10:24, Ramana Radhakrishnan wrote: On 02/11/15 09:01, Christophe Lyon wrote: On 2 November 2015 at 09:51, Yvan Roux wrote: On 2 November 2015 at 09:38, Ramana Radhakrishnan wrote: 2015-10

Re: [PATCH][ARM] PR target/67929 Tighten vfp3_const_double_for_bits checks

2015-11-02 Thread Richard Earnshaw
On 02/11/15 09:26, Ramana Radhakrishnan wrote: > > > On 02/11/15 09:20, Kyrill Tkachov wrote: >> >> On 02/11/15 08:38, Ramana Radhakrishnan wrote: >> 2015-10-12 Kyrylo Tkachov >> >> PR target/67929 >> * gcc.target/arm/pr67929_1.c: New test. This test fails when t

Re: [PATCH][ARM] PR target/67929 Tighten vfp3_const_double_for_bits checks

2015-11-02 Thread Kyrill Tkachov
On 02/11/15 09:28, Yvan Roux wrote: On 2 November 2015 at 10:24, Ramana Radhakrishnan wrote: On 02/11/15 09:01, Christophe Lyon wrote: On 2 November 2015 at 09:51, Yvan Roux wrote: On 2 November 2015 at 09:38, Ramana Radhakrishnan wrote: 2015-10-12 Kyrylo Tkachov PR target/67929

Re: [PATCH][ARM] PR target/67929 Tighten vfp3_const_double_for_bits checks

2015-11-02 Thread Yvan Roux
On 2 November 2015 at 10:24, Ramana Radhakrishnan wrote: > > > On 02/11/15 09:01, Christophe Lyon wrote: >> On 2 November 2015 at 09:51, Yvan Roux wrote: >>> On 2 November 2015 at 09:38, Ramana Radhakrishnan >>> wrote: >>> >>> 2015-10-12 Kyrylo Tkachov >>> >>> PR targ

Re: [PATCH][ARM] PR target/67929 Tighten vfp3_const_double_for_bits checks

2015-11-02 Thread Ramana Radhakrishnan
On 02/11/15 09:20, Kyrill Tkachov wrote: > > On 02/11/15 08:38, Ramana Radhakrishnan wrote: > 2015-10-12 Kyrylo Tkachov > > PR target/67929 > * gcc.target/arm/pr67929_1.c: New test. >>> This test fails when tested on hard-float targets, adding the >>> following line

Re: [PATCH][ARM] PR target/67929 Tighten vfp3_const_double_for_bits checks

2015-11-02 Thread Ramana Radhakrishnan
On 02/11/15 09:01, Christophe Lyon wrote: > On 2 November 2015 at 09:51, Yvan Roux wrote: >> On 2 November 2015 at 09:38, Ramana Radhakrishnan >> wrote: >>> >> >> 2015-10-12 Kyrylo Tkachov >> >> PR target/67929 >> * gcc.target/arm/pr67929_1.c: New test.

Re: [PATCH][ARM] PR target/67929 Tighten vfp3_const_double_for_bits checks

2015-11-02 Thread Kyrill Tkachov
On 02/11/15 08:38, Ramana Radhakrishnan wrote: 2015-10-12 Kyrylo Tkachov PR target/67929 * gcc.target/arm/pr67929_1.c: New test. This test fails when tested on hard-float targets, adding the following line to avoid testing it in such cases will fix the issue, but I wonder if there

Re: [PATCH][ARM] PR target/67929 Tighten vfp3_const_double_for_bits checks

2015-11-02 Thread Christophe Lyon
On 2 November 2015 at 09:51, Yvan Roux wrote: > On 2 November 2015 at 09:38, Ramana Radhakrishnan > wrote: >> > > 2015-10-12 Kyrylo Tkachov > > PR target/67929 > * gcc.target/arm/pr67929_1.c: New test. >>> >>> This test fails when tested on hard-float targets, addin

Re: [PATCH][ARM] PR target/67929 Tighten vfp3_const_double_for_bits checks

2015-11-02 Thread Yvan Roux
On 2 November 2015 at 09:38, Ramana Radhakrishnan wrote: > 2015-10-12 Kyrylo Tkachov PR target/67929 * gcc.target/arm/pr67929_1.c: New test. >> >> This test fails when tested on hard-float targets, adding the >> following line to avoid testing it in such cases w

Re: [PATCH][ARM] PR target/67929 Tighten vfp3_const_double_for_bits checks

2015-11-02 Thread Ramana Radhakrishnan
>>> >>> 2015-10-12 Kyrylo Tkachov >>> >>> PR target/67929 >>> * gcc.target/arm/pr67929_1.c: New test. > > This test fails when tested on hard-float targets, adding the > following line to avoid testing it in such cases will fix the issue, > but I wonder if there is a better dejaGNU dir

Re: [PATCH][ARM] PR target/67929 Tighten vfp3_const_double_for_bits checks

2015-11-01 Thread Yvan Roux
Hi Kyrill, On 27 October 2015 at 13:08, Ramana Radhakrishnan wrote: > On Wed, Oct 14, 2015 at 1:23 PM, Kyrill Tkachov > wrote: >> Hi all, >> >> This patch fixes the referenced PR by rewriting the >> vfp3_const_double_for_bits function in arm.c >> The function is supposed to accept positive CONS

Re: [PATCH][ARM] PR target/67929 Tighten vfp3_const_double_for_bits checks

2015-10-27 Thread Ramana Radhakrishnan
On Wed, Oct 14, 2015 at 1:23 PM, Kyrill Tkachov wrote: > Hi all, > > This patch fixes the referenced PR by rewriting the > vfp3_const_double_for_bits function in arm.c > The function is supposed to accept positive CONST_DOUBLE rtxes whose value > is an exact power of 2 > and whose log2 is between

Re: [PATCH][ARM] PR target/67929 Tighten vfp3_const_double_for_bits checks

2015-10-26 Thread Kyrill Tkachov
Ping. https://gcc.gnu.org/ml/gcc-patches/2015-10/msg01350.html and the backport for 4.9/5 https://gcc.gnu.org/ml/gcc-patches/2015-10/msg01351.html Thanks, Kyrill On 14/10/15 13:23, Kyrill Tkachov wrote: Hi all, This patch fixes the referenced PR by rewriting the vfp3_const_double_for_bits f

[PATCH][ARM] PR target/67929 Tighten vfp3_const_double_for_bits checks

2015-10-14 Thread Kyrill Tkachov
Hi all, This patch fixes the referenced PR by rewriting the vfp3_const_double_for_bits function in arm.c The function is supposed to accept positive CONST_DOUBLE rtxes whose value is an exact power of 2 and whose log2 is between 1 and 32. That is values like 2.0, 4.0, 8.9, 16.0 etc... The cur