RE: [PATCH]Fix computation of offset in ivopt

2013-09-29 Thread bin.cheng
> -Original Message- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- > ow...@gcc.gnu.org] On Behalf Of Oleg Endo > Sent: Wednesday, September 25, 2013 1:41 AM > To: Richard Biener > Cc: Bin Cheng; GCC Patches > Subject: Re: [PATCH]Fix computation of offset in ivopt > > > > >

RE: [PATCH]Fix computation of offset in ivopt

2013-09-29 Thread bin.cheng
> -Original Message- > From: Richard Biener [mailto:richard.guent...@gmail.com] > Sent: Friday, September 27, 2013 4:30 PM > To: Bin Cheng > Cc: GCC Patches > Subject: Re: [PATCH]Fix computation of offset in ivopt > > On Fri, Sep 27, 2013 at 7:07 AM, bin.cheng wrote: > > > > > > case

Re: [PATCH] Invalid unpoisoning of stack redzones on ARM

2013-09-29 Thread Yury Gribov
> Can you please be more verbose Right, I should have been. So as you can see from the asm log in the bug description, prologue writes shadow bytes corresponding to words at frame_shadow_base + { 0, 4, 8, 12, 16, 24, 28}. Epilogue should clear those but instead it zeros out frame_shadow_base

[PATCH, doc]: Fix "@tex should only appear at a line beginning'" warnings

2013-09-29 Thread Uros Bizjak
Hello! Attached patch decorates mail addresses with @mail{} to avoid "@tex should only appear at a line beginning'" warnings. Also the patch adds @uref and http:// prefix for a webpage. 2013-09-29 Uros Bizjak * doc/gcc.texi (titlepage): Use @uref and http:// prefix for website. Use @e

Re: Remove algo logic duplication Round 3

2013-09-29 Thread Paolo Carlini
Hi, "François Dumont" ha scritto: >I also get your remark about the open round bracket, I didn't know that > >round bracket was the other name for parentheses ! I also fix the one >you pointed me, I will be more careful next time. No problem ;) For your linguistic curiosity, I often find myse

[PATCH, doc]: Fix usage of @tie{} command

