Re: [PATCH 0/5] Yet more use of rtx subclasses; possible rl78 bug

2014-09-09 Thread DJ Delorie
> > I fixed this (in patch 5) by introducing a new local rtx "set" for the > > result of single_set, and hence not overwriting "insn" within the loop. > > That said I've only tested that it compiles for rl78, I've not yet > > forced line 3605 to execute, and not simulated the resulting code. > See

[jit] New test case: test-nested-loops.c

2014-09-09 Thread David Malcolm
Committed to branch dmalcolm/jit: gcc/testsuite/ChangeLog.jit: * jit.dg/test-nested-loops.c: New test case. * jit.dg/all-non-failing-tests.h: Add test-nested-loops.c. * jit.dg/test-combination.c (create_code): Likewise. (verify_code): Likewise. * jit.dg/test

Re: RFA: Merge definitions of get_some_local_dynamic_name

2014-09-09 Thread Rainer Orth
Hi Richard, >> It seems the new get_some_local_dynamic_name implementation in >> function.c lost the non-NULL check the sparc.c version had. I'm >> currently testing the following patch: > > Could you do a: > > call debug_rtx (...) > > on the insn that contains a null pointer? Normally insn pa

Re: RFA: Merge definitions of get_some_local_dynamic_name

2014-09-09 Thread Richard Sandiford
Rainer Orth writes: > Hi Richard, >>> It seems the new get_some_local_dynamic_name implementation in >>> function.c lost the non-NULL check the sparc.c version had. I'm >>> currently testing the following patch: >> >> Could you do a: >> >> call debug_rtx (...) >> >> on the insn that contains a

Re: [PATCH ARM] Fix PR target/63209

2014-09-09 Thread Xinliang David Li
Richard, thanks for the review. The revised patch is attached. Is this one OK (after testing is done)? David 2014-09-08 Xinliang David Li PR target/63209 * config/arm/arm.md (movcond_addsi): Handle case where source and target operands are the same 2014-09-08 Xinlia

Re: [PATCH 5/5] single_set takes an insn

