Re: RFA: fix dbr_schedule to leave unique ids unique

2012-10-20 Thread Richard Sandiford
Joern Rennecke writes: > Quoting Richard Sandiford : >> Joern Rennecke writes: >>> Quoting Richard Sandiford : The fact that we even have shared unique ids is pretty bad -- and surely a contradiction in terms -- but I think both ways of handling them rely on the length being the sa

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

2012-10-20 Thread Richard Sandiford
Joern Rennecke 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 = true; >while (somethin

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. Bootstrappe

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

2012-10-20 Thread Joern Rennecke
Quoting Richard Sandiford : 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) PUT_MODE (body, CASE_VECTOR_SHO

[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 hig

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

2012-10-20 Thread Richard Sandiford
Joern Rennecke writes: > Quoting Richard Sandiford : >> 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) >>

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,

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 r

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 >

[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 * lra.c (check_rtx): Don't check UNSPEC

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. Bootstrapped/re

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 > 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-

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 Botcaz

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 ad

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 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 lower-numbered regis

[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 * news.html: Replace reference

[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 PR rtl-optimization/54991 * lra-constraints.c (lra_constraints): Change equiv memory check on reverse equivalence check. (inherit_in_ebb): Invalidate usage insns for multi-word hard regs.

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 p

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:D

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 non-host_in

[PATCH][xtensa] Remove unused variable

2012-10-20 Thread Chung-Lin Tang
Hi Sterling, the last thread pointer builtin changes left an unused 'arg' variable in xtensa_expand_builtin(), which triggered a new warning. Thanks to Jan-Benedict for testing this. Attached patch was committed as obvious. Thanks, Chung-Lin * config/xtensa/xtensa.c (xtensa_expand_builtin