[PATCH] Improve Niagara-3 tuning.

2011-09-17 Thread David Miller
This adjusts the costs of various operations on Niagara-3, in particular FP operation latency increases to 9 cycles compared to Niagara-2's 6 cycles. Integer multiplies increase to 9 cycles as well, which likely indicates that they go through the FPU unit too. I also fixed up the integer divide

[v3] libstdc++/50441

2011-09-17 Thread Paolo Carlini
Hi, tested x86_64-linux, committed to mainline. Paolo. /// 2011-09-17 Paolo Carlini PR libstdc++/50441 * acinclude.m4 ([GLIBCXX_ENABLE_INT128_FLOAT128]): Add. * configure.ac: Call it. * include/std/type_traits (__is_integral_helper<__int128_t>

Re: C++ PATCHes for DR 1328, c++/49267, c++/49458

2011-09-17 Thread Jason Merrill
On 09/05/2011 12:29 AM, Jason Merrill wrote: Also applying the compare_ics hunk to 4.6. This caused c++/50442, so I'm reverting it. Jason

Re: [wwwdocs] Buildstat update for 4.6

2011-09-17 Thread Gerald Pfeifer
On Thu, 15 Sep 2011, Tom G. Christensen wrote: > Latest results for 4.6.x Applied, thanks Tom! Gerald

[wwwdocs] Minor edit on extensions.html

2011-09-17 Thread Gerald Pfeifer
...shortening the text a bit as well. Gerald Index: extensions.html === RCS file: /cvs/gcc/wwwdocs/htdocs/extensions.html,v retrieving revision 1.48 diff -u -r1.48 extensions.html --- extensions.html 10 Sep 2011 21:47:33 -

Re: Doc updates for OpenMP 3.1 support, -grecord-gcc-switches, __builtin_assume_aligned and debug info improvements

2011-09-17 Thread Gerald Pfeifer
On Tue, 2 Aug 2011, Jakub Jelinek wrote:a > --- projects/gomp/index.html 30 Mar 2009 00:07:58 - 1.10 > +++ projects/gomp/index.html 2 Aug 2011 17:01:52 - Two minor editorial changes that I just committed. Gerald Index: projects/gomp/index.html =

Re: [PATCH 6/7] Kill pedantic warnings on system headers macros

2011-09-17 Thread Jason Merrill
On 09/16/2011 04:46 AM, Dodji Seketeli wrote: struct c_declspecs * -finish_declspecs (struct c_declspecs *specs) +finish_declspecs (struct c_declspecs *specs, + location_t where) Let's call this first_token_loc, too. And mention it in the function comment. OK with that chan

[pph] Fix pph_read_tree_header. (issue5050045)