2013-09-29 Thread Uros Bizjak
Hello! 2013-09-29 Uros Bizjak * doc/invoke.texi: Fix usage of @tie{} command. Tested with texinfo-5.1, committed to mainline SVN. Uros. Index: invoke.texi === --- invoke.texi (revision 203011) +++ invoke.texi (working c

Re: RFA [testsuite]: New ARC target specific tests

2013-09-29 Thread Mike Stump
On Sep 28, 2013, at 5:40 AM, Joern Rennecke wrote: > This patch adds a number of tests for ARC target specific options. > > I'm a bit uncertain here if I still need approval for this patch. No, it is not required. Target maintainers can approve the usual tests suite patches. Everything in gcc

[SPARC] Add peephole for memory barriers

2013-09-29 Thread Eric Botcazou
This is a bit gross, but this prevents consecutive memory barriers from being generated, for example on accesses to atomic objects in Ada. Tested on SPARC/Solaris, applied on the mainline. 2013-09-29 Eric Botcazou * config/sparc/sync.md: Add peephole for consecutive memory barriers.

Re: [Patch, Darwin/PPC] fix PR10901

2013-09-29 Thread Mike Stump
On Sep 28, 2013, at 3:26 AM, Iain Sandoe wrote: > We currently generate wrong code for non-local gotos which breaks, amongst > other things, nested functions. > I fixed this a while ago for x86 Darwin and here is a version to fix it on > PPC. > > OK for trunk? (and open branches?) Ok.

Re: [Patch, Darwin/ppc] Fix altivec dwarf reg sizes.

2013-09-29 Thread Mike Stump
On Sep 28, 2013, at 3:03 AM, Iain Sandoe wrote: > Anyway, after considerable debate about this and several approaches, here is > a patch that just ensures we set the altivec register size to its correct > value. > OK for trunk and open branches? Ok.

Re: [Patch, Darwin] improve cross-compiles.

2013-09-29 Thread Mike Stump
On Sep 28, 2013, at 2:35 AM, Iain Sandoe wrote: > I've been experimenting with the idea of building native crosses on my most > capable machine, for the many variants of darwin we now have, and then using > the older/slower hardware for test only. > > This has uncovered a few issues with cross/

Re: [Patch, Darwin] update t-* and x-* fragments after switch to auto-deps.

2013-09-29 Thread Mike Stump
On Sep 28, 2013, at 2:21 AM, Iain Sandoe wrote: > This updates the Darwin port {t,x}-* fragments after the switch to auto-deps > (thanks Tom!). > OK? Ok.

[PATCH, doc]: Fix "@anchor should not appear in @heading" warning

2013-09-29 Thread Uros Bizjak
Hello! Rather trivial fix - put @anchor before @heading, as texi manual suggests. 2013-09-29 Uros Bizjak * doc/install.texi (Host/target specific installation notes for GCC): Put @anchor before @heading. Tested by "make doc" with texinfo 5.1 on Fedora 19. OK for mainline? Uros. Ind

Re: [PATCH] Sanitize block partitioning under -freorder-blocks-and-partition

2013-09-29 Thread Teresa Johnson
On Fri, Sep 27, 2013 at 7:15 AM, Teresa Johnson wrote: > On Thu, Sep 26, 2013 at 3:02 PM, Jan Hubicka wrote: >>> >>> Why not just have probably_never_executed_bb_p return simply return >>> false bb->frequency is non-zero (right now it does the opposite - >> >> We want to have frequencies guessed

Re: Add value range support into memcpy/memset expansion

2013-09-29 Thread Jan Hubicka
> Hi Jan, > > > I also think the misaligned move trick can/should be performed by > > move_by_pieces and we ought to consider sane use of SSE - current > > vector_loop > > with unrolling factor of 4 seems bit extreme. At least buldozer is happy > > with > > 2 and I would expect SSE moves to be

Document x86-tune options

2013-09-29 Thread Jan Hubicka
Hi, this patch adds documetnation to individual options and also adds vertical whitespace. Comitted as obvious, Honza * config/i386/x86-tune.def: Add documentation for each of the options; add whitespace. Index: config/i386/x86-tune.def

Re: [PATCH 2/6] Andes nds32: machine description of nds32 porting (1).

2013-09-29 Thread Richard Sandiford
Chung-Ju Wu writes: >>> +/* Permitting tail calls. */ >>> + >>> +static bool >>> +nds32_warn_func_return (tree decl) >>> +{ >>> + /* Naked functions are implemented entirely in assembly, including the >>> + return sequence, so suppress warnings about this. */ >>> + return !nds32_naked_func

Re: Add value range support into memcpy/memset expansion

2013-09-29 Thread Michael V. Zolotukhin
Hi Jan, > I also think the misaligned move trick can/should be performed by > move_by_pieces and we ought to consider sane use of SSE - current vector_loop > with unrolling factor of 4 seems bit extreme. At least buldozer is happy with > 2 and I would expect SSE moves to be especially useful for

Re: Add value range support into memcpy/memset expansion

2013-09-29 Thread Jan Hubicka
> > We now produce: > > movqb(%rip), %rsi > > movqa(%rip), %rcx > > movq(%rsi), %rax <- first 8 bytes are moved > > leaq8(%rcx), %rdi > > andq$-8, %rdi <- dest is aligned > > movq%rax, (%rcx) > > movq132(%rsi), %r

RFA: Use "m_foo" rather than "foo_" for member variables

2013-09-29 Thread Richard Sandiford
Michael Matz writes: >Trever Saunders writes: >> Richard Biener writes: >> > Btw, I've come around multiple coding-styles in the past and I >> > definitely would prefer m_mode / m_count to mark members vs. mode_ and >> > count_. (and s_XXX for static members IIRC). >> >> I'd prefer m_/s_foo fo