2014-09-09 Thread Jeff Law
On 09/09/14 11:34, David Malcolm wrote: [dropping DJ from CC as this doesn't relate to rl78] [adding Denis to CC due to avr-related discussion] The reason for the as_a here is that avr_out_plus and avr_out_bitop are mostly called with insns [1], but are also called with SET patterns [2], which

[PATCH] rs6000: Use xori for HTM builtins and vector compares

2014-09-09 Thread Segher Boessenkool
These patterns use subfic now. subfic clobbers the carry. Other code already preferably uses xor. Let's do the same here. Bootstrapped and tested as usual, no regressions. Is this okay to apply? Segher 2014-09-09 Segher Boessenkool * config/rs6000/htm.md (tabort, tabortdc, tabo

Re: Fix for "FAIL: tmpdir-gcc.dg-struct-layout-1/t028 c_compat_x_tst.o compile, (internal compiler error)"

2014-09-09 Thread Vladimir Makarov
On 09/05/2014 10:51 AM, David Sherwood wrote: > Hi, > > I have a potential fix for a gcc testsuite failure for aarch64 in big > endian, i.e. > > FAIL: tmpdir-gcc.dg-struct-layout-1/t028 c_compat_x_tst.o compile, (internal > compiler error) > FAIL: tmpdir-gcc.dg-struct-layout-1/t028 c_compat_y_tst.o

Re: auto generate cpp_reason to gcc OPT_W table

2014-09-09 Thread Joseph S. Myers
On Fri, 5 Sep 2014, Manuel L?pez-Ib??ez wrote: > This adds a new option property CppReason which maps to a warning > reason code in cpplib.h. This allows us to auto-generate > cpp_reason_option_codes[], which maps from CPP warning codes to GCC > ones, thus making a bit harder to forget to update t

Re: [patch] No allocation for empty unordered containers

2014-09-09 Thread François Dumont
On 09/09/2014 19:29, Jonathan Wakely wrote: On 14/08/14 21:22 +0200, François Dumont wrote: I am preparing a patch for profile mode so I will submit modification for this mode with this big patch. btw, François, for profile mode I think we should just do something like this patch. I feel quit

Re: [Patch, libgfortran, 4.5] PR25561, 37754: New low level I/O library

2014-09-09 Thread Janne Blomqvist
On Fri, Sep 5, 2014 at 6:52 PM, Thomas Schwinge wrote: > Hi! > > On Tue, 06 Jan 2009 00:10:02 +0200, Janne Blomqvist > wrote: >> attached is a substantially reworked low level I/O library for gfortran. >> [...] > > Due to a recent improvement in GCC, it is now pointing out an issue in > the foll

Re: [patch] No allocation for empty unordered containers

2014-09-09 Thread Jonathan Wakely
On 09/09/14 23:03 +0200, François Dumont wrote: On 09/09/2014 19:29, Jonathan Wakely wrote: On 14/08/14 21:22 +0200, François Dumont wrote: I am preparing a patch for profile mode so I will submit modification for this mode with this big patch. btw, François, for profile mode I think we shoul

[PING][PATCH] Power/GCC: Fix e500 vs non-e500 register save slot issue

2014-09-09 Thread Maciej W. Rozycki
On Mon, 1 Sep 2014, Maciej W. Rozycki wrote: > This fixes an issue with the mode used for register save slots on the > stack where e500 processor support is enabled along non-e500 multilibs. > In that case the HARD_REGNO_CALLER_SAVE_MODE macro definition from > gcc/config/rs6000/e500.h overri

Re: [Ping v2][PATCH] Add patch for debugging compiler ICEs.

2014-09-09 Thread Joseph S. Myers
On Thu, 28 Aug 2014, Maxim Ostapenko wrote: > diff --git a/gcc/diagnostic.c b/gcc/diagnostic.c > index 0cc7593..67b8c5b 100644 > --- a/gcc/diagnostic.c > +++ b/gcc/diagnostic.c > @@ -492,7 +492,7 @@ diagnostic_action_after_output (diagnostic_context > *context, > real_abort (); >dia

Re: [PATCH] rs6000: Use xori for HTM builtins and vector compares

2014-09-09 Thread David Edelsohn
On Tue, Sep 9, 2014 at 3:29 PM, Segher Boessenkool wrote: > These patterns use subfic now. subfic clobbers the carry. Other code > already preferably uses xor. Let's do the same here. > > Bootstrapped and tested as usual, no regressions. Is this okay to apply? > > > Segher > > > 2014-09-09 Se

Re: [PATCH][MIPS] Fix ICE in bitmap routines with LRA and inline assembly language

2014-09-09 Thread Vladimir Makarov
2014-09-09 Robert Suchanek gcc/ * lra-lives.c (process_bb_lives): Replace assignment with bitwise OR assignment. gcc/testsuite/ * gcc.target/mips/20140909.c: New test. --- gcc/lra-lives.c |6 +++--- gcc/testsuite/gcc.target/mips/2014090

Re: [PATCH] support ggc hash_map and hash_set

2014-09-09 Thread Trevor Saunders
On Tue, Sep 09, 2014 at 03:37:26PM +0100, Alan Lawrence wrote: > Following this, we're seeing ICEs in tests in gcc.dg/pch.exp and > g++.dg/pch.exp, > with cross-builds (hosted on x86_64) targetting bare metal AArch64 and ARM > (aarch64-none-elf, aarch64_be-none-elf and arm-none-eabi; I haven't tes

Re: PR debug/60655, debug loc expressions

2014-09-09 Thread Alan Modra
On Tue, Sep 09, 2014 at 04:42:04PM +0200, Richard Biener wrote: > On Tue, Sep 9, 2014 at 4:30 PM, Jakub Jelinek wrote: > > On Tue, Sep 09, 2014 at 04:25:23PM +0200, Richard Biener wrote: > >> why wasn't 'result' built using simplify_gen_* in the first place? I also > > > > It is built using cseli

Re: [Ping v2][PATCH] Add patch for debugging compiler ICEs.

2014-09-09 Thread Jakub Jelinek
On Tue, Sep 09, 2014 at 10:51:23PM +, Joseph S. Myers wrote: > On Thu, 28 Aug 2014, Maxim Ostapenko wrote: > > > diff --git a/gcc/diagnostic.c b/gcc/diagnostic.c > > index 0cc7593..67b8c5b 100644 > > --- a/gcc/diagnostic.c > > +++ b/gcc/diagnostic.c > > @@ -492,7 +492,7 @@ diagnostic_action_af

[PATCH, ARM] Tune thumb1_size_rtx_costs

2014-09-09 Thread Zhenqiang Chen
Hi, Currently most thumb1_size_rtx_costs are "guessed" from performance view, not size. The patch adjusts some of them according the instruction patterns defined in thumb1.md. It also replaces several hard coded "4" to COSTS_N_INSNS (1). No make check regression for Cortex-M0 on qemu. For Cortex

Re: [PATH] Intel offload library

2014-09-09 Thread Andrey Turetskiy
Ping. On Thu, Sep 4, 2014 at 11:41 AM, Andrey Turetskiy wrote: > Ping. > > On Tue, Aug 26, 2014 at 10:41 AM, Andrey Turetskiy > wrote: >> Ping. >> >> On Tue, Aug 19, 2014 at 12:45 PM, Kirill Yukhin >> wrote: >>> Hello, >>> On 12 Aug 10:58, Andrey Turetskiy wrote: All remarks from htt

PR 63166 (wrong devirtualization in ipa-prop)

2014-09-09 Thread Jan Hubicka
Hi, this patch fixes wrong type propagation across global variable. Bootstrapped/regtested x86_64-linux, will commit it shortly. PR ipa/63166 * ipa-prop.c (compute_known_type_jump_func): Fix conditional. * g++.dg/lto/pr63166_0.ii: New testcase. * g++.dg/lto/pr63166

<    1   2