2011-09-17 Thread Lawrence Crowl
Restructure pph_read_tree_header and pph_read_tree to do what their names say they do. In particular, generic tree handling from pph_read_tree_header to pph_read_tree. Add debugging routines pph_tree_code_text (to avoid out of bounds indexing on a bad tree code) and pph_dump_min_decl (to dump the

Re: PATCH: Replace tmp with __tmp

2011-09-17 Thread H.J. Lu
On Sat, Sep 17, 2011 at 3:10 AM, Andreas Schwab wrote: > Uros Bizjak writes: > >> Agreed. Some parets are missing, though: >> >> -  unsigned long long tmp = (__X) ^ (__X - 1); >> -  return tmp; >> +  unsigned long long __tmp = (__X) ^ (__X - 1); >> +  return __tmp; > > There is none missing.  Thi

Re: [PATCH 7/7] Reduce memory waste due to non-power-of-2 allocs

2011-09-17 Thread Jason Merrill
On 09/17/2011 07:08 AM, Dodji Seketeli wrote: OK, so the patch below extracts a public ggc_alloced_size_for_request function from the different implementations of the ggc allocator's interface, and lets new_linemap use that. Maybe "ggc_round_alloc_size"? OK with that change if nobody else has

Re: [PATCH 5/7] Add line map statistics to -fmem-report output

2011-09-17 Thread Jason Merrill
On 09/16/2011 04:13 AM, Dodji Seketeli wrote: It seems odd to add in the locations size here since it's also printed separately. I wanted macro_maps_used_size to really reflect the total used size for macro maps, without having to mentally do the addition of its two components. But at the same

Re: [PATCH 4/7] Support -fdebug-cpp option

2011-09-17 Thread Jason Merrill
OK. Jason

Re: [PATCH 3/7] Emit macro expansion related diagnostics

2011-09-17 Thread Jason Merrill
On 09/16/2011 03:55 AM, Dodji Seketeli wrote: +test.c: In function ‘g’: +test.c:5:14: error: invalid operands to binary << (have ‘double’ and ‘int’) +test.c:2:9: note: in expansion of macro 'OPERATE' +test.c:5:3: note: expanded from here +test.c:5:14: note: in expansion of mac

Re: [Patch] PR c++/26256

2011-09-17 Thread Fabien Chêne
Hi! (Again, sorry for the long delay, but I indeed have very few time to work on it) 2011/6/22 Jason Merrill : > On 06/15/2011 01:58 PM, Fabien Chęne wrote: >> >> Otherwise, perhaps that it would be better if the second declaration >> of E1 does not rely on supplement_binding_1. >> What do you th

Re: hash policy patch

2011-09-17 Thread Robert Dewar
On 9/17/2011 5:38 AM, Paolo Carlini wrote: On 09/17/2011 11:27 AM, François Dumont wrote: Paolo, I know that using float equality comparison is not reliable in general and I have remove the suspicious line but in this case I can't imagine a system where it could fail. As a general policy, in th

[patch] Don't try to reload match_operator

2011-09-17 Thread Richard Earnshaw
This patch fixes a couple of cases where reload incorrectly thinks it should be messing around with the result of a match_operator operand. Given that it's never even tried to validate the contents of such an operand the changes it tries to make are bogus. This is needed for an upcoming patch I ha

Re: PowerPC shrink-wrap support 0 of 3

2011-09-17 Thread Bernd Schmidt
On 09/17/11 09:16, Alan Modra wrote: > This patch series adds shrink-wrap support for PowerPC. The patches > are on top of Bernd's "Initial shrink-wrapping patch": > http://gcc.gnu.org/ml/gcc-patches/2011-08/msg02557.html, but with the > tm.texi patch applied to tm.texi.in. Bootstrapped and regre

Re: [PATCH] non-GNU C++ compilers

2011-09-17 Thread Joseph S. Myers
On Sat, 17 Sep 2011, Marc Glisse wrote: > Is someone willing to review (and possibly commit) these 2 patches? > > Changelog for include/obstack.h in glibc (guess it should be copied almost as > is?): > > 2011-09-11 Ulrich Drepper > * obstack.h [!GNUC] (obstack_free): Avoid cast to int.

Re: [PATCH 7/7] Reduce memory waste due to non-power-of-2 allocs

2011-09-17 Thread Dodji Seketeli
Jason Merrill writes: > On 07/16/2011 10:37 AM, Dodji Seketeli wrote: > > Ideally, I'd prefer some parts of this patch to be integrated into the > > memory allocator. That is, I'd like to see the memory allocator have > > an interface that returns the actual size of memory it has allocated. > >

Re: Fortran Patches

2011-09-17 Thread Janus Weil
>> Regarding the last patch, the GNU style puts a line break after the ")" in: >> >> +  if (!sym) return NULL; >> + > > In principle I'm aware of the GNU coding style, but apparently I > didn't pay enough attention. Sorry again. I'll fix it ... Fixed with r178928. Cheers, Janus

Re: PATCH: Replace tmp with __tmp

2011-09-17 Thread Andreas Schwab
Uros Bizjak writes: > Agreed. Some parets are missing, though: > > - unsigned long long tmp = (__X) ^ (__X - 1); > - return tmp; > + unsigned long long __tmp = (__X) ^ (__X - 1); > + return __tmp; There is none missing. This is not a macro. Andreas. -- Andreas Schwab, sch...@linux-m68k.o

Re: PATCH: Replace tmp with __tmp

2011-09-17 Thread Uros Bizjak
On Sat, Sep 17, 2011 at 8:31 AM, Andreas Schwab wrote: >> diff --git a/gcc/config/i386/bmiintrin.h b/gcc/config/i386/bmiintrin.h >> index af5d9dc..72ab114 100644 >> --- a/gcc/config/i386/bmiintrin.h >> +++ b/gcc/config/i386/bmiintrin.h >> @@ -42,8 +42,8 @@ __tzcnt_u16 (unsigned short __X) >>  ext

Re: hash policy patch

2011-09-17 Thread Paolo Carlini
On 09/17/2011 11:27 AM, François Dumont wrote: Paolo, I know that using float equality comparison is not reliable in general and I have remove the suspicious line but in this case I can't imagine a system where it could fail. As a general policy, in the testsuite we should never assert equality

Re: hash policy patch

2011-09-17 Thread François Dumont
On 09/16/2011 03:00 AM, Paolo Carlini wrote: Ok... but: + us.max_load_factor(.5f); + VERIFY( us.max_load_factor() == .5f ); as we discussed already (didn't we?), this kind of VERIFY is in general very brittle (even if on the widespread base-2 systems probably we are lucky in this *s

Re: [RFC PATCH] AVX2 32-byte integer min/max reductions

2011-09-17 Thread Uros Bizjak
On Fri, Sep 16, 2011 at 8:52 PM, Jakub Jelinek wrote: >> So, either we can fix this by adding >> reduc_{smin,smax,umin,umax}_v{32q,16h,8s,4d}i >> patterns (at that point I guess I should just macroize them together with >> the reduc_{smin,smax,umin,umax}_v{4sf,8sf,4df}) and handle the 4 32-byte

Re: [RFC PATCH] AVX2 32-byte integer {s,u}m{in,ax} and vcond{,u} patterns

2011-09-17 Thread Uros Bizjak
On Fri, Sep 16, 2011 at 6:20 PM, Jakub Jelinek wrote: >> Surprisingly with -mavx2 the integer loops aren't vectorized with >> 32-byte vectors, wonder why.  But looking at the integer umin/umax/smin/smax >> 16-byte reductions they generate good code even without reduc_* patterns, >> apparently usi

Re: [rs6000] Fix PR target/50091

2011-09-17 Thread Eric Botcazou
> Tested on PowerPC/Darwin by Iain and on PowerPC/Linux by me. OK for > mainline and the 4.6/4.5 branches? > > > 2011-09-06 Eric Botcazou > Iain Sandoe > > PR target/50091 > * config/rs6000/rs6000.md (probe_stack): Use explicit operand. > * config/rs6000/rs6000.c (

Re: [PATCH] non-GNU C++ compilers

2011-09-17 Thread Marc Glisse
(the original discussion started here: http://gcc.gnu.org/ml/gcc-patches/2011-08/msg00758.html ) On Mon, 8 Aug 2011, Joseph S. Myers wrote: On Mon, 8 Aug 2011, Marc Glisse wrote: * include/obstack.h (obstack_free): Cast to char* instead of int This header comes from glibc/gnulib. A

PowerPC shrink-wrap support 3 of 3

2011-09-17 Thread Alan Modra
Finally, the powerpc backend changes. These are mostly just mechanical. I'll note that we need both "simple_return" and "return" variants of the conditional returns because they can only be used when no epilogue is required. The "return" variant must use direct_return() as a predicate to check t

PowerPC shrink-wrap support 2 of 3

2011-09-17 Thread Alan Modra
This patch corrects some cases where a return jump jump_label wrongly uses ret_rtx with a simple_return, thus triggering the assert in mark_jump_label_1. PowerPC doesn't need this patch with the final version of my shrink wrap support, but at one stage I was using (define_expand "return" [(simp

PowerPC shrink-wrap support 1 of 3

2011-09-17 Thread Alan Modra
This obvious patch extends my 2011-08-03 fix to simple_return. Necessary for the same reason as the original patch, namely that rs6000.md has a peephole2 that matches and recreates a conditional jump. With the third patch in this series applied, rs6000 will start to use simple_return in conditiona

PowerPC shrink-wrap support 0 of 3

2011-09-17 Thread Alan Modra
This patch series adds shrink-wrap support for PowerPC. The patches are on top of Bernd's "Initial shrink-wrapping patch": http://gcc.gnu.org/ml/gcc-patches/2011-08/msg02557.html, but with the tm.texi patch applied to tm.texi.in. Bootstrapped and regression tested powerpc64-linux all langs except