Re: [PATCH][ARM] one_cmpldi2 in NEON

2011-12-07 Thread Andrew Stubbs
On 06/12/11 23:07, Richard Henderson wrote: On 12/06/2011 01:42 PM, Andrew Stubbs wrote: On Tue 06 Dec 2011 21:05:30 GMT, Richard Henderson wrote: On 12/06/2011 09:59 AM, Andrew Stubbs wrote: +(define_insn *one_cmpldi2_neon + [(set (match_operand:DI 0 s_register_operand =w,?r,?r,?w) +

Re: [PATCH] _GCC_PICFLAG: use -fPIC for s390x targets

2011-12-07 Thread Rainer Orth
Mike Frysinger vap...@gentoo.org writes: Building newer libiberty for s390x targets fails with relocation errors: libiberty/pic/libiberty.a(hashtab.o): In function 'htab_create': libiberty/hashtab.c:408:(.text+0x5e4): relocation truncated to fit: R_390_GOT12 against

Re: [Patch] Increase array sizes in vect-tests to enable 256-bit vectorization

2011-12-07 Thread Michael Zolotukhin
Thanks, Richard. Should somebody else approve the patch or is it ok for commit to trunk? On 5 December 2011 18:04, Richard Guenther richard.guent...@gmail.com wrote: On Mon, Dec 5, 2011 at 2:28 PM, Michael Zolotukhin michael.v.zolotuk...@gmail.com wrote: I'd just duplicate the tests you want

Re: [PATCH] Fix PR50823 and its gazillion dups

2011-12-07 Thread Jan Hubicka
Am Tue 06 Dec 2011 03:41:36 PM CET schrieb Richard Guenther rguent...@suse.de: This removes accounting for the number of remaining calls in the inlining edge badness calculation (as discussed in private with Honza a long time ago - and yes, we disagreed). This Well, the main reason for

Re: Ping #1: [Patch,AVR] Light-weight DImode implementation.

2011-12-07 Thread Georg-Johann Lay
Georg-Johann Lay wrote: Denis Chertykov wrote: The only question that remains is what the -m64 option should be like? [ ] Omit it altogether [ ] Leave it as is (off per default) [ ] Set it on per default As soon as the direction is clear, I'll post a follow-up patch to add the missing

[Dodji Seketeli] Re: [PATCH] PR c++/51289 - ICE with alias template for bound template

2011-12-07 Thread Dodji Seketeli
Friendly pinging this patch. ---BeginMessage--- Dodji Seketeli do...@redhat.com writes: Jason Merrill ja...@redhat.com writes: I guess let's check DECL_ORIGINAL_TYPE instead of TREE_TYPE for alias templates. Like the below that I am currently bootstrapping? Finally this is what passed

Re: [PATCH] Fix PR50823 and its gazillion dups

2011-12-07 Thread Richard Guenther
On Wed, 7 Dec 2011, Jan Hubicka wrote: Am Tue 06 Dec 2011 03:41:36 PM CET schrieb Richard Guenther rguent...@suse.de: This removes accounting for the number of remaining calls in the inlining edge badness calculation (as discussed in private with Honza a long time ago - and yes, we

[testsuite]: Move pr45830.c to gcc.dg/torture and make it pass for AVR

2011-12-07 Thread Georg-Johann Lay
In principle, test case pr45830.c works for target avr, but there is an issue with the -ftree-switch-conversion optimization activated at higher optimization levels: It transforms code size into .data usage and thus exceeds AVRs' RAM size because of big CSWTCH lookup tables located in RAM. The

Re: [Patch] Increase array sizes in vect-tests to enable 256-bit vectorization

2011-12-07 Thread Richard Guenther
On Wed, Dec 7, 2011 at 11:27 AM, Michael Zolotukhin michael.v.zolotuk...@gmail.com wrote: Thanks, Richard. Should somebody else approve the patch or is it ok for commit to trunk? It's ok to commit. Richard. On 5 December 2011 18:04, Richard Guenther richard.guent...@gmail.com wrote: On Mon,

Re: [google] Backport r171347 and r181549 from trunk (strict volatile bitfield) (issue5434084)

2011-12-07 Thread Jakub Jelinek
On Tue, Nov 29, 2011 at 05:59:53PM -0800, Doug Kwan wrote: This is a backport for two upstream patches into our 4.6 branch. I submitted the first patch by Julian a while ago for backport but Richard Earnshaw pointed out a problem with the first patch. The second patch from Joey fixes that

Re: [PATCH] Fix PR tree-optimization/51315

2011-12-07 Thread Eric Botcazou
You are basically (but non-optimally) locally re-implementing what expr.c:get_object_or_type_alignment does, for the bare MEM_REF case (you don't consider offsets that do not change the alignment, nor alignment information present on the SSA name). Gosh. And now I distinctly remember

Re: [google] Backport r171347 and r181549 from trunk (strict volatile bitfield) (issue5434084)

2011-12-07 Thread Richard Guenther
On Wed, Dec 7, 2011 at 1:34 PM, Jakub Jelinek ja...@redhat.com wrote: On Tue, Nov 29, 2011 at 05:59:53PM -0800, Doug Kwan wrote:     This is a backport for two upstream patches into our 4.6 branch. I submitted the first patch by Julian a while ago for backport but Richard Earnshaw pointed out

Re: [PATCH] Remove dead labels to increase superblock scope

2011-12-07 Thread Tom de Vries
On 06/12/11 14:25, Michael Matz wrote: Hi, On Tue, 6 Dec 2011, Tom de Vries wrote: what should be the 'next' returned by delete_insn? There are exactly two calls of delete_insn that take the return value. One (delete_insn_and_edges) just uses it to return it itself (and there are no

Re: [coverage] Some restructuring

2011-12-07 Thread Markus Trippelsdorf
On 2011.12.04 at 18:35 +, Nathan Sidwell wrote: I've committed this patch to break apart the gcov finalization routines, I believe this will make it easier to fix the problem shown up by bug 51113 -- although this patch does not. Notable changes: * rename coverage_begin_output to

Re: [google] Backport r171347 and r181549 from trunk (strict volatile bitfield) (issue5434084)

2011-12-07 Thread Richard Earnshaw
On 07/12/11 13:02, Richard Guenther wrote: On Wed, Dec 7, 2011 at 1:34 PM, Jakub Jelinek ja...@redhat.com wrote: On Tue, Nov 29, 2011 at 05:59:53PM -0800, Doug Kwan wrote: This is a backport for two upstream patches into our 4.6 branch. I submitted the first patch by Julian a while ago for

[PATCH] Testcase for PR48100

2011-12-07 Thread Richard Guenther
Which got fixed on trunk. Tested on x86_64-unknown-linux-gnu, applied. Richard. 2011-12-07 Richard Guenther rguent...@suse.de PR lto/48100 * gcc.dg/lto/20111207-1_0.c: New testcase. * gcc.dg/lto/20111207-1_1.c: Likewise. * gcc.dg/lto/20111207-1_2.c: Likewise

Re: [Patch, Fortran] PR50815 - don't -fcheck=bounds of deferred-length strings

2011-12-07 Thread Tobias Burnus
** PING ** On 11/29/2011 07:35 PM, Tobias Burnus wrote: gfortran had an ICE when trying to insert a check whether the character length between actual and dummy argument matches. This check is pointless for deferred-length character arguments - thus, no bounds check should be generated.

Re: [Patch, Fortran] PR50923 - [4.4-4.7] Print warning function does not return a value

2011-12-07 Thread Tobias Burnus
** PING ** On 11/30/2011 07:19 PM, Tobias Burnus wrote: The bug has been introduced when changing the warning system to do more in the front end. The problem is that for: module m contains function f() end end the sym-attr.referenced gets set - and no warning is printed. I now

Re: [Patch, Fortran] PR51407 - allow BOZ edit descriptors for REAL/COMPLEX

2011-12-07 Thread Tobias Burnus
* ping * ? On 12/04/2011 06:46 PM, Tobias Burnus wrote: Hi all, as Dominique has found, Fortran 2008 allows the BOZ edit descriptors now also with REAL and COMPLEX arguments. (See PR for quotes from the standard.) Build and regtested on x86-64-linux. OK for the trunk? Tobias PS: Thank

Re: [google] Backport r171347 and r181549 from trunk (strict volatile bitfield) (issue5434084)

2011-12-07 Thread Richard Guenther
On Wed, Dec 7, 2011 at 2:32 PM, Richard Earnshaw rearn...@arm.com wrote: On 07/12/11 13:02, Richard Guenther wrote: On Wed, Dec 7, 2011 at 1:34 PM, Jakub Jelinek ja...@redhat.com wrote: On Tue, Nov 29, 2011 at 05:59:53PM -0800, Doug Kwan wrote:     This is a backport for two upstream patches

Re: RFC: ARM 64-bit shifts in NEON

2011-12-07 Thread Andrew Stubbs
On Wed 07 Dec 2011 13:42:37 GMT, Richard Earnshaw wrote: So it looks like the code generated for core registers with thumb2 is pretty rubbish (no real surprise there -- to get the best code you need to make use of the fact that on ARM a shift by a small negative number ( -128) will give zero.

RFA: Fix PR middle-end/40154

2011-12-07 Thread Joern Rennecke
Having a value in the wrong mode is not the only hazard that users of set_unique_reg_note should anticipate; it could also be that the value is computed as a constant by the compiler, and thus the last instruction obtained with get_last_insn is completely (or somewhat) unrelated; or an expander

Re: RFC: ARM 64-bit shifts in NEON

2011-12-07 Thread Richard Earnshaw
On 07/12/11 14:03, Andrew Stubbs wrote: On Wed 07 Dec 2011 13:42:37 GMT, Richard Earnshaw wrote: So it looks like the code generated for core registers with thumb2 is pretty rubbish (no real surprise there -- to get the best code you need to make use of the fact that on ARM a shift by a small

Re: [PATCH] Fix PR tree-optimization/51315

2011-12-07 Thread Eric Botcazou
You are basically (but non-optimally) locally re-implementing what expr.c:get_object_or_type_alignment does, for the bare MEM_REF case (you don't consider offsets that do not change the alignment, nor alignment information present on the SSA name). Adjusted version attached,

Re: [SMS] Support new loop pattern

2011-12-07 Thread Roman Zhuykov
2011/10/12 Ayal Zaks ayal.z...@gmail.com: - the last jump instruction should look like:  pc=(regF!=0)?label:pc, regF is you'd probably want to bump to next instruction if falling through, e.g., pc=(regF!=0)?label:pc+4 It is considered that program counter is increased automatically onhardware

Re: RFC: ARM 64-bit shifts in NEON

2011-12-07 Thread Andrew Stubbs
On Wed 07 Dec 2011 14:20:43 GMT, Richard Earnshaw wrote: Would it not require an unspec to prevent 'clever things' happening to the negative shift, if we were to encode these in the machine description? I'm not too clear on what these 'clever things' might be in the case of shift-by-register

Re: [SMS] Support new loop pattern

2011-12-07 Thread Roman Zhuykov
Apologies for the messed up previous e-mail. 2011/10/12 Ayal Zaks ayal.z...@gmail.com: - the last jump instruction should look like:  pc=(regF!=0)?label:pc, regF is you'd probably want to bump to next instruction if falling through, e.g., pc=(regF!=0)?label:pc+4 It is considered that

[SMS, DDG] Additional edges to instructions with clobber

2011-12-07 Thread Roman Zhuykov
This letter contains the first separate patch for ddg.cIt creates additional nessesary anti-dep edges in data dependency graph. 2011/10/12 Ayal Zaks ayal.z...@gmail.com: The following situation happens when SMS is enabled without register renaming (-fno-modulo-sched-allow-regmoves).  When data

[PATCH][LTO] Fix PR48437

2011-12-07 Thread Richard Guenther
(tree_is_indexable): Exclude block-local extern declarations. * gcc.dg/lto/20111207-2_0.c: New testcase. * gcc.dg/guality/pr48437.c: Likewise. Index: gcc/lto-streamer-out.c === *** gcc/lto-streamer-out.c (revision 182081

Re: [PATCH] Fix PR tree-optimization/51315

2011-12-07 Thread Richard Guenther
On Wed, Dec 7, 2011 at 3:32 PM, Eric Botcazou ebotca...@adacore.com wrote: You are basically (but non-optimally) locally re-implementing what expr.c:get_object_or_type_alignment does, for the bare MEM_REF case (you don't consider offsets that do not change the alignment, nor alignment

[SMS, DDG] Additional edges to instructions with clobber

2011-12-07 Thread Roman Zhuykov
Apologies for the messed up previous e-mail. This letter contains the first separate patch for ddg.cIt creates additional nessesary anti-dep edges in data dependency graph. 2011/10/12 Ayal Zaks ayal.z...@gmail.com: The following situation happens when SMS is enabled without register renaming

Re: [C++ PATCH] Fix ICE in build_value_init (PR c++/51369)

2011-12-07 Thread Jason Merrill
On 12/06/2011 02:48 PM, Jakub Jelinek wrote: - gcc_assert (!processing_template_decl || SCALAR_TYPE_P (type)); + gcc_assert (!processing_template_decl + || (SCALAR_TYPE_P (type) || TREE_CODE (type) == ARRAY_TYPE)); How about SCALAR_TYPE_P (strip_array_types (type))? OK with that

Re: [PATCH][LTO] Fix PR48437

2011-12-07 Thread Jakub Jelinek
On Wed, Dec 07, 2011 at 03:52:35PM +0100, Richard Guenther wrote: *** gcc/testsuite/gcc.dg/guality/pr48437.c(revision 0) --- gcc/testsuite/gcc.dg/guality/pr48437.c(revision 0) *** *** 0 --- 1,15 + /* PR lto/48437 */ + /* { dg-do run } */ + /* { dg-options -g }

Re: [PATCH][LTO] Fix PR48437

2011-12-07 Thread Richard Guenther
On Wed, 7 Dec 2011, Jakub Jelinek wrote: On Wed, Dec 07, 2011 at 03:52:35PM +0100, Richard Guenther wrote: *** gcc/testsuite/gcc.dg/guality/pr48437.c (revision 0) --- gcc/testsuite/gcc.dg/guality/pr48437.c (revision 0) *** *** 0 --- 1,15 + /* PR lto/48437 */

Re: [C++ PATCH] Fix ICE in build_value_init (PR c++/51369)

2011-12-07 Thread Jakub Jelinek
On Wed, Dec 07, 2011 at 10:10:08AM -0500, Jason Merrill wrote: On 12/06/2011 02:48 PM, Jakub Jelinek wrote: - gcc_assert (!processing_template_decl || SCALAR_TYPE_P (type)); + gcc_assert (!processing_template_decl + || (SCALAR_TYPE_P (type) || TREE_CODE (type) == ARRAY_TYPE));

Re: [C++ PATCH] Fix ICE in build_value_init (PR c++/51369)

2011-12-07 Thread Jason Merrill
On 12/07/2011 10:18 AM, Jakub Jelinek wrote: On Wed, Dec 07, 2011 at 10:10:08AM -0500, Jason Merrill wrote: On 12/06/2011 02:48 PM, Jakub Jelinek wrote: - gcc_assert (!processing_template_decl || SCALAR_TYPE_P (type)); + gcc_assert (!processing_template_decl + || (SCALAR_TYPE_P

[SMS, DDG] Additional edges to instructions with clobber

2011-12-07 Thread Roman Zhuykov
Apologies for the messed up previous e-mails. This letter contains the first separate patch for ddg.c It creates additional nessesary anti-dep edges in data dependency graph. 2011/10/12 Ayal Zaks ayal.z...@gmail.com: The following situation happens when SMS is enabled without register

[SMS, DDG] Correctly delete anti-dep edges for renaming

2011-12-07 Thread Roman Zhuykov
This letter contains second separate patch for ddg.c It prevents removing some edges in data dependency graph when renaming is allowed. 2011/10/12 Ayal Zaks ayal.z...@gmail.com: The other bug happens only with -fmodulo-sched-allow-regmoves. Here we eliminate some anti-dependence edges in data

Re: [PATCH][LTO] Fix PR48437

2011-12-07 Thread Diego Novillo
On 12/07/11 09:52, Richard Guenther wrote: Index: gcc/lto-streamer-out.c === *** gcc/lto-streamer-out.c (revision 182081) --- gcc/lto-streamer-out.c (working copy) *** tree_is_indexable (tree t) *** 129,134

Re: [PATCH][LTO] Fix PR48437

2011-12-07 Thread Richard Guenther
On Wed, 7 Dec 2011, Diego Novillo wrote: On 12/07/11 09:52, Richard Guenther wrote: Index: gcc/lto-streamer-out.c === *** gcc/lto-streamer-out.c (revision 182081) --- gcc/lto-streamer-out.c (working copy)

Re: [PATCH][LTO] Fix PR48437

2011-12-07 Thread Diego Novillo
On 12/07/11 10:46, Richard Guenther wrote: On Wed, 7 Dec 2011, Diego Novillo wrote: On 12/07/11 09:52, Richard Guenther wrote: Index: gcc/lto-streamer-out.c === *** gcc/lto-streamer-out.c (revision 182081) ---

Re: [PATCH][LTO] Fix PR48437

2011-12-07 Thread Richard Guenther
On Wed, 7 Dec 2011, Diego Novillo wrote: On 12/07/11 10:46, Richard Guenther wrote: On Wed, 7 Dec 2011, Diego Novillo wrote: On 12/07/11 09:52, Richard Guenther wrote: Index: gcc/lto-streamer-out.c === ***

Re: [PATCH][LTO] Fix PR48437

2011-12-07 Thread Diego Novillo
On 12/07/11 10:54, Richard Guenther wrote: On Wed, 7 Dec 2011, Diego Novillo wrote: On 12/07/11 10:46, Richard Guenther wrote: On Wed, 7 Dec 2011, Diego Novillo wrote: On 12/07/11 09:52, Richard Guenther wrote: Index: gcc/lto-streamer-out.c

Re: [google] Backport r171347 and r181549 from trunk (strict volatile bitfield) (issue5434084)

2011-12-07 Thread Joseph S. Myers
On Wed, 7 Dec 2011, Richard Guenther wrote: code - well, that's ok. Pointing people to the latest official release series (in this case 4.6.x) is also ok, we're keeping too many branches active IMNSHO. As I recall we agreed in London that both 4.3 and 4.4 should be closed (after a final

Re: [PATCH][LTO] Fix PR48437

2011-12-07 Thread Richard Guenther
* gcc.dg/lto/20111207-2_0.c: New testcase. * gcc.dg/guality/pr48437.c: Likewise. Index: gcc/lto-streamer-out.c === *** gcc/lto-streamer-out.c (revision 182081) --- gcc/lto-streamer-out.c (working copy

Re: [PATCH][LTO] Fix PR48437

2011-12-07 Thread Diego Novillo
On Wed, Dec 7, 2011 at 11:16, Richard Guenther rguent...@suse.de wrote: I'm going to apply it tomorrow, when full testing hopefully finished Sure. But remember the zombie kitties! Diego.

Re: [PATCH][ARM] one_cmpldi2 in NEON

2011-12-07 Thread Richard Earnshaw
On 06/12/11 17:59, Andrew Stubbs wrote: This patch adds a one's complement pattern for doing DImode 'not' in NEON registers. There are already patterns for doing one's complement of vectors, and even though it boils down to the same instruction, the DImode case was missing. The patch

Re: [google] Backport r171347 and r181549 from trunk (strict volatile bitfield) (issue5434084)

2011-12-07 Thread Jakub Jelinek
On Wed, Dec 07, 2011 at 04:16:25PM +, Joseph S. Myers wrote: On Wed, 7 Dec 2011, Richard Guenther wrote: code - well, that's ok. Pointing people to the latest official release series (in this case 4.6.x) is also ok, we're keeping too many branches active IMNSHO. As I recall we

Re: Tidy up MD_INCLUDES in config/arm/t-arm

2011-12-07 Thread Richard Earnshaw
On 01/12/11 11:09, Georg-Johann Lay wrote: Richard Earnshaw wrote: On 29/11/11 09:42, Matthew Gretton-Dann wrote: All, Whilst developing the Cortex-A15 integer pipeline patch it was noted that the MD_INCLUDES variable in config/arm/t-arm has not been kept up-to-date. The attached patch

Re: [patch] ARM: Fix miscompilation in arm.md:*minmax_arithsi. (PR target/51408)

2011-12-07 Thread Richard Earnshaw
On 05/12/11 10:42, Richard Earnshaw wrote: On 04/12/11 13:32, kazu_hir...@mentor.com wrote: Hi, Attached is a patch to fix miscompilation in arm.md:*minmax_arithsi. The following testcase, reduced from efgcvt_r.c:fcvt_r in glibc, gets miscompiled: extern void abort (void); int

[Patch, Fortran] PR51448 [4.6/4.7] Fix realloc with RHS conversion function

2011-12-07 Thread Tobias Burnus
This fixes a -frealloc-lhs regression where the RHS is handled by a conversion function whose argument has component refs. Build and regtested on x86-64-linux. OK for the trunk and 4.7? Tobias 2011-12-07 Tobias Burnus bur...@net-b.de PR fortran/51448 * fortran/trans-array.c

Re: [C++ PATCH] ICE with invalid user-defined literals (PR c++/51420)

2011-12-07 Thread Jason Merrill
Applied, thanks. Jason

Re: Fix a bug in ThreadSanitizer pass (issue 5448109)

2011-12-07 Thread Xinliang David Li
ok for google/main. David On Wed, Dec 7, 2011 at 3:04 AM, dvyu...@google.com wrote: On 2011/12/05 17:05:17, dvyukov wrote: This is for google-main branch. Fix taking address of SSA_NAME in ThreadSanitizer pass. Index: gcc/tree-tsan.c

[PATCH] increase timeout in simulate-thread gdb test

2011-12-07 Thread Jack Howarth
Currently we are failing... FAIL: gcc.dg/simulate-thread/atomic-load-int128.c -O1 -g thread simulation test FAIL: gcc.dg/simulate-thread/atomic-load-int128.c -O2 -g thread simulation test FAIL: gcc.dg/simulate-thread/atomic-load-int128.c -O3 -g thread simulation test FAIL:

Re: [PATCH] increase timeout in simulate-thread gdb test

2011-12-07 Thread Uros Bizjak
On Wed, Dec 7, 2011 at 7:43 PM, Jack Howarth howa...@bromo.med.uc.edu wrote: Currently we are failing... FAIL: gcc.dg/simulate-thread/atomic-load-int128.c  -O1 -g  thread simulation test FAIL: gcc.dg/simulate-thread/atomic-load-int128.c  -O2 -g  thread simulation test FAIL:

Re: Update to Fortran invoke documentation about the features -finit-type *really* provides.

2011-12-07 Thread Toon Moene
On 12/06/2011 08:32 PM, Steve Kargl wrote: On Mon, Dec 05, 2011 at 07:21:59PM +0100, Toon Moene wrote: 2011-12-05 Toon Moenet...@moene.org PR/51310 invoke.texi: Itemize the cases for which -finit-type doesn't work. OK for trunk ? (and perhaps later for the 4.6

Re: [PATCH] increase timeout in simulate-thread gdb test

2011-12-07 Thread Iain Sandoe
On 7 Dec 2011, at 18:47, Uros Bizjak wrote: On Wed, Dec 7, 2011 at 7:43 PM, Jack Howarth howa...@bromo.med.uc.edu wrote: Currently we are failing... FAIL: gcc.dg/simulate-thread/atomic-load-int128.c -O1 -g thread simulation test FAIL: gcc.dg/simulate-thread/atomic-load-int128.c -O2 -g

Re: [PATCH] increase timeout in simulate-thread gdb test

2011-12-07 Thread Uros Bizjak
On Wed, Dec 7, 2011 at 7:58 PM, Iain Sandoe develo...@sandoe-acoustics.co.uk wrote: Currently we are failing... FAIL: gcc.dg/simulate-thread/atomic-load-int128.c  -O1 -g  thread simulation test FAIL: gcc.dg/simulate-thread/atomic-load-int128.c  -O2 -g  thread simulation test FAIL:

Re: [PATCH] [MIPS] Add -march=octeon+ support for GCC

2011-12-07 Thread Andrew Pinski
On Tue, Dec 6, 2011 at 6:28 PM, Andrew Pinski pins...@gmail.com wrote: Hi,  This patch adds -march=octeon+ to GCC. OK? Bootstrapped and tested on mips64-linux-gnu configured with --with-arch=octeon+ . Thanks, Andrew Pinski gcc/ChangeLog: * mips/mips-cpus.def (octeon+): New CPU. *

Re: [Patch, Fortran] PR51448 [4.6/4.7] Fix realloc with RHS conversion function

2011-12-07 Thread Mikael Morin
On Wednesday 07 December 2011 17:45:52 Tobias Burnus wrote: This fixes a -frealloc-lhs regression where the RHS is handled by a conversion function whose argument has component refs. Build and regtested on x86-64-linux. OK for the trunk and 4.7? Tobias OK. Mikael

Re: PR 51386

2011-12-07 Thread François Dumont
Attached patch applied: 2011-12-07 François Dumont fdum...@gcc.gnu.org PR libstdc++/51386 * include/bits/hashtable_policy.h (_Prime_rehash_policy::_M_next_bkt): Fix computation of _M_prev_resize so that hashtable do not keep on being rehashed when

Re: [Patch, Fortran] PR51407 - allow BOZ edit descriptors for REAL/COMPLEX

2011-12-07 Thread Mikael Morin
On Wednesday 07 December 2011 14:54:36 Tobias Burnus wrote: * ping * ? On 12/04/2011 06:46 PM, Tobias Burnus wrote: Hi all, as Dominique has found, Fortran 2008 allows the BOZ edit descriptors now also with REAL and COMPLEX arguments. (See PR for quotes from the standard.)

Re: [Patch, Fortran] PR 51378 Fix component-access check

2011-12-07 Thread Mikael Morin
On Friday 02 December 2011 22:01:19 Tobias Burnus wrote: Found via Reinhold Bader's test suite: If a component is public, it remains public even if the extended type has PRIVATE. Build and regtested on x86-64-linux. OK for the trunk? OK. Mikael

[PATCH] Avoid using constructor attribute in libcpp (PR bootstrap/50237)

2011-12-07 Thread Jakub Jelinek
Hi! While this isn't real fix for this PR, which should be fixed in configury, I think avoiding ctors in host code is useful for portability and Eric agreed with that in the PR. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2011-12-07 Jakub Jelinek ja...@redhat.com

[C++ PATCH] Error out on [N] = style designated initializers for classes (PR c++/51229)

2011-12-07 Thread Jakub Jelinek
Hi! We already error out on .foo = style designators for arrays, but for [N] = style designators for classes we'd just ICE. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? We still have a bunch of accepts-invalid, no idea how to solve them, because reshape_init_r is called

[C++ PATCH] Fix ICE in cxx_incomplete_type_diagnostic (PR c++/51429)

2011-12-07 Thread Jakub Jelinek
Hi! This testcase regressed recently when Paolo added the user friendlier invalid use of member function vs. invalid use of member distinction, when TREE_OPERAND (value, 1) is BASELINK, DECL_FUNCTION_MEMBER_P on it dies with a checking ICE. The following patch calls get_first_fn if it is

Re: [C++ PATCH] Error out on [N] = style designated initializers for classes (PR c++/51229)

2011-12-07 Thread Jason Merrill
On 12/07/2011 03:25 PM, Jakub Jelinek wrote: Hi! We already error out on .foo = style designators for arrays, but for [N] = style designators for classes we'd just ICE. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? OK. The last one is a accepts-invalid even for C.

Re: [C++ PATCH] Fix ICE in cxx_incomplete_type_diagnostic (PR c++/51429)

2011-12-07 Thread Jason Merrill
OK. Jason

Re: [PATCH][ARM] one_cmpldi2 in NEON

2011-12-07 Thread Andrew Stubbs
On 07/12/11 16:25, Richard Earnshaw wrote: 2011-12-06 Andrew Stubbsa...@codesourcery.com gcc/ * config/arm/arm.md (one_cmpldi2): Rename to ... (one_cmpldi2_core): ... this, and modify it to prevent it being used for NEON. (one_cmpldi2): New

[C++ PATCH] Avoid ICE on auto non-static data members (PR c++/51401)

2011-12-07 Thread Jakub Jelinek
Hi! By my reading auto is not valid on non-static data members, if they aren't NSDMI, then we'd error on them (but differently between templates and non-templates), but with NSDMI in template we just ICE because init is DEFAULT_ARG. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for

Re: [Patch, Fortran] PR50815 - don't -fcheck=bounds of deferred-length strings

2011-12-07 Thread Mikael Morin
On Wednesday 07 December 2011 14:53:20 Tobias Burnus wrote: ** PING ** On 11/29/2011 07:35 PM, Tobias Burnus wrote: gfortran had an ICE when trying to insert a check whether the character length between actual and dummy argument matches. This check is pointless for deferred-length

Re: [PATCH] Avoid using constructor attribute in libcpp (PR bootstrap/50237)

2011-12-07 Thread Richard Henderson
On 12/07/2011 12:20 PM, Jakub Jelinek wrote: PR bootstrap/50237 * internal.h (_cpp_init_lexer): New prototype. * init.c (init_library): Call it. * lex.c (init_vectorized_lexer): Remove constructor attribute, add inline keyword.

[Patch, i386] [4.6] Backport movd*_internal_rex64 fix from trunk and [4.6/4.7] add testcase

2011-12-07 Thread Teresa Johnson
An issue turned up in our internal 4.6 based testing that has been fixed on trunk. This patch backports the fix to 4.6. I also have a small test case that I will add to both 4.6 and 4.7. Bootstrapped and checked with x86_64-unknown-linux-gnu. Can someone review? Thanks, Teresa 2011-12-07

Re: [Patch, i386] [4.6] Backport movd*_internal_rex64 fix from trunk and [4.6/4.7] add testcase

2011-12-07 Thread Uros Bizjak
Hello! An issue turned up in our internal 4.6 based testing that has been fixed on trunk. This patch backports the fix to 4.6. I also have a small test case that I will add to both 4.6 and 4.7. Bootstrapped and checked with x86_64-unknown-linux-gnu. 2011-12-07 Teresa Johnson

Re: [Patch, i386] [4.6] Backport movd*_internal_rex64 fix from trunk and [4.6/4.7] add testcase

2011-12-07 Thread Teresa Johnson
On Wed, Dec 7, 2011 at 2:12 PM, Uros Bizjak ubiz...@gmail.com wrote: Hello! An issue turned up in our internal 4.6 based testing that has been fixed on trunk. This patch backports the fix to 4.6. I also have a small test case that I will add to both 4.6 and 4.7. Bootstrapped and checked

[Committed] Fix PR libffi/50051 (MIPS libffi does not compile for mips64octeon-linux-gnu)

2011-12-07 Thread Andrew Pinski
Hi, The problem here is Cavium's octeon assembler rejects floating point if the arch is set to either octeon or octeon2. This fixes the issue by adding: .set mips4 so that floating point instructions are enabled. Committed as approved by Anthony in the bugzilla. Thanks, Andrew Pinski

Re: [v3] RFC: rename __calculate_memory_order

2011-12-07 Thread Benjamin Kosnik
* include/bits/atomic_base.h (__calculate_memory_order): Rename to... (__cmpexch_failure_order): This, and rewrite as constexpr function. (compare_exchange_strong, compare_exchange_weak): Use it. * include/std/atomic (compare_exchange_strong, compare_exchange_weak):

-finstrument-functions leads to unsats to _mm instrinsic wrappers

2011-12-07 Thread Xinliang David Li
Build the test case in the patch file with -finstrument-functions, the link will fail with unsat. The problem is gcc instruments the artificial wrappers that will won't be emitted. The patch fixes the problem. Bootstrap and tested on x86-64/linux. Ok for mainline? thanks, David Index:

Re: -finstrument-functions leads to unsats to _mm instrinsic wrappers

2011-12-07 Thread Andrew Pinski
On Wed, Dec 7, 2011 at 4:01 PM, Xinliang David Li davi...@google.com wrote: Build the test case in the patch file with -finstrument-functions, the link will fail with unsat. The problem is gcc instruments the artificial wrappers that will won't be emitted. The patch fixes the problem.

Re: -finstrument-functions leads to unsats to _mm instrinsic wrappers

2011-12-07 Thread Xinliang David Li
On Wed, Dec 7, 2011 at 4:25 PM, Andrew Pinski pins...@gmail.com wrote: On Wed, Dec 7, 2011 at 4:01 PM, Xinliang David Li davi...@google.com wrote: Build the test case in the patch file with -finstrument-functions, the link will fail with unsat. The problem is gcc instruments the artificial

Re: [C++ PATCH] Avoid ICE on auto non-static data members (PR c++/51401)

2011-12-07 Thread Jason Merrill
OK. Jason

[4.7][google] Adding a new option -fstack-protector-strong. (issue5461043)

2011-12-07 Thread Han Shen
Hi, this patch provides a new stack protection option - fstack-protector-strong. Background - some times stack-protector is too-simple while stack-protector-all over-kills, for example, to build one of our core systems, we forcibly add -fstack-protector-all to all compile commands, which

Re: [4.7][google] Adding a new option -fstack-protector-strong. (issue5461043)

2011-12-07 Thread Andrew Pinski
On Wed, Dec 7, 2011 at 5:07 PM, Han Shen shen...@google.com wrote: +  /* Examine each basic block for address taking of local variables. */ I don't think you need to look at the basic blocks to figure out if a local variable has its address taken. You can look through referenced variables and

Re: [Patch, i386] [4.6] Backport movd*_internal_rex64 fix from trunk and [4.6/4.7] add testcase

2011-12-07 Thread Uros Bizjak
On Wed, Dec 7, 2011 at 11:32 PM, Teresa Johnson tejohn...@google.com wrote: An issue turned up in our internal 4.6 based testing that has been fixed on trunk. This patch backports the fix to 4.6. I also have a small test case that I will add to both 4.6 and 4.7. Bootstrapped and checked with