Re: [Patch, libstdc++/63920] Fix regex_constants::match_not_null behavior

2014-11-25 Thread Tim Shen
On Mon, Nov 24, 2014 at 3:17 AM, Jonathan Wakely jwak...@redhat.com wrote: OK for trunk - thanks. Committed. :) Thanks! -- Regards, Tim Shen

Re: [PATCH] Fix find_base_term in 32-bit -fpic code (PR lto/64025)

2014-11-25 Thread Uros Bizjak
On Tue, Nov 25, 2014 at 8:40 AM, Uros Bizjak ubiz...@gmail.com wrote: On Tue, Nov 25, 2014 at 12:25 AM, Jakub Jelinek ja...@redhat.com wrote: The fallback delegitimization I've added as last option mainly for debug info purposes, when we don't know if the base is a PIC register or say a PIC

Re: [PATCH RFC]Pair load store instructions using a generic scheduling fusion pass

2014-11-25 Thread Bin.Cheng
On Mon, Nov 24, 2014 at 10:28 PM, James Greenhalgh james.greenha...@arm.com wrote: On Fri, Nov 14, 2014 at 02:43:12AM +, Bin.Cheng wrote: On Fri, Nov 7, 2014 at 7:13 AM, Jeff Law l...@redhat.com wrote: On 11/05/14 02:30, Bin.Cheng wrote: Thanks very much for reviewing. I refined the

Re: [PATCH AARCH64]load store pair optimization using sched_fusion pass.

