Re: [Patch, RTL] Eliminate redundant vec_select moves.

2013-12-10 Thread H.J. Lu
On Tue, Dec 10, 2013 at 1:09 PM, H.J. Lu wrote: > On Tue, Dec 10, 2013 at 12:39 PM, Richard Henderson wrote: >> On 12/10/2013 10:44 AM, Richard Sandiford wrote: >>> Sorry, I don't understand. I never said it was invalid. I said >>> (subreg:SF (reg:V4SF X) 1) was invalid if (reg:V4SF X) represen

PATCH: PR target/59458: alternative 13 in *movsf_internal is mishandled

2013-12-10 Thread H.J. Lu
Hi, We have (define_insn "*movsf_internal" [(set (match_operand:SF 0 "nonimmediate_operand" "=Yf*f,m ,Yf*f,?r ,?m,v,v,v,m,?r,?Yi,!*y,!*y,!m,!r ,!*Ym") (match_operand:SF 1 "general_operand" "Yf*fm,Yf*f,G ,rmF,rF,C,v,m,v,Yj,r ,*y ,m ,*y,*Yn,r"))] alternative 13

Re: [Patch, RTL] Eliminate redundant vec_select moves.

2013-12-10 Thread Tejas Belagod
On 10 December 2013 21:51, H.J. Lu wrote: > On Tue, Dec 10, 2013 at 1:09 PM, H.J. Lu wrote: >> On Tue, Dec 10, 2013 at 12:39 PM, Richard Henderson wrote: >>> On 12/10/2013 10:44 AM, Richard Sandiford wrote: Sorry, I don't understand. I never said it was invalid. I said (subreg:SF (re

[Fortran] RFC / RFA patch for using build_predict_expr instead of builtin_expect / PR 58721

2013-12-10 Thread Tobias Burnus
Pre-remark: I had hoped that something like the following patch would work. However, it will lead to a bunch of run-time segfaults in the test suite - but the original dump seems to be fine and I also fail to spot the problem when looking at the patch. Thus, instead of posting a working patch,

Re: [Patch, RTL] Eliminate redundant vec_select moves.

2013-12-10 Thread H.J. Lu
On Tue, Dec 10, 2013 at 2:25 PM, Tejas Belagod wrote: > On 10 December 2013 21:51, H.J. Lu wrote: >> On Tue, Dec 10, 2013 at 1:09 PM, H.J. Lu wrote: >>> On Tue, Dec 10, 2013 at 12:39 PM, Richard Henderson wrote: On 12/10/2013 10:44 AM, Richard Sandiford wrote: > Sorry, I don't understa

Re: [buildrobot] score: Silence warnings to fix config-list.mk build

2013-12-10 Thread Jan-Benedict Glaw
Hi Jeff, On Thu, 2013-12-05 21:14:23 -0700, Jeff Law wrote: > On 12/05/13 21:10, Jan-Benedict Glaw wrote: > > On Thu, 2013-12-05 20:49:06 -0700, Jeff Law wrote: > > > I'd just change this one to be correct for the GNU style. If > > > you wanted to follow-up with another patch to fix these > > >

[PATCH, libgfortran]: Emit SSE instructions when __SSE_MATH__ is defined

2013-12-10 Thread Tobias Burnus
The patch does the same for libgfortran as Uros did for libgcc/libatomic, cf. http://gcc.gnu.org/ml/gcc-patches/2013-12/msg00878.html "This is also how glibc generates exceptions." Build and tested on x86-64-gnu-linux. OK ? Tobias 2013-12-10 Tobias Burnus * config/fpu-387.h (sigill_hdlr,

Re: [PATCH PR41488]Recognize more induction variables by simplifying PEELED chrec in scalar evolution

2013-12-10 Thread Jakub Jelinek
On Tue, Dec 10, 2013 at 10:46:32AM +0800, bin.cheng wrote: > This is the new version patch computing the difference in tree affine then > comparing against integer_zero_node. > Bootstrap and test on current upstream. I will commit it if there is no > other objection. This breaks bootstrap on x86_

Re: Fix PR rtl-optimization/58295

2013-12-10 Thread Eric Botcazou
> The patch does indeed fix unsigned-extend-1.c on arm and bootstraps > succesfully on arm-none-linux-gnueabihf. Thanks for the testing. > Thanks for fixing this, that test failure has been spoiling my test runs for > quite a while now :) You're welcome. Now applied on mainline and 4.8 branch,

Re: PATCH: PR target/59458: alternative 13 in *movsf_internal is mishandled

