Re: [PATCH, stage1] Make parloops gate more strict

2015-03-18 Thread Richard Biener
On Wed, Mar 18, 2015 at 12:03 PM, Tom de Vries tom_devr...@mentor.com wrote: On 18-03-15 11:16, Richard Biener wrote: On Fri, Mar 13, 2015 at 4:28 PM, Tom de Vries tom_devr...@mentor.com wrote: On 13-03-15 13:36, Richard Biener wrote: On Fri, Mar 13, 2015 at 1:07 PM, Jakub Jelinek

Re: [PATCH][expmed] Calculate mult-by-const cost properly in mult_by_coeff_cost

2015-03-18 Thread Kyrill Tkachov
On 18/03/15 09:36, Bin.Cheng wrote: On Wed, Mar 18, 2015 at 5:06 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: On 17/03/15 19:11, Jeff Law wrote: On 03/16/2015 04:12 AM, Kyrill Tkachov wrote: Hi all, Eyeballing the mult_by_coeff_cost function I think it has a typo/bug. It's supposed to

[PATCH, ARM, PR64208] LRA ICE Fix

2015-03-18 Thread Yvan Roux
Hi, This is a fix for PR64208 where LRA loops when dealing with iwmmxt_arm_movdi insn. As explain in the PR, the issue was introduced on trunk and 4.9 branch by fix of PR rtl-optimization/60969 and then workaround by r211798 (-fuse-caller-save enable for ARM). The changes in IRA cost made by

Re: [PATCH] PR target/65248: Copy relocation against protected symbol doesn't work

2015-03-18 Thread Uros Bizjak
On Fri, Mar 6, 2015 at 2:42 PM, H.J. Lu hongjiu...@intel.com wrote: On Thu, Mar 05, 2015 at 05:31:51PM -0800, H.J. Lu wrote: Protected data symbol means that it can't be pre-emptied. It doesn't mean its address won't be external. This is true for pointer to protected function. With copy

Re: Fix for PRs 36043, 58744 and 65408

2015-03-18 Thread Richard Biener
On Wed, Mar 18, 2015 at 5:22 AM, Alan Modra amo...@gmail.com wrote: On Tue, Mar 17, 2015 at 01:28:41PM -0600, Jeff Law wrote: On 03/14/2015 07:02 AM, Alan Modra wrote: PR target/65408 PR target/58744 PR middle-end/36043 * calls.c (load_register_parameters): Don't load

Re: [PATCH] Run DCE after if conversion

2015-03-18 Thread Richard Biener
On Tue, Mar 17, 2015 at 7:29 PM, Jeff Law l...@redhat.com wrote: On 03/17/2015 02:17 AM, Andreas Krebbel wrote: Just to have some numbers I did run a -j1 GCC bootstrap twice with and without the patch on x86_64. Best results for both are: clean: 21459s patched: 21314s There rather

Re: [PATCH] Document -m{arch,tune}=knl (PR target/65222)

2015-03-18 Thread Uros Bizjak
On Tue, Mar 10, 2015 at 8:19 PM, Jakub Jelinek ja...@redhat.com wrote: This patch documents knl. The PR is asking for also documenting slm, but I believe those were intentionally removed from the documentation as deprecated aliases, at least that is my reading of the 2013-12-23 H.J. Lu

Re: [PATCH, ARM, PR64208] LRA ICE Fix