2014-11-25 Thread Bin.Cheng
Ping. Anybody have a look? Thanks, bin On Tue, Nov 18, 2014 at 4:34 PM, Bin Cheng bin.ch...@arm.com wrote: Hi, This is the patch implementing ldp/stp optimization for aarch64. It consists of two parts. The first one is peephole part, which further includes ldp/stp patterns (both peephole

Re: [PATCH] Fix find_base_term in 32-bit -fpic code (PR lto/64025)

2014-11-25 Thread Jakub Jelinek
On Tue, Nov 25, 2014 at 09:13:10AM +0100, Uros Bizjak wrote: On Tue, Nov 25, 2014 at 8:40 AM, Uros Bizjak ubiz...@gmail.com wrote: On Tue, Nov 25, 2014 at 12:25 AM, Jakub Jelinek ja...@redhat.com wrote: The fallback delegitimization I've added as last option mainly for debug info

Re: [PATCH] Fix building of gengtype

2014-11-25 Thread Jakub Jelinek
On Tue, Nov 25, 2014 at 12:35:09AM +0100, Jakub Jelinek wrote: My last 2 bootstraps failed, both because of a race while building host gengtype (each time different gengtype*.o). Found bootstrap failures even with this patch (dunno what changed on my box that I started getting these last night,

Re: [Patch, libstdc++/63497] Avoid dereferencing invalid iterator in regex_executor

2014-11-25 Thread Tim Shen
On Wed, Oct 22, 2014 at 8:19 PM, Tim Shen tims...@google.com wrote: Committed. Thank you too! I'm backporting this patch to gcc-4_9-branch. Do we usually boot test it and then commit directly, or it should be reviewed again? -- Regards, Tim Shen commit

RE: [PATCH, AARCH64] Fix ICE in CCMP (PR64015)

2014-11-25 Thread Zhenqiang Chen
-Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Richard Henderson Sent: Monday, November 24, 2014 4:57 PM To: Zhenqiang Chen; gcc-patches@gcc.gnu.org Cc: Marcus Shawcroft Subject: Re: [PATCH, AARCH64] Fix ICE in CCMP

[PATCH, PR63995, CHKP] Use single static bounds var for varpool nodes sharing asm name

2014-11-25 Thread Ilya Enkovich
Hi, This patch partly fixes PR bootstrap/63995 by avoiding duplicating static bounds vars. With this fix bootstrap still fails at stage 2 and 3 comparison. Bootstrapped and checked on x86_64-unknown-linux-gnu. OK for trunk? Thanks, Ilya -- gcc/ 2014-11-25 Ilya Enkovich

Re: [Patch, libstdc++/63775] Fix regex bracket expression parsing

2014-11-25 Thread Tim Shen
On Wed, Nov 12, 2014 at 11:45 PM, Tim Shen tims...@google.com wrote: Committed with comment fix and slight change on testcase (VERIFY(false) at end of the try block -- must throw). Is it possible to backport this patch to 4.9 branch? It's an important fix, but I'm not sure if there's any

[PATCH, PR64056, i386] Fix chkp tests requiring mempcpy

2014-11-25 Thread Ilya Enkovich
Hi, This patch adds check for mempcpy availability for tests requiring it. Checked with RUNTESTFLAGS=--target_board='unix{-m32,}' i386.exp=chkp-*. OK for trunk? Thanks, Ilya -- 2014-11-25 Ilya Enkovich ilya.enkov...@intel.com PR target/64056 *

Re: [PATCH, ciklplus]: Use -ffloat-store for 32bit x86 in cilk-plus/AN/builtin_fn_{custom,mutating}.c

2014-11-25 Thread Richard Biener
On Mon, Nov 24, 2014 at 10:33 PM, Jeff Law l...@redhat.com wrote: On 11/22/14 11:50, Uros Bizjak wrote: Hello! These two tests fix PR target/63847 [1], where x87 excess precision causes testcase to fail. The problem was triggered by -fpic, please see the PR for analysis. The patch adds

Re: [PATCH] Fix linemap_line_start (PR preprocessor/60436)

2014-11-25 Thread Richard Biener
On Tue, Nov 25, 2014 at 12:22 AM, Jakub Jelinek ja...@redhat.com wrote: Hi! As mentioned in the PR, when preprocessing very large files, if there are huge numbers of lines where no #line is emitted, we might not detect overflowinging into adhoc locations. Apparently in the add_map case we

Re: [PATCH, AARCH64] Fix ICE in CCMP (PR64015)

2014-11-25 Thread Richard Henderson
On 11/25/2014 09:41 AM, Zhenqiang Chen wrote: I want to confirm with you two things before I rework it. (1) expand_insn needs an optab_handler as input. Do I need to define a ccmp_optab with different mode support in optabs.def? No, look again: expand_insn needs an enum insn_code as input.

Re: [PATCH] Add verify_sese

2014-11-25 Thread Richard Biener
On Tue, Nov 25, 2014 at 1:01 AM, Tom de Vries tom_devr...@mentor.com wrote: Richard, I ran into a problem with my oacc kernels directive patch series where tail-merge added another entry into a region that was previously single-entry-single-exit. That resulted in hitting this assert in

Re: [RFC] First steps towards segregating types.

2014-11-25 Thread Richard Biener
On Tue, Nov 25, 2014 at 2:06 AM, Joseph Myers jos...@codesourcery.com wrote: On Mon, 24 Nov 2014, Richard Biener wrote: TREE_LIST should die (with the typical replacement being vecsomething); most lists do not need all the overhead of individually allocated objects with (code, flags, type,

Re: [Patch] Improving jump-thread pass for PR 54742

2014-11-25 Thread Richard Biener
On Mon, Nov 24, 2014 at 11:05 PM, Sebastian Pop seb...@gmail.com wrote: Jeff Law wrote: On 11/23/14 15:22, Sebastian Pop wrote: The second patch attached limits the search for FSM jump threads to loops. With that patch, we are now down to 470 jump threads in an x86_64-linux bootstrap (and

Re: [PATCH, ciklplus]: Use -ffloat-store for 32bit x86 in cilk-plus/AN/builtin_fn_{custom,mutating}.c

2014-11-25 Thread Uros Bizjak
On Tue, Nov 25, 2014 at 10:23 AM, Richard Biener richard.guent...@gmail.com wrote: On Mon, Nov 24, 2014 at 10:33 PM, Jeff Law l...@redhat.com wrote: On 11/22/14 11:50, Uros Bizjak wrote: Hello! These two tests fix PR target/63847 [1], where x87 excess precision causes testcase to fail. The

Re: [PATCH, PR63995, CHKP] Use single static bounds var for varpool nodes sharing asm name

2014-11-25 Thread Richard Biener
On Tue, Nov 25, 2014 at 9:45 AM, Ilya Enkovich enkovich@gmail.com wrote: Hi, This patch partly fixes PR bootstrap/63995 by avoiding duplicating static bounds vars. With this fix bootstrap still fails at stage 2 and 3 comparison. Bootstrapped and checked on x86_64-unknown-linux-gnu. OK

Re: [PATCH, PR64056, i386] Fix chkp tests requiring mempcpy

2014-11-25 Thread Richard Biener
On Tue, Nov 25, 2014 at 10:11 AM, Ilya Enkovich enkovich@gmail.com wrote: Hi, This patch adds check for mempcpy availability for tests requiring it. Checked with RUNTESTFLAGS=--target_board='unix{-m32,}' i386.exp=chkp-*. OK for trunk? Ok. Thanks, Richard. Thanks, Ilya --

Re: [PATCH] Fix regressions in libgomp testsuite: set flag_fat_lto_objects for offload

2014-11-25 Thread Richard Biener
On Mon, Nov 24, 2014 at 5:44 PM, Ilya Verbin iver...@gmail.com wrote: On 17 Nov 10:57, Richard Biener wrote: On Fri, Nov 14, 2014 at 6:08 PM, Ilya Verbin iver...@gmail.com wrote: On 14 Nov 09:01, H.J. Lu wrote: On Fri, Nov 14, 2014 at 8:51 AM, Ilya Verbin iver...@gmail.com wrote: On 14

Re: [PATCH, PR63995, CHKP] Use single static bounds var for varpool nodes sharing asm name

2014-11-25 Thread Ilya Enkovich
2014-11-25 12:43 GMT+03:00 Richard Biener richard.guent...@gmail.com: On Tue, Nov 25, 2014 at 9:45 AM, Ilya Enkovich enkovich@gmail.com wrote: Hi, This patch partly fixes PR bootstrap/63995 by avoiding duplicating static bounds vars. With this fix bootstrap still fails at stage 2 and 3

Re: [Patch] Improving jump-thread pass for PR 54742

2014-11-25 Thread Markus Trippelsdorf
On 2014.11.24 at 22:05 +, Sebastian Pop wrote: I got my scripts installed on the gcc-farm. I first used an x86_64 gcc75 and valgrind was crashing not recognizing how to decode an instruction. Then I moved to gcc112 a powerpc64-linux where I got this data from stage2 cc1plus compiling the

Re: [Patch, libstdc++/63497] Avoid dereferencing invalid iterator in regex_executor

2014-11-25 Thread Jonathan Wakely
On 25/11/14 00:41 -0800, Tim Shen wrote: On Wed, Oct 22, 2014 at 8:19 PM, Tim Shen tims...@google.com wrote: Committed. Thank you too! I'm backporting this patch to gcc-4_9-branch. Do we usually boot test it and then commit directly, or it should be reviewed again? I approved it for the

Re: [PATCH, ciklplus]: Use -ffloat-store for 32bit x86 in cilk-plus/AN/builtin_fn_{custom,mutating}.c

2014-11-25 Thread Uros Bizjak
On Tue, Nov 25, 2014 at 10:38 AM, Uros Bizjak ubiz...@gmail.com wrote: These two tests fix PR target/63847 [1], where x87 excess precision causes testcase to fail. The problem was triggered by -fpic, please see the PR for analysis. The patch adds -ffloat-store for 32bit x86 target, a

RE: [PATCH] Fix PR ipa/61190, updated

2014-11-25 Thread Bernd Edlinger
Hi Honza, On Mon, 24 Nov 2014 16:57:42 +0100, Jan Hubicka wrote: +cgraph_node::call_for_symbol_thunks_and_aliases_1 (bool (*callback) + (cgraph_node *, void *), + void *data, + bool include_overwritable, + bool exclude_virtual_thunks) Instead of adding _1 variant into public API, please

Re: [PATCH, PR63995, CHKP] Use single static bounds var for varpool nodes sharing asm name

2014-11-25 Thread Richard Biener
On Tue, Nov 25, 2014 at 11:19 AM, Ilya Enkovich enkovich@gmail.com wrote: 2014-11-25 12:43 GMT+03:00 Richard Biener richard.guent...@gmail.com: On Tue, Nov 25, 2014 at 9:45 AM, Ilya Enkovich enkovich@gmail.com wrote: Hi, This patch partly fixes PR bootstrap/63995 by avoiding

Re: [PATCH, 1/8] Expand oacc kernels after pass_build_ealias

2014-11-25 Thread Tom de Vries
On 24-11-14 11:56, Tom de Vries wrote: On 15-11-14 18:19, Tom de Vries wrote: On 15-11-14 13:14, Tom de Vries wrote: Hi, I'm submitting a patch series with initial support for the oacc kernels directive. The patch series uses pass_parallelize_loops to implement parallelization of loops in

Re: [PATCH, 2/8] Add pass_oacc_kernels

2014-11-25 Thread Tom de Vries
On 15-11-14 18:20, Tom de Vries wrote: On 15-11-14 13:14, Tom de Vries wrote: Hi, I'm submitting a patch series with initial support for the oacc kernels directive. The patch series uses pass_parallelize_loops to implement parallelization of loops in the oacc kernels region. The patch series

Re: [PATCH, 3/8] Add pass_ch_oacc_kernels to pass_oacc_kernels

2014-11-25 Thread Tom de Vries
On 15-11-14 18:21, Tom de Vries wrote: On 15-11-14 13:14, Tom de Vries wrote: Hi, I'm submitting a patch series with initial support for the oacc kernels directive. The patch series uses pass_parallelize_loops to implement parallelization of loops in the oacc kernels region. The patch series

Re: [PATCH, 4/8] Add pass_tree_loop_{init,done} to pass_oacc_kernels

2014-11-25 Thread Tom de Vries
On 15-11-14 18:21, Tom de Vries wrote: On 15-11-14 13:14, Tom de Vries wrote: Hi, I'm submitting a patch series with initial support for the oacc kernels directive. The patch series uses pass_parallelize_loops to implement parallelization of loops in the oacc kernels region. The patch series

Re: [PATCH, 5/8] Add pass_loop_im to pass_oacc_kernels

2014-11-25 Thread Tom de Vries
On 15-11-14 18:22, Tom de Vries wrote: On 15-11-14 13:14, Tom de Vries wrote: Hi, I'm submitting a patch series with initial support for the oacc kernels directive. The patch series uses pass_parallelize_loops to implement parallelization of loops in the oacc kernels region. The patch series

Re: [PATCH, 6/8] Add pass_ccp to pass_oacc_kernels

2014-11-25 Thread Tom de Vries
On 15-11-14 18:22, Tom de Vries wrote: On 15-11-14 13:14, Tom de Vries wrote: Hi, I'm submitting a patch series with initial support for the oacc kernels directive. The patch series uses pass_parallelize_loops to implement parallelization of loops in the oacc kernels region. The patch series

Re: [PATCH, 7/8] Add pass_parloops_oacc_kernels to pass_oacc_kernels

2014-11-25 Thread Tom de Vries
On 15-11-14 18:23, Tom de Vries wrote: On 15-11-14 13:14, Tom de Vries wrote: Hi, I'm submitting a patch series with initial support for the oacc kernels directive. The patch series uses pass_parallelize_loops to implement parallelization of loops in the oacc kernels region. The patch series

[C++ Patch] PR 63786

2014-11-25 Thread Paolo Carlini
Hi, we are crashing on this kind of invalid code because we don't early check the case with check_for_bare_parameter_packs. Tested x86_64-linux. Thanks, Paolo. // /cp 2014-11-25 Paolo Carlini paolo.carl...@oracle.com PR c++/63786 * parser.c

Re: [PATCH, PR63995, CHKP] Use single static bounds var for varpool nodes sharing asm name

2014-11-25 Thread Ilya Enkovich
2014-11-25 14:11 GMT+03:00 Richard Biener richard.guent...@gmail.com: On Tue, Nov 25, 2014 at 11:19 AM, Ilya Enkovich enkovich@gmail.com wrote: 2014-11-25 12:43 GMT+03:00 Richard Biener richard.guent...@gmail.com: On Tue, Nov 25, 2014 at 9:45 AM, Ilya Enkovich enkovich@gmail.com

[PATCH] Remove unnecessary calls to strchr.

2014-11-25 Thread Ilya Tocar
Hi, As proposed in https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63853 this patch replaces some function calls with pointer arithmetic. I didn't mention PR in Changelog, as they are not actually related. Ok for trunk? gcc/ * gcc.c (handle_foffload_option): Remove unnecessary calls to

Re: [PATCH] Remove unnecessary calls to strchr.

2014-11-25 Thread Jakub Jelinek
On Tue, Nov 25, 2014 at 03:15:04PM +0300, Ilya Tocar wrote: As proposed in https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63853 this patch replaces some function calls with pointer arithmetic. I didn't mention PR in Changelog, as they are not actually related. Ok for trunk? @@ -3408,8 +3408,7

[PATCH] sreal class fix for PR64050 and PR64060

2014-11-25 Thread Martin Liška
Hello. Following patch is fix sreal problems that are mentioned in PR64050, PR64060. I added new GCC plugin test where I test sreal arithmetics and number comparison. Patch can bootstrap on ppc64-linux-pc and x86_64-linux-pc and can pass regression tests. Thanks, Martin gcc/ChangeLog:

PATCH: PR rtl-optimization/64037: Miscompilation with -Os and enum class : char parameter

2014-11-25 Thread H.J. Lu
Hi, The enclosed testcase fails on x86 when compiled with -Os since we pass a byte parameter with a byte load in caller and read it as an int in callee. The reason it only shows up with -Os is x86 backend encodes a byte load with an int load if -O isn't used. When a byte load is used, the upper

Re: [PATCH v2] gcc/c-family/c-cppbuiltin.c: Let buffer enough to print host wide integer value

2014-11-25 Thread Chen Gang
On 11/25/14 7:56, Joseph Myers wrote: On Sun, 23 Nov 2014, Chen Gang wrote: + gcc_assert (wi::fits_to_tree_p(value, integer_type_node)); Watch formatting: space before '(' in the wi::fits_to_tree_p call. Applies elsewhere in this patch as well. OK, thanks, I shall notice next. When

Re: [C++ Patch] PR 63786

2014-11-25 Thread Jason Merrill
OK. Jason

[PATCH] Fix PR61927

2014-11-25 Thread Richard Biener
I am testing the following patch which reverts order of group and pattern analysis to 4.8 state. It doesn't really matter but it avoids pattern analysis to know about groups which its failure causes the wrong-code in the PR. Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Help

[PATCH] Fix PR64065(?)

2014-11-25 Thread Richard Biener
The following might fix PR64065 but is certainly a bug. Bootstrap and regtest running on x86_64-unknown-linux-gnu. Richard. 2014-11-25 Richard Biener rguent...@suse.de PR lto/64065 * lto-streamer-out.c (output_struct_function_base): Stream last_clique field.

Re: [PATCH] rs6000: Replace a stray addic with addi

2014-11-25 Thread David Edelsohn
On Mon, Nov 24, 2014 at 10:18 PM, Segher Boessenkool seg...@kernel.crashing.org wrote: Tested as usual... okay for trunk? Segher 2014-11-24 Segher Boessenkool seg...@kernel.crashing.org gcc/ * config/rs6000/sysv4.h (ASM_OUTPUT_REG_POP): Use addi instead of addic.

Re: [PATCH] rs6000: Remove iorxor/IORXOR code attrs

2014-11-25 Thread David Edelsohn
On Mon, Nov 24, 2014 at 10:11 PM, Segher Boessenkool seg...@kernel.crashing.org wrote: As Richard pointed out, those do nothing more than code/CODE. Tested etc.; okay for trunk? Segher 2014-11-21 Segher Boessenkool seg...@kernel.crashing.org gcc/ * config/rs6000/rs6000.md

Re: [PATCH] AIX: Filename-based shared library versioning for libgcc_s

2014-11-25 Thread David Edelsohn
On Tue, Nov 11, 2014 at 10:42 AM, Michael Haubenwallner michael.haubenwall...@ssi-schaefer.com wrote: On 11/11/2014 04:02 PM, David Edelsohn wrote: Michael, Why does the configure change match with p*-*-aix... instead of power* or powerpc*? Yes, it's unique and will match, but why make it

[PATCH][AArch64]Fix ICE at -O0 on vld1_lane intrinsics

2014-11-25 Thread Alan Lawrence
vld1_lane intrinsics ICE at -O0 because they contain a call to the vset_lane intrinsics, through which the lane index is not constant-propagated. (They are fine at -O1 and higher!). This fixes the ICE by replacing said call by a macro. Rather than defining many individual macros

[PATCH] Fix PR62238

2014-11-25 Thread Richard Biener
I will test the following patch fixing a tree sharing issue in PR62238 and plugging a SSA name leak. The issue here is that force_gimple_operand and friends modify trees in-place, injecting SSA name uses to them. If you end up not emitting their definitions or and up re-using those trees in not

[PATCH] libgcc: Add CFI directives to the floating point support code for ARM.

2014-11-25 Thread Martin Galvan
This patch adds CFI directives to the floating point support code for ARM. Previously, if we tried to do a backtrace from that code in a debug session we'd get something like this: (gdb) bt #0 __nedf2 () at ../../../../../../gcc-4.9.2/libgcc/config/arm/ieee754-df.S:1082 #1 0x0db6 in

Re: [PATCH] gcc parallel make check

2014-11-25 Thread Tom de Vries
On 15-09-14 18:05, Jakub Jelinek wrote: libstdc++-v3/ * testsuite/Makefile.am (check_p_numbers0, check_p_numbers1, check_p_numbers2, check_p_numbers3, check_p_numbers4, check_p_numbers5, check_p_numbers6, check_p_numbers, check_p_subdirs): New variables.

C++ PATCH to lookup_template_variable

2014-11-25 Thread Jason Merrill
We need to use unknown_type_node for non-dependent arguments, too; we don't know what type the variable has until we look up the specialization. Tested x86_64-pc-linux-gnu, applying to trunk. commit c348ed4ea7152054ff623a3efbca7fab49227a5f Author: Jason Merrill ja...@redhat.com Date: Mon Nov

RE: [PATCH][MIPS] Fix P5600 memory cost

2014-11-25 Thread Matthew Fortune
Hi Prachi, OK with fixes to the changelog entry: latency not latency. Remember to tab in the changelog entry and split the line as it will exceed 80 chars. Also two spaces between the date/name and name/email. E.g. 2014-11-05 Prachi Godbole prachi.godb...@imgtec.com *

Re: [PATCH] sreal class fix for PR64050 and PR64060

2014-11-25 Thread Richard Biener
On Tue, Nov 25, 2014 at 1:55 PM, Martin Liška mli...@suse.cz wrote: Hello. Following patch is fix sreal problems that are mentioned in PR64050, PR64060. I added new GCC plugin test where I test sreal arithmetics and number comparison. Patch can bootstrap on ppc64-linux-pc and

Re: [PATCH] Add verify_sese

2014-11-25 Thread Tom de Vries
On 25-11-14 10:28, Richard Biener wrote: On Tue, Nov 25, 2014 at 1:01 AM, Tom de Vries tom_devr...@mentor.com wrote: Richard, I ran into a problem with my oacc kernels directive patch series where tail-merge added another entry into a region that was previously single-entry-single-exit. That

Re: PATCH: PR rtl-optimization/64037: Miscompilation with -Os and enum class : char parameter

2014-11-25 Thread Richard Biener
On Tue, Nov 25, 2014 at 1:57 PM, H.J. Lu hongjiu...@intel.com wrote: Hi, The enclosed testcase fails on x86 when compiled with -Os since we pass a byte parameter with a byte load in caller and read it as an int in callee. The reason it only shows up with -Os is x86 backend encodes a byte

Re: PATCH: PR rtl-optimization/64037: Miscompilation with -Os and enum class : char parameter

2014-11-25 Thread Richard Biener
On Tue, Nov 25, 2014 at 4:01 PM, Richard Biener richard.guent...@gmail.com wrote: On Tue, Nov 25, 2014 at 1:57 PM, H.J. Lu hongjiu...@intel.com wrote: Hi, The enclosed testcase fails on x86 when compiled with -Os since we pass a byte parameter with a byte load in caller and read it as an int

[PATCH, MIPS, COMMITTED] Testsuite fixes for soft-float configurations

2014-11-25 Thread Matthew Fortune
Re: https://gcc.gnu.org/ml/gcc-patches/2014-11/msg02879.html The new FPXX tests now work correctly for soft-float configurations. Tests should only need to specify one of the 5 floating-point options and any other options are then inferred from that. The FPXX tests were the first tests to really

Re: [PATCH 1/2] teach mklog to get name / email from git config when available

2014-11-25 Thread Diego Novillo
On 20/11/2014, 16:51 , Tom de Vries wrote: OK for trunk? This is fine. Thanks. Diego.

Re: [PATCH] Add verify_sese

2014-11-25 Thread Richard Biener
On Tue, Nov 25, 2014 at 3:59 PM, Tom de Vries tom_devr...@mentor.com wrote: On 25-11-14 10:28, Richard Biener wrote: On Tue, Nov 25, 2014 at 1:01 AM, Tom de Vries tom_devr...@mentor.com wrote: Richard, I ran into a problem with my oacc kernels directive patch series where tail-merge added

Re: PATCH: PR rtl-optimization/64037: Miscompilation with -Os and enum class : char parameter

2014-11-25 Thread H.J. Lu
On Tue, Nov 25, 2014 at 7:01 AM, Richard Biener richard.guent...@gmail.com wrote: On Tue, Nov 25, 2014 at 1:57 PM, H.J. Lu hongjiu...@intel.com wrote: Hi, The enclosed testcase fails on x86 when compiled with -Os since we pass a byte parameter with a byte load in caller and read it as an int

Re: [PATCH][AArch64] Remove crypto extension from default for cortex-a53, cortex-a57

2014-11-25 Thread Kyrill Tkachov
On 25/11/14 01:36, Gerald Pfeifer wrote: On Tuesday 2014-11-18 09:38, Kyrill Tkachov wrote: Here's what I propose. + li The cryptographic extensions to the ARMv8-A architecture are no + longer enabled by default when specifying the + code-mcpu=cortex-a53/code,

Re: [Patch] Improving jump-thread pass for PR 54742

2014-11-25 Thread Jeff Law
On 11/24/14 21:55, Jeff Law wrote: On 11/24/14 18:09, Sebastian Pop wrote: Sebastian Pop wrote: I removed the return -1 and started a bootstrap on powerpc64-linux. Bootstrap passed on top of the 4 previous patches on powerpc64-linux. I will report the valgrind output. The output from

Re: [PATCH] crtstuff: Add missing semicolon

2014-11-25 Thread Jeff Law
On 11/24/14 20:44, Segher Boessenkool wrote: I wonder how this survived so long, I must be building some strange configs (it failed on an avr cross). Okay for trunk? Segher 2014-11-24 Segher Boessenkool seg...@kernel.crashing.org libgcc/ * crtstuff.c (__do_glbal_ctors_1): Add

[PATCH] pr31397 - implement -Wsuggest-override

2014-11-25 Thread tsaunders
From: Trevor Saunders tsaund...@mozilla.com Hi, this is a new warning to find places where virtual functions are over ridden, but not marked override. included test passes, I expect comments so regtest is pending, and ChangeLog is omitted. Trev --- gcc/c-family/c.opt

Re: [PATCH] Fix a bug in bt-load.c

2014-11-25 Thread Jeff Law
On 11/24/14 20:56, Segher Boessenkool wrote: This caused ICEs on sh64. `min_cost' and `def' here are supposed to refer to the same element; removing it from the heap before asking the heap for the key doesn't work (and at the end of the loop here we will ask for min_key on an empty heap, which

Re: [PATCH] mn10300: Fix an ICE

2014-11-25 Thread Jeff Law
On 11/24/14 20:37, Segher Boessenkool wrote: `lcc' is not an insn but just a pattern. This caused a build error in libgcc. Tested with a cross compiler build (which fails without and succeeds with the patch). Not tested much more; this compiler really likes to ICE, something with ipa-icf. Is

Re: [PING][PATCH] Change contrib/test_installed for testing cross compilers

2014-11-25 Thread Jeff Law
On 11/24/14 09:51, Alan Lawrence wrote: Having just been experimenting with testing of installed compilers - yes something like this could be useful, however: to do cross-testing I found I also (a) had to set my target_list; so either an extra flag for that, or maybe just a generic

[PATCH] Enhance ASAN_CHECK optimization

2014-11-25 Thread Yury Gribov
Hi all, This patch improves current optimization of ASAN_CHECKS performed by sanopt pass. In addition to searching the sanitized pointer in asan_check_map, it also tries to search for definition of this pointer. This allows more checks to be dropped when definition is not a gimple value

Re: [PATCH] mn10300: Fix an ICE

2014-11-25 Thread Segher Boessenkool
On Tue, Nov 25, 2014 at 09:44:35AM -0700, Jeff Law wrote: On 11/24/14 20:37, Segher Boessenkool wrote: `lcc' is not an insn but just a pattern. This caused a build error in libgcc. A good example of a case that would have been caught if we get to a point where stuff in the insn chain are

Re: [PATCH] mn10300: Fix an ICE

2014-11-25 Thread Jeff Law
On 11/25/14 10:14, Segher Boessenkool wrote: On Tue, Nov 25, 2014 at 09:44:35AM -0700, Jeff Law wrote: On 11/24/14 20:37, Segher Boessenkool wrote: `lcc' is not an insn but just a pattern. This caused a build error in libgcc. A good example of a case that would have been caught if we get

Re: [PATCH] Fix a bug in bt-load.c

2014-11-25 Thread Segher Boessenkool
On Tue, Nov 25, 2014 at 09:41:40AM -0700, Jeff Law wrote: On 11/24/14 20:56, Segher Boessenkool wrote: This caused ICEs on sh64. `min_cost' and `def' here are supposed to refer to the same element; removing it from the heap before asking the heap for the key doesn't work (and at the end of

Re: [PATCH] Fix a bug in bt-load.c

2014-11-25 Thread Jeff Law
On 11/25/14 10:26, Segher Boessenkool wrote: On Tue, Nov 25, 2014 at 09:41:40AM -0700, Jeff Law wrote: On 11/24/14 20:56, Segher Boessenkool wrote: This caused ICEs on sh64. `min_cost' and `def' here are supposed to refer to the same element; removing it from the heap before asking the heap

Re: [Patch] Improving jump-thread pass for PR 54742

2014-11-25 Thread Sebastian Pop
Jeff Law wrote: On 11/24/14 21:55, Jeff Law wrote: On 11/24/14 18:09, Sebastian Pop wrote: Sebastian Pop wrote: I removed the return -1 and started a bootstrap on powerpc64-linux. Bootstrap passed on top of the 4 previous patches on powerpc64-linux. I will report the valgrind output.

[PATCH, libgfortran]: Remove unused variable

2014-11-25 Thread Uros Bizjak
Hello! 2014-11-25 Uros Bizjak ubiz...@gmail.com * intrinsics/env.c (getenv): Remove unused variable res_len. Bootstrapped on x86_64-linux-gnu. Almost trivial, but ... OK for mainline? Uros. Index: intrinsics/env.c === ---

Re: [PATCH, libgfortran]: Remove unused variable

2014-11-25 Thread Steve Kargl
On Tue, Nov 25, 2014 at 07:17:17PM +0100, Uros Bizjak wrote: 2014-11-25 Uros Bizjak ubiz...@gmail.com * intrinsics/env.c (getenv): Remove unused variable res_len. Bootstrapped on x86_64-linux-gnu. Almost trivial, but ... OK for mainline? Yes. -- Steve

Re: [PATCH 2/5] combine: handle I2 a parallel of two SETs

2014-11-25 Thread Jeff Law
On 11/14/14 12:19, Segher Boessenkool wrote: If I2 is a PARALLEL of two SETs, split it into two instructions, I1 and I2. If there already was an I1, rename it to I0. If there already was an I0, don't do anything. This surprisingly simple patch is enough to let combine handle such PARALLELs

Re: [PATCH 3/5] combine: add regno field to LOG_LINKS

2014-11-25 Thread Jeff Law
On 11/14/14 12:19, Segher Boessenkool wrote: With this new field in place, we can have LOG_LINKS for insns that set more than one register and distribute them properly in distribute_links. This then allows many more PARALLELs to be combined. Also split off new functions can_combine_{def,use}_p

Re: Document __builtin_*_overflow

2014-11-25 Thread Gerald Pfeifer
Hi Jakub, On Wednesday 2014-11-12 14:13, Jakub Jelinek wrote: This patch mentions __builtin_*_overflow in gcc-5/changes.html. Ok for CVS? I've fallen a bit behind with GCC patches, sorry. What do you think about this follow-up patch on top of yours? Gerald Index: changes.html

[ping^4] [libgomp] make it possible to use OMP on both sides of a fork

2014-11-25 Thread Nathaniel Smith
Ping^4 for: https://gcc.gnu.org/ml/gcc-patches/2014-11/msg00519.html On Tue, Nov 18, 2014 at 12:53 AM, Nathaniel Smith n...@pobox.com wrote: Hello, Ping for https://gcc.gnu.org/ml/gcc-patches/2014-11/msg00519.html Patches posted early enough during Stage 1 and not yet fully reviewed may

Re: [patch] Restore bootstrap on powerpc*-apple-darwin*

2014-11-25 Thread FX
2014-11-24 Rohit rohitarul...@freescale.com PR bootstrap/63703 * config/rs6000/darwin.h (REGISTER_NAMES): Update based on 32 newly added GCC hard register numbers for SPE high registers. IMO, it's obvious, and as you say, doesn't touch any other target. After further

Re: Document __builtin_*_overflow

2014-11-25 Thread Jakub Jelinek
On Tue, Nov 25, 2014 at 07:50:02PM +0100, Gerald Pfeifer wrote: Hi Jakub, On Wednesday 2014-11-12 14:13, Jakub Jelinek wrote: This patch mentions __builtin_*_overflow in gcc-5/changes.html. Ok for CVS? I've fallen a bit behind with GCC patches, sorry. What do you think about this

[PATCH, libobjc]: Remove ‘...’ is static but used in inline function ‘...’ which is not static

2014-11-25 Thread Uros Bizjak
Hello! Recently, gcc bootstrap started to emit following warnings when building libobjc: libobjc/sendmsg.c:338:13: warning: ‘get_implementation’ is static but used in inline function ‘get_imp’ which is not static libobjc/sendmsg.c:335:15: warning: ‘sarray_get_safe’ is static but used in inline

Re: [PATCH 4/5] combine: distribute_log_links for PARALLELs of SETs

2014-11-25 Thread Jeff Law
On 11/14/14 12:19, Segher Boessenkool wrote: Now that LOG_LINKS are per regno, we can distribute them on PARALLELs just fine. Do so. This makes PARALLELs not lose their LOG_LINKS early when e.g. a trivial reg-reg move is combined, so that they can be used in more useful combinations as well.

Re: [patch c++]: Fix PR/53904

2014-11-25 Thread Jason Merrill
On 11/20/2014 02:48 PM, Kai Tietz wrote: this issue fixes a type-overflow issue caused by trying to cast a UHWI via tree_to_shwi. As soon as value gets larger then SHWI_MAX, we get an error for it. So we need to cast it via tree_to_uhwi, and then casting it to the signed variant. The problem

[PATCH v3] gcc/c-family/c-cppbuiltin.c: Let buffer enough to print host wide integer value

2014-11-25 Thread Chen Gang
The original length 18 is not enough for HOST_WIDE_INT printing, need use 20 instead of. Also need additional bytes for printing related prefix and suffix, and give a related check. It passes testsuite under fedora 20 x86_64-unknown-linux-gnu. 2014-11-26 Chen Gang gang.chen.5...@gmail.com

Re: [PATCH, libobjc]: Remove ‘...’ is static but used in inline function ‘...’ which is not static

2014-11-25 Thread Andrew Pinski
On Tue, Nov 25, 2014 at 11:09 AM, Uros Bizjak ubiz...@gmail.com wrote: Hello! Recently, gcc bootstrap started to emit following warnings when building libobjc: libobjc/sendmsg.c:338:13: warning: ‘get_implementation’ is static but used in inline function ‘get_imp’ which is not static

Re: PATCH: PR rtl-optimization/64037: Miscompilation with -Os and enum class : char parameter

2014-11-25 Thread H.J. Lu
On Tue, Nov 25, 2014 at 7:04 AM, Richard Biener richard.guent...@gmail.com wrote: On Tue, Nov 25, 2014 at 4:01 PM, Richard Biener richard.guent...@gmail.com wrote: On Tue, Nov 25, 2014 at 1:57 PM, H.J. Lu hongjiu...@intel.com wrote: Hi, The enclosed testcase fails on x86 when compiled with

Re: [PATCH] gcc parallel make check

2014-11-25 Thread Jakub Jelinek
On Tue, Nov 25, 2014 at 03:27:40PM +0100, Tom de Vries wrote: This patch fixes that by ensuring that we print that unsupported message only once. The resulting test result comparison diff is: 2014-11-25 Tom de Vries t...@codesourcery.com *

patch to fix PR63527

2014-11-25 Thread Vladimir Makarov
The following patch fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63527 The patch was tested and bootstrapped on x86/x86-64. Committed as rev. 218509. https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63527 Index: ira-lives.c ===

Re: [PATCH] mn10300: Fix an ICE

2014-11-25 Thread David Malcolm
On Tue, 2014-11-25 at 10:15 -0700, Jeff Law wrote: On 11/25/14 10:14, Segher Boessenkool wrote: On Tue, Nov 25, 2014 at 09:44:35AM -0700, Jeff Law wrote: On 11/24/14 20:37, Segher Boessenkool wrote: `lcc' is not an insn but just a pattern. This caused a build error in libgcc. A good

Re: [patch, build] Restore bootstrap in building libcc1 on darwin

2014-11-25 Thread Mike Stump
On Nov 23, 2014, at 4:06 PM, FX fxcoud...@gmail.com wrote: One question to build maintainers, and one patch submitted to top-level configure.ac So, not sure who wants to review this. From the darwin perspective, Ok.

Re: patch to fix PR63527

2014-11-25 Thread H.J. Lu
On Tue, Nov 25, 2014 at 12:22 PM, Vladimir Makarov vmaka...@redhat.com wrote: The following patch fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63527 The patch was tested and bootstrapped on x86/x86-64. Committed as rev. 218509. https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63527 I

Re: [PATCH] Fix PR ipa/61190, updated

2014-11-25 Thread Jan Hubicka
Index: gcc/ipa-pure-const.c === --- gcc/ipa-pure-const.c (revision 215888) +++ gcc/ipa-pure-const.c (working copy) @@ -744,6 +744,8 @@ analyze_function (struct cgraph_node *fn, bool ipa { /* Thunk gets propagated

Re: [patch, build] Restore bootstrap in building libcc1 on darwin

2014-11-25 Thread Phil Muldoon
On 25/11/14 20:37, Mike Stump wrote: On Nov 23, 2014, at 4:06 PM, FX fxcoud...@gmail.com wrote: One question to build maintainers, and one patch submitted to top-level configure.ac So, not sure who wants to review this. From the darwin perspective, Ok. I mean from my limited viewpoint it

Re: [PATCH] pr61324 pr 63649 - fix crash in ipa_comdats

2014-11-25 Thread Jan Hubicka
From: Trevor Saunders tsaund...@mozilla.com Hi, the interesting symbol in the test case for pr61324 is __GLOBAL__sub_I_s. It refers to nothing, and is called by nothing, however it is kept (I believe because of -fkeep-inline-functions). That means ipa_comdats never tries to put Aha,

Re: patch to fix PR63527

2014-11-25 Thread H.J. Lu
On Tue, Nov 25, 2014 at 12:54 PM, H.J. Lu hjl.to...@gmail.com wrote: On Tue, Nov 25, 2014 at 12:22 PM, Vladimir Makarov vmaka...@redhat.com wrote: The following patch fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63527 The patch was tested and bootstrapped on x86/x86-64. Committed

Re: [Patch] Improving jump-thread pass for PR 54742

2014-11-25 Thread Sebastian Pop
Sebastian Pop wrote: I will bootstrap and regression test this patch on x86_64-linux and powerpc64-linux. I will also run it on our internal benchmarks, coremark, and the llvm test-suite. I will also include a longer testcase that makes sure we do not regress on coremark. Done all the

Re: [Patch, Fortran] convert almost all {warning,error}_now to common diagnostic

2014-11-25 Thread FX
(a) those majority which might need buffering (gfc_error, gfc_warning); Is there a plan for those in the longer term? Bootstrapped and regtested on x86-64-gnu-linux. OK for the trunk? OK

Re: [PATCH 3/5] combine: add regno field to LOG_LINKS

2014-11-25 Thread Segher Boessenkool
On Tue, Nov 25, 2014 at 11:46:52AM -0700, Jeff Law wrote: On 11/14/14 12:19, Segher Boessenkool wrote: With this new field in place, we can have LOG_LINKS for insns that set more than one register and distribute them properly in distribute_links. This then allows many more PARALLELs to be

  1   2   >