2013-12-10 Thread Uros Bizjak
On Tue, Dec 10, 2013 at 11:23 PM, H.J. Lu wrote: > Hi, > > We have > > (define_insn "*movsf_internal" > [(set (match_operand:SF 0 "nonimmediate_operand" > "=Yf*f,m ,Yf*f,?r ,?m,v,v,v,m,?r,?Yi,!*y,!*y,!m,!r ,!*Ym") > (match_operand:SF 1 "general_operand" > "Yf*fm,Yf*

Re: [PING] [PATCH][RFA][PR middle-end/59285] Handle BARRIERS between blocks in rtl_merge_blocks

2013-12-10 Thread Steven Bosscher
On Tue, Dec 10, 2013 at 7:30 AM, Jeff Law wrote: > > > Ping! I'd prefer going with the first patch to merge_if_blocks (with a big "???" or FIXME), to contain this problem to where it occurs. Otherwise, before you know it, there are other places in the compiler that assume it's OK to merge blocks t

Re: PATCH: PR target/59458: alternative 13 in *movsf_internal is mishandled

2013-12-10 Thread H.J. Lu
On Tue, Dec 10, 2013 at 3:19 PM, Uros Bizjak wrote: > On Tue, Dec 10, 2013 at 11:23 PM, H.J. Lu wrote: >> Hi, >> >> We have >> >> (define_insn "*movsf_internal" >> [(set (match_operand:SF 0 "nonimmediate_operand" >> "=Yf*f,m ,Yf*f,?r ,?m,v,v,v,m,?r,?Yi,!*y,!*y,!m,!r ,!*Ym") >>

Re: [Patch, i386] PR 59422 - Support more targets for function multi versioning

2013-12-10 Thread Uros Bizjak
Hello! > PR gcc/59422 > > This patch extends the supported targets for function multi versiong to also > include Haswell, Silvermont, and the most recent AMD models. It also > prioritizes AVX2 versions over AMD specific pre-AVX2 versions. Please add a ChangeLog entry and attach the complete patch

Re: [RFC, patch] Detect lack of 32-bit devel environment on x86_64-linux targets

2013-12-10 Thread Paolo Bonzini
Il 09/12/2013 12:08, Gerald Pfeifer ha scritto: > On Mon, 9 Dec 2013, FX wrote: >> > Look at this as a diagnostics bug: our current diagnostics for this >> > pretty common situation sucks. It comes late in the compilation, and >> > the message itself isn’t helpful. > Totally seconded. > > Paolo,

Re: [PATCH PR41488]Recognize more induction variables by simplifying PEELED chrec in scalar evolution

2013-12-10 Thread Bin.Cheng
On Wed, Dec 11, 2013 at 6:50 AM, Jakub Jelinek wrote: > On Tue, Dec 10, 2013 at 10:46:32AM +0800, bin.cheng wrote: >> This is the new version patch computing the difference in tree affine then >> comparing against integer_zero_node. >> Bootstrap and test on current upstream. I will commit it if t

Re: [Patch, RTL] Eliminate redundant vec_select moves.

2013-12-10 Thread H.J. Lu
On Tue, Dec 10, 2013 at 2:33 PM, H.J. Lu wrote: > On Tue, Dec 10, 2013 at 2:25 PM, Tejas Belagod > wrote: >> On 10 December 2013 21:51, H.J. Lu wrote: >>> On Tue, Dec 10, 2013 at 1:09 PM, H.J. Lu wrote: On Tue, Dec 10, 2013 at 12:39 PM, Richard Henderson wrote: > On 12/10/2013

Re: [PING] [PATCH][RFA][PR middle-end/59285] Handle BARRIERS between blocks in rtl_merge_blocks

2013-12-10 Thread Jeff Law
On 12/10/13 16:22, Steven Bosscher wrote: On Tue, Dec 10, 2013 at 7:30 AM, Jeff Law wrote: Ping! I'd prefer going with the first patch to merge_if_blocks (with a big "???" or FIXME), to contain this problem to where it occurs. Otherwise, before you know it, there are other places in the comp

Re: [PATCH PR41488]Recognize more induction variables by simplifying PEELED chrec in scalar evolution

2013-12-10 Thread Bin.Cheng
On Wed, Dec 11, 2013 at 4:31 AM, H.J. Lu wrote: > On Mon, Dec 9, 2013 at 6:46 PM, bin.cheng wrote: >> >>> >> Hi, >> This is the new version patch computing the difference in tree affine then >> comparing against integer_zero_node. >> Bootstrap and test on current upstream. I will commit it if th

Re: [PATCH PR41488]Recognize more induction variables by simplifying PEELED chrec in scalar evolution

2013-12-10 Thread Bin.Cheng
On Wed, Dec 11, 2013 at 6:50 AM, Jakub Jelinek wrote: > On Tue, Dec 10, 2013 at 10:46:32AM +0800, bin.cheng wrote: >> This is the new version patch computing the difference in tree affine then >> comparing against integer_zero_node. >> Bootstrap and test on current upstream. I will commit it if t

[RFA][PATCH][PR tree-optimization/45685]

2013-12-10 Thread Jeff Law
So for this source, compiled for x86_64 with -O3: typedef unsigned long int uint64_t; typedef long int int64_t; int summation_helper_1(int64_t* products, uint64_t count) { int s = 0; uint64_t i; for(i=0; i0) ? 1 : -1; products[i] *= val; if

Re: [PATCH PR41488]Recognize more induction variables by simplifying PEELED chrec in scalar evolution

2013-12-10 Thread Jeff Law
On 12/10/13 23:35, Bin.Cheng wrote: I know little about GC, so when ggc_collect may be called (between two passes)? If so, I have to call free_affine_expand_cache just after the calling to tree_to_affine_combination_expand in SCEV because it's an analyzer and as you pointed out, the analyzing re

*ping* Re: PR c++/58567: Fix ICE on invalid code with -fopenmp in cp/pt.c

2013-12-10 Thread Tobias Burnus
Tobias Burnus wrote: A rather simple fix for an ICE on invalid bug (low-priority 4.8/4.9 regression). Bootstrapped and regtested without new failure on x86-64-gnu-linux. OK for the trunk and 4.8? Tobias

Re: [PATCH, libgfortran]: Emit SSE instructions when __SSE_MATH__ is defined

2013-12-10 Thread Janne Blomqvist
On Wed, Dec 11, 2013 at 12:42 AM, Tobias Burnus wrote: > The patch does the same for libgfortran as Uros did for libgcc/libatomic, > cf. http://gcc.gnu.org/ml/gcc-patches/2013-12/msg00878.html > > "This is also how glibc generates exceptions." > > Build and tested on x86-64-gnu-linux. > OK ? Ok,

<    1   2