Re: [PATCH, ARM] Fix PR44557 (Thumb-1 ICE)

2012-10-20 Thread Ramana Radhakrishnan
On Tue, Oct 16, 2012 at 10:25 AM, Chung-Lin Tang clt...@codesourcery.com wrote: On 12/9/27 6:25 AM, Janis Johnson wrote: On 09/26/2012 01:58 AM, Chung-Lin Tang wrote: +/* { dg-do compile } */ +/* { dg-options -mthumb -O1 -march=armv5te -fno-omit-frame-pointer -fno-forward-propagate } */

Re: RFA: fix dbr_schedule to leave unique ids unique

2012-10-20 Thread Richard Sandiford
Joern Rennecke joern.renne...@embecosm.com writes: Quoting Richard Sandiford rdsandif...@googlemail.com: Joern Rennecke joern.renne...@embecosm.com writes: Quoting Richard Sandiford rdsandif...@googlemail.com: The fact that we even have shared unique ids is pretty bad -- and surely a

Re: Ping: RFA: add lock_length attribute to break branch-shortening cycles

2012-10-20 Thread Richard Sandiford
Joern Rennecke joern.renne...@embecosm.com writes: @@ -1165,6 +1175,7 @@ shorten_branches (rtx first ATTRIBUTE_UN get the current insn length. If it has changed, reflect the change. When nothing changes for a full pass, we are done. */ + bool first_pass ATTRIBUTE_UNUSED =

Fix C FE __builtin_unreachable definition

2012-10-20 Thread Jan Hubicka
Hi, this patch fixes BUILT_IN_UNREACHABLE declaration of C frontned. The function is also pure (so DSE can do its job). As a special case ECF flags for CONST NORETURN also add looping, so this declaration is correct. The implicit declaration of the builtin is already set this way.

Re: Ping: RFA: add lock_length attribute to break branch-shortening cycles

2012-10-20 Thread Joern Rennecke
Quoting Richard Sandiford rdsandif...@googlemail.com: I think instead the set-up loop should have: if (GET_CODE (body) == ADDR_VEC || GET_CODE (body) == ADDR_DIFF_VEC) { #ifdef CASE_VECTOR_SHORTEN_MODE if (increasing GET_CODE (body) == ADDR_DIFF_VEC)

[PATCH, ARM] Subregs of VFP registers in big-endian mode

2012-10-20 Thread Julian Brown
Hi, Quite a few tests fail for big-endian multilibs which use VFP instructions at present. One reason for many of these is glaringly obvious once you notice it: for D registers interpreted as two S registers, the lower-numbered register is always the less-significant part of the value, and the

Re: Ping: RFA: add lock_length attribute to break branch-shortening cycles

2012-10-20 Thread Richard Sandiford
Joern Rennecke joern.renne...@embecosm.com writes: Quoting Richard Sandiford rdsandif...@googlemail.com: I think instead the set-up loop should have: if (GET_CODE (body) == ADDR_VEC || GET_CODE (body) == ADDR_DIFF_VEC) { #ifdef CASE_VECTOR_SHORTEN_MODE if (increasing

Re: Tidy store_bit_field_1 co.

2012-10-20 Thread Eric Botcazou
* expmed.c (lowpart_bit_field_p): New function. (store_bit_field_1): Remove unit, offset, bitpos and byte_offset from the outermost scope. Express conditions in terms of bitnum rather than offset, bitpos and byte_offset. Split the plain move cases into two, one

loop-unroll.c TLC 1/4

2012-10-20 Thread Jan Hubicka
Hi, the TLC path I sent last week became outdated for few reaons. I decided to split it up for easier reviewing. This is simple correcntess issue I am comitting as obvoius - my last update to loop-iv missed the fact that loop-iv bounds may depend on further conditions. In that case we can not

Re: Fix array bound niter estimate (PR middle-end/54937)

2012-10-20 Thread Jan Hubicka
What about the conservative variant of simply else delta = double_int_one; I think it would be bad idea: it makes us to completely unroll one interation too many that bloats code for no benefit. No optimization cancels the path in CFG because of

[lra] patch to fix testsuite regressions

2012-10-20 Thread Vladimir Makarov
After recent patches there were too many regressions of LRA on GCC testsuite on x86 and x86-64. The following patch fixes all of them. It was successfully bootstrapped on x86/x86-64. Committed as rev. 192637. 2012-10-20 Vladimir Makarov vmaka...@redhat.com * lra.c (check_rtx):

loop-unroll.c TLC 2/4

2012-10-20 Thread Jan Hubicka
Hi, this patch fixes heuristic on decide_unroll_constant_iterations to take into account the profile: even when the loop is known to have constant loop iteration bound, it doesn't need to really iterate many times. So use profile and loop_max to double check that this is the case.

Re: [PATCH, ARM] Fix PR44557 (Thumb-1 ICE)

2012-10-20 Thread Janis Johnson
On 10/19/2012 11:41 PM, Ramana Radhakrishnan wrote: On Tue, Oct 16, 2012 at 10:25 AM, Chung-Lin Tang clt...@codesourcery.com wrote: On 12/9/27 6:25 AM, Janis Johnson wrote: On 09/26/2012 01:58 AM, Chung-Lin Tang wrote: +/* { dg-do compile } */ +/* { dg-options -mthumb -O1 -march=armv5te

Re: [RFC] Fix PR rtl-optimization/54315 (partially)

2012-10-20 Thread Eric Botcazou
The patch was fully tested on x86_64-suse-linux, where it removes half of the useless stores in the original testcase for PR rtl-optimization/54315, and manually tested for arm-linux-gnueabi (for now), where it also removes stores for small structures. Comments? 2012-10-08 Eric Botcazou

Re: [Committed] S/390: Add support for the new IBM zEnterprise EC12

2012-10-20 Thread Gerald Pfeifer
On Wed, 10 Oct 2012, Andreas Krebbel wrote: the attached patch adds initial support for the latest release of the IBM mainframe series - the IBM zEnterprise EC12 (zEC12). Nice. Can you please also add a note to the release notes at gcc-4.8/changes.html ? In principle, I'm also in favor of

Re: [PATCH, ARM] Subregs of VFP registers in big-endian mode

2012-10-20 Thread Andrew Pinski
On Sat, Oct 20, 2012 at 4:38 AM, Julian Brown jul...@codesourcery.com wrote: Hi, Quite a few tests fail for big-endian multilibs which use VFP instructions at present. One reason for many of these is glaringly obvious once you notice it: for D registers interpreted as two S registers, the

[wwwdocs,Java] Replace sources.redhat.com by sourceware.org

2012-10-20 Thread Gerald Pfeifer
...and some other simplifications and improvements I noticed on the way. This was triggered by a note that the sources.redhat.com DNS entry is going to go away at some point in the future that I got yesterday. Applied. Gerald 2012-10-21 Gerald Pfeifer ger...@pfeifer.com *

[lra] patch to fix PR54991

2012-10-20 Thread Vladimir Makarov
The following patch fixes PR54991. Committed as rev. 192645. 2012-10-20 Vladimir Makarov vmaka...@redhat.com PR rtl-optimization/54991 * lra-constraints.c (lra_constraints): Change equiv memory check on reverse equivalence check. (inherit_in_ebb): Invalidate usage insns for

Committed, libgcc MMIX: implement static marking of program and data memory

2012-10-20 Thread Hans-Peter Nilsson
With a simulator that doesn't just allocate zeros on any access, it's necessary to tell the simulator the bounds of defined memory, both for static and dynamically allocated memory. This patch implements static code and data allocatation; zero'd data and constants may not be otherwise loaded. A

Ping: [RFA:] Fix frame-pointer-clobbering in builtins.c:expand_builtin_setjmp_receiver

2012-10-20 Thread Hans-Peter Nilsson
CC:ing middle-end maintainers this time. I was a bit surprised when Eric Botcazou wrote in his review, quoted below, that he's not one of you. Maybe approve that too? On Mon, 15 Oct 2012, Hans-Peter Nilsson wrote: On Fri, 12 Oct 2012, Eric Botcazou wrote: (insn 168 49 51 3 (set (reg/f:DI

Committed: skip testsuite/23_containers/bitset/45713.cc for mmix-*-*.

2012-10-20 Thread Hans-Peter Nilsson
For mmix-knuth-mmixware, MAX_FIXED_MODE_SIZE is the default, GET_MODE_BITSIZE (DImode), which of course isn't larger than the size-type, the same size on this 64-bit target. I don't think making it larger (i.e. TImode) would help: that seems instead likely to introduce awkward spurious