2015-03-18 Thread Kyrill Tkachov
Hi Yvan, On 18/03/15 10:19, Yvan Roux wrote: Hi, This is a fix for PR64208 where LRA loops when dealing with iwmmxt_arm_movdi insn. As explain in the PR, the issue was introduced on trunk and 4.9 branch by fix of PR rtl-optimization/60969 and then workaround by r211798 (-fuse-caller-save

Re: [PATCH][expmed] Calculate mult-by-const cost properly in mult_by_coeff_cost

2015-03-18 Thread Kyrill Tkachov
On 17/03/15 19:11, Jeff Law wrote: On 03/16/2015 04:12 AM, Kyrill Tkachov wrote: Hi all, Eyeballing the mult_by_coeff_cost function I think it has a typo/bug. It's supposed to return the cost of multiplying by a constant 'coeff'. It calculates that by taking the cost of a MULT rtx by that

Re: [PATCH] Add pre-reload splitter for low part SI/DImode extraction out of vector regs (PR target/65078)

2015-03-18 Thread Uros Bizjak
On Tue, Mar 17, 2015 at 7:15 PM, Jakub Jelinek ja...@redhat.com wrote: Hi! This patch fixes a regression where since the removal of specialized builtin from _mm_storel_epi64 we force the extraction of DImode (or SImode) low value out of 16/32/64 byte vector registers into memory. As the

Re: C++ PATCH for c++/65398 (valid constexpr rejected)

2015-03-18 Thread Richard Biener
On Fri, Mar 13, 2015 at 3:40 PM, Marek Polacek pola...@redhat.com wrote: We started to reject this (IMHO valid) testcase with r214941 that did away with try_move_mult_to_index -- meaning that we are no longer able to fold *(s[0] + 1) into s[1], while we are able to fold *(s + 1) into s[1].

Re: [patch] libstdc++/64847 add autoconf checks for pthread_rwlock_t

2015-03-18 Thread Jonathan Wakely
On 13/03/15 13:46 +, Jonathan Wakely wrote: On 12/03/15 18:49 +, Jonathan Wakely wrote: I assumed that Pthreads was enough to ensure pthread_rwlock_t but https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64847 shows that isn't true for HPUX (seems it was optional prior to POSIX 1003.1-2001).

Re: [PATCH][expmed] Calculate mult-by-const cost properly in mult_by_coeff_cost

2015-03-18 Thread Bin.Cheng
On Wed, Mar 18, 2015 at 5:06 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: On 17/03/15 19:11, Jeff Law wrote: On 03/16/2015 04:12 AM, Kyrill Tkachov wrote: Hi all, Eyeballing the mult_by_coeff_cost function I think it has a typo/bug. It's supposed to return the cost of multiplying by a

Re: [PATCH, stage1] Make parloops gate more strict

2015-03-18 Thread Richard Biener
On Fri, Mar 13, 2015 at 4:28 PM, Tom de Vries tom_devr...@mentor.com wrote: On 13-03-15 13:36, Richard Biener wrote: On Fri, Mar 13, 2015 at 1:07 PM, Jakub Jelinek ja...@redhat.com wrote: On Fri, Mar 13, 2015 at 01:04:57PM +0100, Richard Biener wrote: Not really (I don't like -fdump-passes

Re: [PATCH, stage1] Make parloops gate more strict

2015-03-18 Thread Tom de Vries
On 18-03-15 11:16, Richard Biener wrote: On Fri, Mar 13, 2015 at 4:28 PM, Tom de Vries tom_devr...@mentor.com wrote: On 13-03-15 13:36, Richard Biener wrote: On Fri, Mar 13, 2015 at 1:07 PM, Jakub Jelinek ja...@redhat.com wrote: On Fri, Mar 13, 2015 at 01:04:57PM +0100, Richard Biener

Re: [PATCH] PR target/65240, Fix Power{7,8} insn constraint issue with -O3 -ffast-math

2015-03-18 Thread Richard Biener
On Wed, Mar 18, 2015 at 12:18 AM, Michael Meissner meiss...@linux.vnet.ibm.com wrote: On Thu, Mar 12, 2015 at 11:37:14AM -0400, David Edelsohn wrote: Please check on the performance implications of removing the special constant support. I know that it is late, but I think that ripping it out

[PATCH] Fix PR ipa/65439

2015-03-18 Thread Martin Liška
Hi. Sorry for previous mail, which has wrongly assigned To address. Original message: This is fix for PR65439 which extends number of expected equivalences for IPA ICF that can be different on darwin target. Ready for trunk? Thanks Martin From 1fb6c73e6ee74a2923aaccbac9658b46005aeba5 Mon

Re: [PATCH, libmpx, i386, PR driver/65444] Pass '-z bndplt' when building dynamic objects with MPX

2015-03-18 Thread Ilya Enkovich
2015-03-18 15:42 GMT+03:00 Richard Biener richard.guent...@gmail.com: On Wed, Mar 18, 2015 at 1:25 PM, H.J. Lu hjl.to...@gmail.com wrote: On Wed, Mar 18, 2015 at 5:13 AM, Ilya Enkovich enkovich@gmail.com wrote: 2015-03-18 15:08 GMT+03:00 H.J. Lu hjl.to...@gmail.com: On Wed, Mar 18, 2015

Re: [PATCH, libmpx, i386, PR driver/65444] Pass '-z bndplt' when building dynamic objects with MPX

2015-03-18 Thread Ilya Enkovich
2015-03-18 16:31 GMT+03:00 H.J. Lu hjl.to...@gmail.com: On Wed, Mar 18, 2015 at 6:24 AM, Ilya Enkovich enkovich@gmail.com wrote: 2015-03-18 15:42 GMT+03:00 Richard Biener richard.guent...@gmail.com: On Wed, Mar 18, 2015 at 1:25 PM, H.J. Lu hjl.to...@gmail.com wrote: On Wed, Mar 18, 2015 at

Re: [PATCH, libmpx, i386, PR driver/65444] Pass '-z bndplt' when building dynamic objects with MPX

2015-03-18 Thread H.J. Lu
On Wed, Mar 18, 2015 at 5:05 AM, Ilya Enkovich enkovich@gmail.com wrote: 2015-03-18 15:02 GMT+03:00 H.J. Lu hjl.to...@gmail.com: On Wed, Mar 18, 2015 at 4:56 AM, Ilya Enkovich enkovich@gmail.com wrote: Hi, This patch fixes PR target/65444 by passing '-z bndplt' to linker when

[CHKP] Never expand instrumentation thunks

2015-03-18 Thread Ilya Enkovich
Hi, This patch disables attempts to expand instrumentation thunks which appear when we create specialized function versions. Problem was found during SPEC2006 insatrumentation with '-Ofast -flto'. Unfortunately I couldn't make a small reproducer. Bootstrapped and tested on

Re: [PATCH] Fix PR ipa/65432

2015-03-18 Thread Richard Biener
On Wed, Mar 18, 2015 at 2:24 PM, Martin Liška mli...@suse.cz wrote: Hello. Following patch wraps symtab_node::{asm_}name with xstrdup_for_dump. Ready for trunk? /* Gets symbol name of the item. */ const char *name (void) { -return node-name (); +return xstrdup_for_dump

Re: [PATCH] Fix align/misalign info in DR_PTR_INFO on vectorizer created SSA_NAMEs (PR tree-optimization/65450)

2015-03-18 Thread Richard Biener
On Wed, 18 Mar 2015, Jakub Jelinek wrote: Hi! As mentioned in the PR, DR_PTR_INFO is for the base of the DR, thus the points-to into in it of course applies to any new SSA_NAME pointers derived from the DR, but the alignment info might not. vect_create_addr_base_for_vector_ref seems to be

Re: [PATCH][ARM] New testcase to check parameter passing bug

2015-03-18 Thread Kyrill Tkachov
Hi Honggyu, On 16/03/15 00:53, Honggyu Kim wrote: new file mode 100644 index 000..3790764 --- /dev/null +++ b/gcc/testsuite/gcc.dg/pr65358.c @@ -0,0 +1,33 @@ +/* { dg-do run */ Forgot to close the brace here after 'run' Dejagnu happily ignores that and transforms this into a compile-only

Re: [PATCH][expmed][cleanup] Use std::swap instead of manual swapping

2015-03-18 Thread Kyrill Tkachov
On 16/03/15 10:10, Kyrill Tkachov wrote: Hi all, This patch replaces manual swapping in synth_mult with std::swap. Not much else to say about this. This code could arguably be refactored a bit but that's another story. I believe these are considered obvious at this point. I'll apply it in 24

Re: [PATCH, libmpx, i386, PR driver/65444] Pass '-z bndplt' when building dynamic objects with MPX

2015-03-18 Thread H.J. Lu
On Wed, Mar 18, 2015 at 4:56 AM, Ilya Enkovich enkovich@gmail.com wrote: Hi, This patch fixes PR target/65444 by passing '-z bndplt' to linker when appropriate. Bootstrapped and tested on x86_64-unknown-linux-gnu. Will commit it to trunk in a couple of days if no objections arise.

Re: [PATCH, libmpx, i386, PR driver/65444] Pass '-z bndplt' when building dynamic objects with MPX

2015-03-18 Thread H.J. Lu
On Wed, Mar 18, 2015 at 5:13 AM, Ilya Enkovich enkovich@gmail.com wrote: 2015-03-18 15:08 GMT+03:00 H.J. Lu hjl.to...@gmail.com: On Wed, Mar 18, 2015 at 5:05 AM, Ilya Enkovich enkovich@gmail.com wrote: 2015-03-18 15:02 GMT+03:00 H.J. Lu hjl.to...@gmail.com: On Wed, Mar 18, 2015 at

[PATCH] Fix PR ipa/65432

2015-03-18 Thread Martin Liška
Hello. Following patch wraps symtab_node::{asm_}name with xstrdup_for_dump. Ready for trunk? Thanks, Martin From 06d7667b7e2be23e21b3ea6599ebb2303074b310 Mon Sep 17 00:00:00 2001 From: mliska mli...@suse.cz Date: Wed, 18 Mar 2015 13:59:49 +0100 Subject: [PATCH] Fix PR ipa/65432 gcc/ChangeLog:

[PATCH] Shrink data-ref

2015-03-18 Thread Richard Biener
This removes dead code/data from tree-data-ref.h. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2015-03-18 Richard Biener rguent...@suse.de * tree-data-ref.h (struct access_matrix): Remove. (AM_LOOP_NEST, AM_NB_INDUCTION_VARS, AM_PARAMETERS,

Re: [PATCH, ARM, PR64208] LRA ICE Fix

2015-03-18 Thread Yvan Roux
HI Kyrill, On 18 March 2015 at 11:24, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: Hi Yvan, On 18/03/15 10:19, Yvan Roux wrote: Hi, This is a fix for PR64208 where LRA loops when dealing with iwmmxt_arm_movdi insn. As explain in the PR, the issue was introduced on trunk and 4.9 branch

[PATCH] Fix PR ipa/65439

2015-03-18 Thread Martin Liška
Hi. This is fix for PR65439 which extends number of expected equivalences for IPA ICF that can be different on darwin target. Ready for trunk? Thanks Martin From 1fb6c73e6ee74a2923aaccbac9658b46005aeba5 Mon Sep 17 00:00:00 2001 From: mliska mli...@suse.cz Date: Wed, 18 Mar 2015 09:52:29 +0100

Re: [PATCH, libmpx, i386, PR driver/65444] Pass '-z bndplt' when building dynamic objects with MPX

2015-03-18 Thread H.J. Lu
On Wed, Mar 18, 2015 at 6:41 AM, Ilya Enkovich enkovich@gmail.com wrote: 2015-03-18 16:31 GMT+03:00 H.J. Lu hjl.to...@gmail.com: On Wed, Mar 18, 2015 at 6:24 AM, Ilya Enkovich enkovich@gmail.com wrote: 2015-03-18 15:42 GMT+03:00 Richard Biener richard.guent...@gmail.com: On Wed, Mar

Re: [PATCH] Fix PR ipa/65432

2015-03-18 Thread Jakub Jelinek
On Wed, Mar 18, 2015 at 02:24:26PM +0100, Martin Liška wrote: From 06d7667b7e2be23e21b3ea6599ebb2303074b310 Mon Sep 17 00:00:00 2001 From: mliska mli...@suse.cz Date: Wed, 18 Mar 2015 13:59:49 +0100 Subject: [PATCH] Fix PR ipa/65432 gcc/ChangeLog: 2015-03-18 Martin Liska mli...@suse.cz

Re: [PATCH, libmpx, i386, PR driver/65444] Pass '-z bndplt' when building dynamic objects with MPX

2015-03-18 Thread Ilya Enkovich
2015-03-18 15:02 GMT+03:00 H.J. Lu hjl.to...@gmail.com: On Wed, Mar 18, 2015 at 4:56 AM, Ilya Enkovich enkovich@gmail.com wrote: Hi, This patch fixes PR target/65444 by passing '-z bndplt' to linker when appropriate. Bootstrapped and tested on x86_64-unknown-linux-gnu. Will commit it

Re: [PATCH, libmpx, i386, PR driver/65444] Pass '-z bndplt' when building dynamic objects with MPX

2015-03-18 Thread Ilya Enkovich
2015-03-18 15:08 GMT+03:00 H.J. Lu hjl.to...@gmail.com: On Wed, Mar 18, 2015 at 5:05 AM, Ilya Enkovich enkovich@gmail.com wrote: 2015-03-18 15:02 GMT+03:00 H.J. Lu hjl.to...@gmail.com: On Wed, Mar 18, 2015 at 4:56 AM, Ilya Enkovich enkovich@gmail.com wrote: Hi, This patch fixes PR

[PATCH, libmpx, i386, PR driver/65444] Pass '-z bndplt' when building dynamic objects with MPX

2015-03-18 Thread Ilya Enkovich
Hi, This patch fixes PR target/65444 by passing '-z bndplt' to linker when appropriate. Bootstrapped and tested on x86_64-unknown-linux-gnu. Will commit it to trunk in a couple of days if no objections arise. Thanks, Ilya -- gcc/ 2015-03-18 Ilya Enkovich ilya.enkov...@intel.com

Re: [PATCH] Document -m{arch,tune}=knl (PR target/65222)

2015-03-18 Thread H.J. Lu
On Wed, Mar 18, 2015 at 2:49 AM, Uros Bizjak ubiz...@gmail.com wrote: On Tue, Mar 10, 2015 at 8:19 PM, Jakub Jelinek ja...@redhat.com wrote: This patch documents knl. The PR is asking for also documenting slm, but I believe those were intentionally removed from the documentation as

Re: [PATCH, libmpx, i386, PR driver/65444] Pass '-z bndplt' when building dynamic objects with MPX

2015-03-18 Thread Richard Biener
On Wed, Mar 18, 2015 at 1:25 PM, H.J. Lu hjl.to...@gmail.com wrote: On Wed, Mar 18, 2015 at 5:13 AM, Ilya Enkovich enkovich@gmail.com wrote: 2015-03-18 15:08 GMT+03:00 H.J. Lu hjl.to...@gmail.com: On Wed, Mar 18, 2015 at 5:05 AM, Ilya Enkovich enkovich@gmail.com wrote: 2015-03-18

Re: [PATCH, libmpx, i386, PR driver/65444] Pass '-z bndplt' when building dynamic objects with MPX

2015-03-18 Thread H.J. Lu
On Wed, Mar 18, 2015 at 6:24 AM, Ilya Enkovich enkovich@gmail.com wrote: 2015-03-18 15:42 GMT+03:00 Richard Biener richard.guent...@gmail.com: On Wed, Mar 18, 2015 at 1:25 PM, H.J. Lu hjl.to...@gmail.com wrote: On Wed, Mar 18, 2015 at 5:13 AM, Ilya Enkovich enkovich@gmail.com wrote:

[PATCH] Fix align/misalign info in DR_PTR_INFO on vectorizer created SSA_NAMEs (PR tree-optimization/65450)

2015-03-18 Thread Jakub Jelinek
Hi! As mentioned in the PR, DR_PTR_INFO is for the base of the DR, thus the points-to into in it of course applies to any new SSA_NAME pointers derived from the DR, but the alignment info might not. vect_create_addr_base_for_vector_ref seems to be handling it right, but vect_create_data_ref_ptr

Re: [PATCH, libmpx, i386, PR driver/65444] Pass '-z bndplt' when building dynamic objects with MPX

2015-03-18 Thread Jakub Jelinek
On Wed, Mar 18, 2015 at 04:59:05PM +0300, Ilya Enkovich wrote: Which is a weird thing to do just to have a warning instead of an error. You don't guarantee MPX PLT generation by always passing '-z bndplt' but remove an opportunity to use gold at all. With current check you may use any linker

Re: [PATCH, libmpx, i386, PR driver/65444] Pass '-z bndplt' when building dynamic objects with MPX

2015-03-18 Thread H.J. Lu
On Wed, Mar 18, 2015 at 7:02 AM, Jakub Jelinek ja...@redhat.com wrote: On Wed, Mar 18, 2015 at 04:59:05PM +0300, Ilya Enkovich wrote: Which is a weird thing to do just to have a warning instead of an error. You don't guarantee MPX PLT generation by always passing '-z bndplt' but remove an

Re: [PATCH] Fix PR ipa/65432

2015-03-18 Thread Martin Liška
TOn 03/18/2015 02:33 PM, Richard Biener wrote: On Wed, Mar 18, 2015 at 2:24 PM, Martin Liška mli...@suse.cz wrote: Hello. Following patch wraps symtab_node::{asm_}name with xstrdup_for_dump. Ready for trunk? /* Gets symbol name of the item. */ const char *name (void) { -

Re: [PATCH] Fix PR ipa/65432

2015-03-18 Thread Jakub Jelinek
On Wed, Mar 18, 2015 at 02:59:10PM +0100, Martin Liška wrote: From 5d2e883524ba23155d80cd4ad6d58ba5c73a1f90 Mon Sep 17 00:00:00 2001 From: mliska mli...@suse.cz Date: Wed, 18 Mar 2015 13:59:49 +0100 Subject: [PATCH] Fix PR ipa/65432 gcc/ChangeLog: 2015-03-18 Martin Liska mli...@suse.cz

Re: [PATCH] Fix PR ipa/65432

2015-03-18 Thread Martin Liška
On 03/18/2015 03:05 PM, Jakub Jelinek wrote: On Wed, Mar 18, 2015 at 02:59:10PM +0100, Martin Liška wrote: From 5d2e883524ba23155d80cd4ad6d58ba5c73a1f90 Mon Sep 17 00:00:00 2001 From: mliska mli...@suse.cz Date: Wed, 18 Mar 2015 13:59:49 +0100 Subject: [PATCH] Fix PR ipa/65432 gcc/ChangeLog:

Re: [PATCH, libmpx, i386, PR driver/65444] Pass '-z bndplt' when building dynamic objects with MPX

2015-03-18 Thread H.J. Lu
On Wed, Mar 18, 2015 at 7:31 AM, H.J. Lu hjl.to...@gmail.com wrote: On Wed, Mar 18, 2015 at 7:02 AM, Jakub Jelinek ja...@redhat.com wrote: On Wed, Mar 18, 2015 at 04:59:05PM +0300, Ilya Enkovich wrote: Which is a weird thing to do just to have a warning instead of an error. You don't guarantee

Patch ping

2015-03-18 Thread Jakub Jelinek
Hi! I'd like to ping following patch: PR65238 - P1 - http://gcc.gnu.org/ml/gcc-patches/2015-03/msg00647.html - fix __has_{,cpp_}attribute() with -traditional-cpp Thanks. Jakub

Re: [PATCH, libmpx, i386, PR driver/65444] Pass '-z bndplt' when building dynamic objects with MPX

2015-03-18 Thread Markus Trippelsdorf
On 2015.03.18 at 10:03 -0400, Robert Dewar wrote: Do we really want to quote to this level? This message has 11 levels of quotes, the most I have ever seen. If everyone does this, the whole thread is in every message and that seems unnecessary. I don't know if there are gcc guidelines on

Re: [PATCH] Handle vector COND_EXPRs in vector genericization (PR tree-optimization/65427)

2015-03-18 Thread Kyrill Tkachov
On 16/03/15 18:26, Jakub Jelinek wrote: On Mon, Mar 16, 2015 at 07:15:59PM +0100, Richard Biener wrote: On March 16, 2015 5:21:02 PM GMT+01:00, Jakub Jelinek ja...@redhat.com wrote: On the following testcase, gimple LIM creates a vector COND_EXPR (scalar condition, vector lhs, rhs2 and rhs3),

Re: [PATCH, libmpx, i386, PR driver/65444] Pass '-z bndplt' when building dynamic objects with MPX

2015-03-18 Thread Robert Dewar
Do we really want to quote to this level? This message has 11 levels of quotes, the most I have ever seen. If everyone does this, the whole thread is in every message and that seems unnecessary. I don't know if there are gcc guidelines on this??? On 3/18/2015 9:59 AM, Ilya Enkovich wrote:

Re: [PATCH, libmpx, i386, PR driver/65444] Pass '-z bndplt' when building dynamic objects with MPX

2015-03-18 Thread Ilya Enkovich
2015-03-18 16:52 GMT+03:00 H.J. Lu hjl.to...@gmail.com: On Wed, Mar 18, 2015 at 6:41 AM, Ilya Enkovich enkovich@gmail.com wrote: 2015-03-18 16:31 GMT+03:00 H.J. Lu hjl.to...@gmail.com: On Wed, Mar 18, 2015 at 6:24 AM, Ilya Enkovich enkovich@gmail.com wrote: 2015-03-18 15:42 GMT+03:00

Re: [PATCH] Handle vector COND_EXPRs in vector genericization (PR tree-optimization/65427)

2015-03-18 Thread Jakub Jelinek
On Wed, Mar 18, 2015 at 03:10:44PM +, Kyrill Tkachov wrote: I've also been thinking about lowering it to control flow, but only if it couldn't be done in say two halves comparison as in the testcase. I suppose doing say 2 V4SImode COND_EXPRs would still be beneficial over control flow,

Re: [PATCH] Handle vector COND_EXPRs in vector genericization (PR tree-optimization/65427)

2015-03-18 Thread Kyrill Tkachov
On 18/03/15 15:14, Jakub Jelinek wrote: On Wed, Mar 18, 2015 at 03:10:44PM +, Kyrill Tkachov wrote: I've also been thinking about lowering it to control flow, but only if it couldn't be done in say two halves comparison as in the testcase. I suppose doing say 2 V4SImode COND_EXPRs would

Re: [PATCH, libmpx, i386, PR driver/65444] Pass '-z bndplt' when building dynamic objects with MPX

2015-03-18 Thread Ilya Enkovich
2015-03-18 17:42 GMT+03:00 H.J. Lu hjl.to...@gmail.com: On Wed, Mar 18, 2015 at 7:31 AM, H.J. Lu hjl.to...@gmail.com wrote: On Wed, Mar 18, 2015 at 7:02 AM, Jakub Jelinek ja...@redhat.com wrote: Yeah, I agree, the configure check is a reasonable thing to do. We should either always pass -z

Re: PR 13631 Problems in messages

2015-03-18 Thread Jonathan Wakely
On 03/12/14 14:54 +, Jonathan Wakely wrote: On 02/12/14 23:58 +0100, François Dumont wrote: DR libstdc++/13631 s/DR/PR/ * include/bits/codecvt.h (codecvtchar, char, mbstate_t): friend class std::messageschar. (codecvtwchar_t, char, mbstate_t): friend class

Re: [debug-early] equate new DIE with DW_AT_specificationto a previous declaration

2015-03-18 Thread Aldy Hernandez
On 03/17/2015 07:12 PM, Jason Merrill wrote: On 03/17/2015 03:58 PM, Aldy Hernandez wrote: The problem is that, for -fno-implicit-templates, the decl is now DECL_EXTERNAL, which means we never equate this new DIE with DW_AT_specification to the DECL. That is, we never fall through here:

Re: [PATCH, libmpx, i386, PR driver/65444] Pass '-z bndplt' when building dynamic objects with MPX

2015-03-18 Thread H.J. Lu
On Wed, Mar 18, 2015 at 9:14 AM, Ilya Enkovich enkovich@gmail.com wrote: 2015-03-18 17:42 GMT+03:00 H.J. Lu hjl.to...@gmail.com: On Wed, Mar 18, 2015 at 7:31 AM, H.J. Lu hjl.to...@gmail.com wrote: On Wed, Mar 18, 2015 at 7:02 AM, Jakub Jelinek ja...@redhat.com wrote: Yeah, I agree, the

Re: [PATCH][RFC] Fix PR63155

2015-03-18 Thread Alan Lawrence
Following this patch (r221318), we're seeing what appears to be a miscompile of glibc on AArch64. This causes quite a bunch of tests to fail, segfaults etc., if LD_LIBRARY_PATH leads to a libc.so.6 built with that patch vs without (same glibc sources). We are still working on a reduced

Re: [PATCH] Handle vector COND_EXPRs in vector genericization (PR tree-optimization/65427)

2015-03-18 Thread Kyrill Tkachov
On 18/03/15 15:20, Kyrill Tkachov wrote: On 18/03/15 15:14, Jakub Jelinek wrote: On Wed, Mar 18, 2015 at 03:10:44PM +, Kyrill Tkachov wrote: I've also been thinking about lowering it to control flow, but only if it couldn't be done in say two halves comparison as in the testcase. I

Re: [PATCH, ARM, PR64208] LRA ICE Fix

2015-03-18 Thread Yvan Roux
On 18 March 2015 at 12:42, Yvan Roux yvan.r...@linaro.org wrote: HI Kyrill, On 18 March 2015 at 11:24, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: Hi Yvan, On 18/03/15 10:19, Yvan Roux wrote: Hi, This is a fix for PR64208 where LRA loops when dealing with iwmmxt_arm_movdi insn. As

Re: PR 13631 Problems in messages

2015-03-18 Thread Jonathan Wakely
On 18/03/15 16:17 +, Jonathan Wakely wrote: The new get_glibc_msg function doesn't compile with old versions of glibc, _M_name_messages is not defined (the code is also not exception safe, but that was not introduced by the recent changes). We can probably drop that old code completely,

Re: [PATCH, libmpx, i386, PR driver/65444] Pass '-z bndplt' when building dynamic objects with MPX

2015-03-18 Thread H.J. Lu
On Wed, Mar 18, 2015 at 10:13 AM, Ilya Enkovich enkovich@gmail.com wrote: 2015-03-18 19:45 GMT+03:00 H.J. Lu hjl.to...@gmail.com: On Wed, Mar 18, 2015 at 9:14 AM, Ilya Enkovich enkovich@gmail.com wrote: 2015-03-18 17:42 GMT+03:00 H.J. Lu hjl.to...@gmail.com: On Wed, Mar 18, 2015 at

[PATCH][2/3][PR65458] Mark omp thread functions as parallelized

2015-03-18 Thread Tom de Vries
Hi, this patch fixes PR65458. The patch marks omp thread functions as parallelized, which means the parloops pass no longer attempts to modify that function. Bootstrapped and reg-tested on x86_64. OK for stage4 trunk? Thanks, - Tom Mark omp thread functions as parallelized 2015-03-18 Tom

[patch] c++/65046 Use abi-tag to fix building with -Werror=abi-tag

2015-03-18 Thread Jonathan Wakely
This adds the abi_tag attribute in a few places it's missing, so that -Werror=abi-tag builds will work after Jason's upcoming patches to make functions inherit the tag from their return type. For the __sso_string type I disable the warning, because the fact it uses std::__cxx11::string

Re: [PATCH, libmpx, i386, PR driver/65444] Pass '-z bndplt' when building dynamic objects with MPX

2015-03-18 Thread Ilya Enkovich
2015-03-18 20:14 GMT+03:00 H.J. Lu hjl.to...@gmail.com: On Wed, Mar 18, 2015 at 10:13 AM, Ilya Enkovich enkovich@gmail.com wrote: 2015-03-18 19:45 GMT+03:00 H.J. Lu hjl.to...@gmail.com: On Wed, Mar 18, 2015 at 9:14 AM, Ilya Enkovich enkovich@gmail.com wrote: Instrumented binary

Re: [PATCH, libmpx, i386, PR driver/65444] Pass '-z bndplt' when building dynamic objects with MPX

2015-03-18 Thread H.J. Lu
On Wed, Mar 18, 2015 at 10:34 AM, Ilya Enkovich It doesn't mean we should leave a door open. Are we supposed to detect this with MPX: [hjl@skylakeclient bug-1]$ cat x.c #include string.h int main () { char buf[10]; memset(buf, 'a', 11); return 0; } [hjl@skylakeclient bug-1]$

[PATCH][3/3][PR65460] Mark offloaded functions as parallelized

2015-03-18 Thread Tom de Vries
Hi, this patch fixes PR65460. The patch marks offloaded functions as parallelized, which means the parloops pass no longer attempts to modify that function. Bootstrapped and reg-tested on x86_64. OK for stage4 trunk? Thanks, - Tom Mark offloaded functions as parallelized 2015-03-18 Tom

[PATCH][1/3] Make parallelize_loops static

2015-03-18 Thread Tom de Vries
Hi, this patch makes parallelize_loops static. I think the fact that it was extern was a left-over from the times that the parloops pass was not located in tree-parloops.c. bootstrapped and reg-tested on x86_64. OK for stage4 trunk? Thanks, - Tom Make parallelize_loops static 2015-03-18

Re: [PATCH, libmpx, i386, PR driver/65444] Pass '-z bndplt' when building dynamic objects with MPX

2015-03-18 Thread Ilya Enkovich
2015-03-18 20:39 GMT+03:00 H.J. Lu hjl.to...@gmail.com: On Wed, Mar 18, 2015 at 10:34 AM, Ilya Enkovich What is USER? The one who build GCC may not be same person who uses GCC. The person who build GCC determines its default behavior. User either uses it with default settings or

Re: [PATCH, stage1] Make parloops gate more strict

2015-03-18 Thread Tom de Vries
On 18-03-15 12:18, Richard Biener wrote: On Wed, Mar 18, 2015 at 12:03 PM, Tom de Vries tom_devr...@mentor.com wrote: On 18-03-15 11:16, Richard Biener wrote: On Fri, Mar 13, 2015 at 4:28 PM, Tom de Vries tom_devr...@mentor.com wrote: On 13-03-15 13:36, Richard Biener wrote: On Fri, Mar

Re: [PATCH][2/3][PR65458] Mark omp thread functions as parallelized

2015-03-18 Thread Jakub Jelinek
On Wed, Mar 18, 2015 at 06:21:51PM +0100, Tom de Vries wrote: this patch fixes PR65458. The patch marks omp thread functions as parallelized, which means the parloops pass no longer attempts to modify that function. Bootstrapped and reg-tested on x86_64. OK for stage4 trunk? This will

Re: [PATCH] PR target/65248: Copy relocation against protected symbol doesn't work

2015-03-18 Thread Mike Stump
On Mar 18, 2015, at 2:55 AM, Uros Bizjak ubiz...@gmail.com wrote: Here is the updated patch with testcases. Tested on Linux/x86. OK for trunk? This patch needs global reviewer approval (I have added Jakub to CC) and Darwin maintainer approval. So, my concern would be this, does the bug

Re: [PATCH] Fix PR ipa/65439

2015-03-18 Thread Martin Liška
On 03/18/2015 02:06 PM, Martin Liška wrote: Hi. Sorry for previous mail, which has wrongly assigned To address. Original message: This is fix for PR65439 which extends number of expected equivalences for IPA ICF that can be different on darwin target. Ready for trunk? Thanks Martin I've

RE: [PATCH] Fix another wrong-code bug with -fstrict-volatile-bitfields

2015-03-18 Thread Bernd Edlinger
Hi, On Mon, 16 Mar 2015 20:31:53, Hans-Peter Nilsson wrote: On Mon, 16 Mar 2015, Eric Botcazou wrote: If we have BIGGEST_ALIGNMENT=16 that means we have likely a 16 bit architecture. I doubt that the strict alignment code makes any sense for modesize BIGGEST_ALIGNMENT. Note that m68k is a

Re: [PATCH] Fix PR ipa/65439

2015-03-18 Thread Jan Hubicka
Hi. Sorry for previous mail, which has wrongly assigned To address. Original message: This is fix for PR65439 which extends number of expected equivalences for IPA ICF that can be different on darwin target. Ready for trunk? OK Honza Thanks Martin From

[PATCH v2] New testcase to check parameter passing bug

2015-03-18 Thread Honggyu Kim
Hi, I have modified the test-case to check parameter passing bug based on the comments from Kyrill Tkachov, Christophe Lyon, and Segher Boessenkool as follows: 1. move from gcc.target/arm to gcc.dg 2. change dg-do compile to dg-do run Please let me know if there's still something to fix more.

Re: [debug-early] Cache DW_TAG_label's if appropriate

2015-03-18 Thread Aldy Hernandez
On 03/18/2015 01:59 PM, Jason Merrill wrote: On 03/18/2015 01:03 PM, Aldy Hernandez wrote: By the way Jason, is there any reason we previously called equate_decl_number_to_die() only for abstract instances?: So that multiple concrete instances can find the abstract instance. Ah, I see.

Fix PR 65177: diamonds are not valid execution threads for jump threading

2015-03-18 Thread Sebastian Pop
Hi, the attached patch fixes PR 65177 in which the code generator of FSM jump thread would create a diamond on the copied path: see https://gcc.gnu.org/PR65177#c18 for a detailed description. The patch is renaming SEME into jump_thread as the notion of SEME is more general than the special case

[PATCH] Teach ipa-split about TSAN_FUNC_EXIT (PR sanitizer/65400)

2015-03-18 Thread Jakub Jelinek
Hi! TSAN_FUNC_EXIT internal function is special, because we drop it during inlining, so for fnsplit we need to be careful about it, otherwise we can end up with unbalanced pairs of tsan entry/exit marker functions. This patch gives up unless it is one of the two most common cases with

Re: [PATCH v2] New testcase to check parameter passing bug

2015-03-18 Thread Honggyu Kim
On Thu, Mar 19, 2015 at 10:40:44AM +0900, Honggyu Kim wrote: --- gcc/testsuite/ChangeLog|4 gcc/testsuite/gcc.dg/pr65358.c | 33 + 2 files changed, 37 insertions(+) create mode 100644 gcc/testsuite/gcc.dg/pr65358.c diff --git

[PATCH] Add testcase for PR64265

2015-03-18 Thread Jakub Jelinek
Hi! Back when I've added the EH support for tsan (aka TSAN_FUNC_EXIT), we didn't have the barrier_wait hack to make tsan threads reliable, so I didn't commit a testcase. This one fails with that patch reverted and succeeds with the current trunk, bootstrapped/regtested on x86_64-linux and

[rl78] fix ICE with far operands to and/ior

2015-03-18 Thread DJ Delorie
Committed. * config/rl78/rl78-virt.md (andqi3_virt): Allow far operands. (iorqi3_virt): Likewise. Index: config/rl78/rl78-virt.md === --- config/rl78/rl78-virt.md(revision 221505) +++ config/rl78/rl78-virt.md

[PATCH, rtl-optimization]: Fix PR60851, ICE: in extract_constrain_insn_cached, at recog.c

2015-03-18 Thread Uros Bizjak
Hello! Attached patch activates safety net recognition of pseudo regs for LRA-enabled targets. LRA is able to spill pseudos to memory to satisfy memory-only constraints, so constrain_operands should recognize this functionality. This is actually the same approach as how constants are handled a

Re: [PATCH] PR target/65248: Copy relocation against protected symbol doesn't work

2015-03-18 Thread H.J. Lu
On Wed, Mar 18, 2015 at 11:58 AM, Mike Stump mikest...@comcast.net wrote: On Mar 18, 2015, at 2:55 AM, Uros Bizjak ubiz...@gmail.com wrote: Here is the updated patch with testcases. Tested on Linux/x86. OK for trunk? This patch needs global reviewer approval (I have added Jakub to CC) and

[debug-early] Cache DW_TAG_label's if appropriate

2015-03-18 Thread Aldy Hernandez
I'm running the gdb testsuite with the debug-early branch compiler, and fixing regressions. gdb.base/label.c was causing gdb to ICE because we were generating multiple DW_TAG_label's for the same DECL. Consequently, I have fixed gen_label_die to handle multiple calls with the same DECL. By

Re: [PATCH, libmpx, i386, PR driver/65444] Pass '-z bndplt' when building dynamic objects with MPX

2015-03-18 Thread Ilya Enkovich
2015-03-18 19:45 GMT+03:00 H.J. Lu hjl.to...@gmail.com: On Wed, Mar 18, 2015 at 9:14 AM, Ilya Enkovich enkovich@gmail.com wrote: 2015-03-18 17:42 GMT+03:00 H.J. Lu hjl.to...@gmail.com: On Wed, Mar 18, 2015 at 7:31 AM, H.J. Lu hjl.to...@gmail.com wrote: MPX is a security feature. Knowing

Re: [PATCH][1/3] Make parallelize_loops static

2015-03-18 Thread Richard Biener
On March 18, 2015 6:21:34 PM GMT+01:00, Tom de Vries tom_devr...@mentor.com wrote: Hi, this patch makes parallelize_loops static. I think the fact that it was extern was a left-over from the times that the parloops pass was not located in tree-parloops.c. bootstrapped and reg-tested on

Re: [PATCH][RFC] Fix PR63155

2015-03-18 Thread Richard Biener
On March 18, 2015 4:59:30 PM GMT+01:00, Alan Lawrence alan.lawre...@arm.com wrote: Following this patch (r221318), we're seeing what appears to be a miscompile of glibc on AArch64. This causes quite a bunch of tests to fail, segfaults etc., if LD_LIBRARY_PATH leads to a libc.so.6 built with

Re: [PATCH][RFC] Fix PR63155

2015-03-18 Thread Andrew Pinski
On Wed, Mar 18, 2015 at 8:59 AM, Alan Lawrence alan.lawre...@arm.com wrote: Following this patch (r221318), we're seeing what appears to be a miscompile of glibc on AArch64. This causes quite a bunch of tests to fail, segfaults etc., if LD_LIBRARY_PATH leads to a libc.so.6 built with that patch

Re: [debug-early] Cache DW_TAG_label's if appropriate

2015-03-18 Thread Jason Merrill
On 03/18/2015 01:03 PM, Aldy Hernandez wrote: By the way Jason, is there any reason we previously called equate_decl_number_to_die() only for abstract instances?: So that multiple concrete instances can find the abstract instance. Why isn't DECL_ABSTRACT_P set the first time you get here for

Re: [PATCH][RFA] [PR rtl-optimization/64317] Enhance postreload-gcse.c to eliminate more redundant loads

2015-03-18 Thread Andrew Pinski
On Tue, Mar 17, 2015 at 11:27 AM, Jeff Law l...@redhat.com wrote: On 03/17/2015 04:35 AM, Richard Biener wrote: I'll test both. In the common case, the cost is going to be the basic bookkeeping so that we can compute the transparent property. The actual computation of transparency and