Re: libgo patch committed: Fix build on MIPS GNU/Linux

2012-05-02 Thread Andrew Pinski
On Tue, May 1, 2012 at 12:18 PM, Ian Lance Taylor i...@google.com wrote: Andrew Pinski pins...@gmail.com writes: I tested the trunk after this patch on mips64-linux-gnu (with glibc 2.15) and the testsuite looks good. Thanks for testing. FAIL: go.test/test/nilptr.go execution,  -O2 -g The

Re: [patch, committed] invoke.texi: clean up texinfo markup

2012-05-02 Thread Gerald Pfeifer
Hi Sandra, On Fri, 6 Apr 2012, Sandra Loosemore wrote: This is another installment in my series of cleanups to invoke.texi. In this patch I have taken a break from nit-picking grammar and have nit-picked some Texinfo markup issues instead. kudos for the work you are doing on this front!

Re: [PATCH] libatomic, v2

2012-05-02 Thread Gerald Pfeifer
Hi Richard, On Tue, 1 May 2012, Richard Henderson wrote: Now committed. mind writing up something for the release notes? If you provide me text, I can take care of marking it up and committing it. Gerald

RE: [PATCH, reload] Fix bug pr52804, RELOAD pass reloads wrong register on ARM for cortex-m0

2012-05-02 Thread Bin Cheng
Ping. Hi, could anyone help me with this bug please, if you have time? Any comments will be appreciated. Thanks very much. -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On Behalf Of Bin Cheng Sent: Friday, April 20, 2012 1:51 PM To:

Re: [Patch, fortran] PR41600 - [OOP] SELECT TYPE with associate-name = exp: Arrays not supported

2012-05-02 Thread Paul Richard Thomas
Dear Tobias, Thanks for completing the review. I should be able to commit tonight. Thanks for the patch. I think it is OK. Regarding: !       if (ref  ref-type != REF_ARRAY  seen_array) !       { !         gfc_error (CLASS selector at %L is an array with CLASS !                    

[PATCH] Fix overzealous DSE on sparc

2012-05-02 Thread David Miller
On targets such as sparc, where ARG_POINTER_REGNUM == FRAME_POINTER_REGNUM, some of the invariants currently built into DSE simply do not hold. Unlike how DSE assumes, we will in fact see stores to frame pointer relative addresses for setting up outgoing arguments to a CALL. The result is that

[patch] PR53153

2012-05-02 Thread Steven Bosscher
Hello, The attached patch fixes PR53153. The problem is that tree-ssa-forwprop.c propagates away a cast, but leaves out-of-range case labels in the switch. Before my patch for r186579, the code in stmt.c would remove such case labels before any of the RTL expanders got to see them. After my

Re: [PATCH] teach phi-opt to produce -(a COND b)

2012-05-02 Thread Richard Guenther
On Tue, 1 May 2012, Richard Henderson wrote: On 04/27/2012 03:01 AM, Paolo Bonzini wrote: This patch teaches phiopt to look at phis whose arguments are -1 and 0, and produce negated setcc statements. Is this really a win over a COND_EXPR, i.e. (a b ? -1 : 0)? There is quite a bit of

Re: [PATCH RFC] Warn when optimization flag is given without -O

2012-05-02 Thread Richard Guenther
On Sat, Apr 28, 2012 at 4:48 PM, Alexander Monakov amona...@ispras.ru wrote: Looks like people who want to play with optimization options are still tripped by the fact that optimizations are not enabled unless -O is given (even though it's documented in the manual now).  Can we add a warning

Re: Optimize calls to functions that return one of their arguments

2012-05-02 Thread Richard Guenther
On Sat, Apr 28, 2012 at 5:31 PM, Bernd Schmidt ber...@codesourcery.com wrote: This patch allows us to recognize that even if the argument to memcpy lives across the call, we can allocate it to a call-used register by reusing the return value of the function. First, the patch sets the existing

Re: [patch] Fix cygwin ada install [was Re: Yet another issue with gcc current trunk with ada on cygwin]

2012-05-02 Thread Arnaud Charlet
Pascal, I'd suggest you go ahead and revert this patch. This patch has already been reverted by Eric on Apr 10 on gcc-4.7 branch. I'm talking about the FSF trunk repository. Arno

Re: [patch] Fix cygwin ada install [was Re: Yet another issue with gcc current trunk with ada on cygwin]

2012-05-02 Thread Pascal Obry
Arno, This patch has already been reverted by Eric on Apr 10 on gcc-4.7 branch. I'm talking about the FSF trunk repository. Ah ok, but I do not have commit access to this repository. Eric maybe? -- Pascal Obry -- gpg --keyserver keys.gnupg.net --recv-key F949BD3B

Re: [patch] Fix cygwin ada install [was Re: Yet another issue with gcc current trunk with ada on cygwin]

2012-05-02 Thread Arnaud Charlet
I'm talking about the FSF trunk repository. Ah ok, but I do not have commit access to this repository. Eric maybe? If you send me a patch with changelog, I'll take care of it. Arno

Re: [PATCH] Don't ignore compute_all_dependences failures in phiopt (PR tree-optimization/53163)

2012-05-02 Thread Richard Guenther
On Mon, Apr 30, 2012 at 3:39 PM, Jakub Jelinek ja...@redhat.com wrote: Hi! compute_all_dependences in 4.7+ can fail, and cond_if_else_store_replacement isn't prepared to handle the chrec_dont_know DDR added there in case of failure (with NULL DDR_A/DDR_B). Fixed thusly,

Re: [PATCH 1/2] Minor refactoring of tree-vect-patterns.c

2012-05-02 Thread Richard Guenther
On Mon, Apr 30, 2012 at 6:19 PM, Ulrich Weigand uweig...@de.ibm.com wrote: Hello, in working on a fix for PR 52633, I noticed that tree-vect-patterns.c now contains a number of copies of rather similar code (of which my patch would have added another copy), so it seems to make sense to do a

Re: [PATCH 2/2] Minor refactoring of tree-vect-patterns.c

2012-05-02 Thread Richard Guenther
On Mon, Apr 30, 2012 at 6:19 PM, Ulrich Weigand uweig...@de.ibm.com wrote: Hello, as a second step in refactoring this patch introduces a routine vect_find_single_use to determine whether a defining statement has one single use within the current vectorization domain. The helper is then

RFA: PR target/53120, constraint modifier + on operand tied by matching-constraint, 0.

2012-05-02 Thread nick clifton
Hi DJ, As pointed out by Hans-Peter in his patch for PR 53120 for the cris backend, the m32c/bitops.md file contains a pattern that uses both the + modifier and the 0 constraint. The patch below is a straightforward fix for this, replacing the 0 constraint with a match_dup. Tested with

Re: [PATCH] Add -fdump-rtl-pass-quiet

2012-05-02 Thread Andrew Stubbs
On 19/04/12 13:58, Andrew Stubbs wrote: In the meantime, Mr Maintainers, can I commit my patch while we wait for the new world order? I'm happy to change the option name quiet to something else if necessary. Ping. I think David may have a point about 'quiet' being an inappropriate name for

Re: Backported r185231 from trunk. (issue 6139063)

2012-05-02 Thread Richard Guenther
On Mon, Apr 30, 2012 at 9:54 PM, asha...@chromium.org wrote: Reviewers: xur, davidxl, iant2, Message: I backported the following patch: Btw, there were various followup commits fixing issues on some ports. Richard. 2012-03-12  Richard Guenther  rguent...@suse.de        * gthr.h

Re: [2/6] Fold prev/next into gimple: fewer gimple_seq_alloc calls

2012-05-02 Thread Richard Guenther
On Wed, May 2, 2012 at 3:06 AM, Michael Matz m...@suse.de wrote: Hi, This patch gets rid of some easy gimple_seq_alloc calls that we don't even need currently because most gimple_seq modifier will lazily allocate one. Most of the time it's enough to simply initialize a seq to NULL and call

Re: [PATCH RFC] Warn when optimization flag is given without -O

2012-05-02 Thread Alexander Monakov
On Wed, 2 May 2012, Richard Guenther wrote: That's surely too coarse - you will warn about options that _will_ have an effect. There is currently no easy way to glob all affected options. Yeah, that's unfortunate. Do we want to have a separate flag (CL_OPTIMIZATION_O0) to mark such

Re: [3/6] Fold prev/next into gimple

2012-05-02 Thread Richard Guenther
On Wed, May 2, 2012 at 3:14 AM, Michael Matz m...@suse.de wrote: Hi, this introduces a new helper (gsi_replace_with_seq) which can replace a single statement with a sequence, and makes use of it in gimplify_and_update_call_from_tree.  This make sure that the statements aren't inserted into

Re: [PATCH] Don't ignore compute_all_dependences failures in phiopt (PR tree-optimization/53163)

2012-05-02 Thread Jakub Jelinek
On Wed, May 02, 2012 at 11:24:13AM +0200, Richard Guenther wrote: Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk/4.7? Ok. Thanks. BTW, tree-ssa-loop-prefetch.c seems to have the same problem, but no idea how that should be handled in there... I

Re: [5/6] Fold prev/next into gimple

2012-05-02 Thread Richard Guenther
On Wed, May 2, 2012 at 3:28 AM, Michael Matz m...@suse.de wrote: Hi, this patch basically is the one that makes all interfaces that possibly change a gimple_seq take a pointer to one, this time _including_ the statement iterator. For that I opted to use the same idiom as our basic block

Re: [committed] Run DImode atomic tests on ppc64

2012-05-02 Thread Uros Bizjak
Hello! * lib/target-supports.exp (check_effective_target_sync_long_long_runtime): True for check_effective_target_powerpc64. @@ -3795,6 +3795,8 @@ proc check_effective_target_sync_long_long_runtime { } { [check_effective_target_lp64]

Re: [patch] PR53153

2012-05-02 Thread Richard Guenther
On Wed, May 2, 2012 at 10:32 AM, Steven Bosscher stevenb@gmail.com wrote: Hello, The attached patch fixes PR53153. The problem is that tree-ssa-forwprop.c propagates away a cast, but leaves out-of-range case labels in the switch. Before my patch for r186579, the code in stmt.c would

Re: [patch] Fix cygwin ada install [was Re: Yet another issue with gcc current trunk with ada on cygwin]

2012-05-02 Thread Pascal Obry
Arno, The ChangeLog: Revert commit r184558 as breaking Ada runtime. The pseudo reloc feature works only when linking against DLL. Remove this pacth as it install the import libraries for the shared Ada runtime. This breaks any Ada program using the Ada shared runtime. With

Re: [PATCH RFC] Warn when optimization flag is given without -O

2012-05-02 Thread Richard Guenther
On Wed, May 2, 2012 at 12:03 PM, Alexander Monakov amona...@ispras.ru wrote: On Wed, 2 May 2012, Richard Guenther wrote: That's surely too coarse - you will warn about options that _will_ have an effect. There is currently no easy way to glob all affected options. Yeah, that's

Re: [PATCH] Don't ignore compute_all_dependences failures in phiopt (PR tree-optimization/53163)

2012-05-02 Thread Richard Guenther
On Wed, May 2, 2012 at 12:04 PM, Jakub Jelinek ja...@redhat.com wrote: On Wed, May 02, 2012 at 11:24:13AM +0200, Richard Guenther wrote: Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk/4.7? Ok. Thanks. BTW, tree-ssa-loop-prefetch.c seems to have the

[PATCH] [i386] fma3 instruction generation for 'march=native' in AMD processors

2012-05-02 Thread Gopalasubramanian, Ganesh
For AMD architectures with both fma3 and fma4 instructions' support, GCC generates fma4 by default. Instead, we like to generate fma3 instruction. Below patch enables the fma3 instruction generation for -march=native. Ok for trunk? Index: gcc/config/i386/driver-i386.c

Re: [PATCH] Make sizetypes no longer sign-extending

2012-05-02 Thread Richard Guenther
On Fri, 27 Apr 2012, Eric Botcazou wrote: Ah, and all ACATS fails and -FAIL: gnat.dg/loop_optimization3.adb (test for excess errors) -FAIL: gnat.dg/loop_optimization3.adb execution test -FAIL: gnat.dg/test_8bitlong_overflow.adb (test for excess errors) -FAIL:

Re: [PATCH] [i386] fma3 instruction generation for 'march=native' in AMD processors

2012-05-02 Thread Jakub Jelinek
On Wed, May 02, 2012 at 11:12:33AM +, Gopalasubramanian, Ganesh wrote: For AMD architectures with both fma3 and fma4 instructions' support, GCC generates fma4 by default. Instead, we like to generate fma3 instruction. Below patch enables the fma3 instruction generation for -march=native.

Re: [RFH / Patch] PR 51222

2012-05-02 Thread Paolo Carlini
On 05/02/2012 05:01 AM, Paolo Carlini wrote: Anyway, the attached appear to also pass the testsuite, I could test it more completely tomorrow. If the FIELD_DECL bits seem safe to have... ;) Now fully booted and tested. I also booted and tested a variant, attached, which changes

Re: [patch] More thorough checking in reg_fits_class_p

2012-05-02 Thread Jim MacArthur
On 30/04/12 16:19, Richard Sandiford wrote: Richard Earnshawrearn...@arm.com writes: On 30/04/12 15:39, Richard Sandiford wrote: Richard Earnshawrearn...@arm.com writes: On 30/04/12 15:07, Richard Sandiford wrote: Richard Earnshawrearn...@arm.com writes: On 26/04/12 14:20, Jim MacArthur

Re: [PATCH] Improve COND_EXPR expansion

2012-05-02 Thread William J. Schmidt
On Mon, 2012-04-30 at 20:22 -0700, Andrew Pinski wrote: Hi, This patch improves the expansion of COND_EXPR into RTL, directly using conditional moves. I had to fix a bug in the x86 backend where emit_conditional_move could cause a crash as we had a comparison mode of DImode which is not

Re: [PATCH] Fix overzealous DSE on sparc

2012-05-02 Thread Kenneth Zadeck
this looks ok to me, but I am going to defer this to richard for final acceptance. his contribution to this pass was his deep knowledge of rtl so that we could get the scanning correct and this is clearly in that domain. He may have some trick that does not throw all of the baby out with

Re: [PATCH] libatomic, v2

2012-05-02 Thread Andreas Krebbel
On 05/01/2012 05:53 PM, Richard Henderson wrote: On 04/27/2012 12:26 PM, Richard Henderson wrote: I think the library is ready for merge back to mainline, but I will wait until at least Monday for feedback and proximity to my desk to deal with potential fallout. Now committed. With that

Re: [patch] Fix cygwin ada install [was Re: Yet another issue with gcc current trunk with ada on cygwin]

2012-05-02 Thread Arnaud Charlet
OK, revision 184558 now reverted. Arno -- Index: gcc/ada/ChangeLog === --- gcc/ada/ChangeLog (revision 187037) +++ gcc/ada/ChangeLog (working copy) @@ -1,3 +1,8 @@ +2012-05-02 Pascal Obry o...@adacore.com + + Revert +

Re: [patch] PR53153

2012-05-02 Thread Steven Bosscher
On Wed, May 2, 2012 at 12:14 PM, Richard Guenther richard.guent...@gmail.com wrote: On Wed, May 2, 2012 at 10:32 AM, Steven Bosscher stevenb@gmail.com wrote: Hello, The attached patch fixes PR53153. The problem is that tree-ssa-forwprop.c propagates away a cast, but leaves

Re: [patch] More thorough checking in reg_fits_class_p

2012-05-02 Thread Richard Sandiford
Jim MacArthur jim.macart...@arm.com writes: New Changelog text: 2012-05-02 Jim MacArthurjim.macart...@arm.com * recog.c (reg_fits_class_p): Check both regno and regno + offset are hard registers. Thanks. I still think the final: +HARD_REGISTER_NUM_P (end_hard_regno (regno +

Re: [PATCH] libatomic, v2

2012-05-02 Thread Richard Henderson
On 05/02/2012 05:48 AM, Andreas Krebbel wrote: configure.ac:26: error: Please use exactly Autoconf 2.64 instead of 2.65. ../config/override.m4:12: _GCC_AUTOCONF_VERSION_CHECK is expanded from... configure.ac:26: the top level autom4te: m4 failed with exit status: 1 aclocal-1.11: autom4te failed

Re: libgo patch committed: Fix build on MIPS GNU/Linux

2012-05-02 Thread Ian Lance Taylor
Andrew Pinski pins...@gmail.com writes: n32 failures: --- FAIL: runtime_test.TestGcSys (0.56 seconds) mfinal_test.go:35: used 4718592 extra bytes mfinal_test.go:37: using too much memory: 4718592 bytes FAIL FAIL: runtime

[patch] MULTILIB_DEFAULTS needs an update for ARM multilib builds

2012-05-02 Thread Matthias Klose
I did see gcc-4.7 fail to build for an ARM soft-float/hard-float multilib configuration. The reason is that gcc -print-multi-directory doesn't print anything for the non-default, and gcc -print-multi-lib only prints `.' (and then not building the runtime libs for the non-default). The reason is

Re: User directed Function Multiversioning via Function Overloading (issue5752064)

2012-05-02 Thread H.J. Lu
On Tue, May 1, 2012 at 7:45 PM, Sriraman Tallam tmsri...@google.com wrote: Hi H.J,   Done now. Patch attached. Thanks, -Sri. On Tue, May 1, 2012 at 5:08 PM, H.J. Lu hjl.to...@gmail.com wrote: On Tue, May 1, 2012 at 4:51 PM, Sriraman Tallam tmsri...@google.com wrote: Hi, New patch

Re: [patch] More thorough checking in reg_fits_class_p

2012-05-02 Thread Richard Earnshaw
On 02/05/12 14:00, Richard Sandiford wrote: Jim MacArthur jim.macart...@arm.com writes: New Changelog text: 2012-05-02 Jim MacArthurjim.macart...@arm.com * recog.c (reg_fits_class_p): Check both regno and regno + offset are hard registers. Thanks. I still think the final: +

Re: [PATCH] libatomic, v2

2012-05-02 Thread Andreas Krebbel
On 05/02/2012 03:14 PM, Richard Henderson wrote: On 05/02/2012 05:48 AM, Andreas Krebbel wrote: configure.ac:26: error: Please use exactly Autoconf 2.64 instead of 2.65. ../config/override.m4:12: _GCC_AUTOCONF_VERSION_CHECK is expanded from... configure.ac:26: the top level autom4te: m4

Re: [patch] More thorough checking in reg_fits_class_p

2012-05-02 Thread Richard Sandiford
Richard Earnshaw rearn...@arm.com writes: On 02/05/12 14:00, Richard Sandiford wrote: Jim MacArthur jim.macart...@arm.com writes: New Changelog text: 2012-05-02 Jim MacArthurjim.macart...@arm.com * recog.c (reg_fits_class_p): Check both regno and regno + offset are hard registers.

Re: [PATCH] Fix for PR51879 - Missed tail merging with non-const/pure calls

2012-05-02 Thread Tom de Vries
On 27/04/12 11:01, Richard Guenther wrote: SNIP I see you do not handle SNIP struct S { int i; }; struct S foo (void); struct S bar (void) { struct S s1, s2; if (...) s = foo (); else s = foo (); because the calls have a LHS that is not an SSA name. Indeed, the gvn patch

Spital nou de pediatrie

2012-05-02 Thread cramer
http://www.youtube.com/watch?feature=player_embeddedv=phjGxHn3uKU To unsubscribe please send email to unsubscr...@cc.psd-prahova.ro

Re: [patch] MULTILIB_DEFAULTS needs an update for ARM multilib builds

2012-05-02 Thread Richard Earnshaw
On 02/05/12 14:26, Matthias Klose wrote: I did see gcc-4.7 fail to build for an ARM soft-float/hard-float multilib configuration. The reason is that gcc -print-multi-directory doesn't print anything for the non-default, and gcc -print-multi-lib only prints `.' (and then not building the

Re: [patch] MULTILIB_DEFAULTS needs an update for ARM multilib builds

2012-05-02 Thread Matthias Klose
On 02.05.2012 16:53, Richard Earnshaw wrote: On 02/05/12 14:26, Matthias Klose wrote: I did see gcc-4.7 fail to build for an ARM soft-float/hard-float multilib configuration. The reason is that gcc -print-multi-directory doesn't print anything for the non-default, and gcc -print-multi-lib only

Re: [patch] MULTILIB_DEFAULTS needs an update for ARM multilib builds

2012-05-02 Thread Richard Earnshaw
On 02/05/12 15:59, Matthias Klose wrote: On 02.05.2012 16:53, Richard Earnshaw wrote: On 02/05/12 14:26, Matthias Klose wrote: I did see gcc-4.7 fail to build for an ARM soft-float/hard-float multilib configuration. The reason is that gcc -print-multi-directory doesn't print anything for the

Re: [RFH / Patch] PR 51222

2012-05-02 Thread Jason Merrill
On 05/01/2012 11:01 PM, Paolo Carlini wrote: Well, not really. The first time walk_template_parms_r is called, t is a SIZEOF_EXPR which remains unhandled, the function returns NULL_TREE. The second time, t is a COMPONENT_REF, thus !TYPE_P (t) is true, TREE_TYPE is NULL, the function immediately

Re: User directed Function Multiversioning via Function Overloading (issue5752064)

2012-05-02 Thread Sriraman Tallam
On Wed, May 2, 2012 at 6:42 AM, H.J. Lu hjl.to...@gmail.com wrote: On Tue, May 1, 2012 at 7:45 PM, Sriraman Tallam tmsri...@google.com wrote: Hi H.J,   Done now. Patch attached. Thanks, -Sri. On Tue, May 1, 2012 at 5:08 PM, H.J. Lu hjl.to...@gmail.com wrote: On Tue, May 1, 2012 at 4:51

Re: [PATCH, i386, middle-end, tessuite] Intel TSX's HLE.

2012-05-02 Thread Kirill Yukhin
Otherwise, this looks good. Thanks, I've applied inputs! Comitted to MT: http://gcc.gnu.org/ml/gcc-cvs/2012-05/msg00047.html Thanks, K

Re: [patch] MULTILIB_DEFAULTS needs an update for ARM multilib builds

2012-05-02 Thread Matthias Klose
On 02.05.2012 17:02, Richard Earnshaw wrote: On 02/05/12 15:59, Matthias Klose wrote: On 02.05.2012 16:53, Richard Earnshaw wrote: On 02/05/12 14:26, Matthias Klose wrote: I did see gcc-4.7 fail to build for an ARM soft-float/hard-float multilib configuration. The reason is that gcc

Re: [PATCH, reload] Fix bug pr52804, RELOAD pass reloads wrong register on ARM for cortex-m0

2012-05-02 Thread Ulrich Weigand
Bin Cheng wrote: In short, I think the confliction of reloads with type RELOAD_FOR_INPADDR_ADDRESS and type RELOAD_FOR_INPUT_ADDRESS should be handled in reload_reg_reaches_end_p. Also I think RELOAD_FOR_OUTPUT_ADDRESS/RELOAD_FOR_OUTADDR_ADDRESS have the issue symmetrically, though I have

Re: User directed Function Multiversioning via Function Overloading (issue5752064)

2012-05-02 Thread H.J. Lu
On Wed, May 2, 2012 at 8:08 AM, Sriraman Tallam tmsri...@google.com wrote: On Wed, May 2, 2012 at 6:42 AM, H.J. Lu hjl.to...@gmail.com wrote: On Tue, May 1, 2012 at 7:45 PM, Sriraman Tallam tmsri...@google.com wrote: Hi H.J,   Done now. Patch attached. Thanks, -Sri. On Tue, May 1, 2012

Re: PR 53115

2012-05-02 Thread H.J. Lu
On Tue, May 1, 2012 at 1:23 PM, François Dumont frs.dum...@gmail.com wrote: unordered_multilmap test added, attached patch applied to 4.7 branch and trunk. This bug was not so difficult to fix. It would even have been quite easy to detect with a good test coverage tool showing that not all

Re: [patch] skip tpf configure tests

2012-05-02 Thread Benjamin De Kosnik
* crossconfig.m4: Since we know that all TPF builds are cross- builds and cannot run configuration-time link tests, do not allow it; just go with known supported linker options. * configure: Regenerate (called as GLIBCXX_CROSSCONFIG). OK -benjamin

[v3] hashtable fw decl fix

2012-05-02 Thread Benjamin De Kosnik
This patchlette needed for versioned builds, just makes declaration/foward declaration consistent. -benjamin tested x86/linux tested x86/linux versioned namespacesdiff --git a/libstdc++-v3/include/bits/hashtable_policy.h b/libstdc++-v3/include/bits/hashtable_policy.h index 3b43510..a42d3d3

Re: Patches to enable -ftrack-macro-expansion by default

2012-05-02 Thread Uros Bizjak
On Tue, May 1, 2012 at 6:04 PM, Dodji Seketeli do...@redhat.com wrote:     pfile-context = XNEW (cpp_context); +  memset (pfile-context, 0, sizeof (cpp_context)); You can use XCNEW instead of XNEW + memset. OK with that change. Thank you. Here is the patch I will commit when Uros

[PATCH] Fix predcom VTA ICE (PR debug/53174)

2012-05-02 Thread Jakub Jelinek
Hi! single_nonlooparound_use ignores debug uses (otherwise we end up with -fcompare-debug failures), but when removing stmts we need to reset the debug uses, otherwise e.g. the testcase below ICEs. Alex, if you have better ideas how to reconstruct the debug stmts instead, feel free to follow-up,

Re: Patches to enable -ftrack-macro-expansion by default

2012-05-02 Thread Dodji Seketeli
Uros Bizjak ubiz...@gmail.com writes: On Tue, May 1, 2012 at 6:04 PM, Dodji Seketeli do...@redhat.com wrote:     pfile-context = XNEW (cpp_context); +  memset (pfile-context, 0, sizeof (cpp_context)); You can use XCNEW instead of XNEW + memset. OK with that change. Thank you. Here is

RE: [PATCH 12/13] Adjust relevant test cases wrt -ftrack-macro-expansion=[0|2]

2012-05-02 Thread Greta Yorsh
There are a couple more test that need adjusting: gcc.dg/fixed-point/operator-bitwise.c gcc.dg/fixed-point/composite-type.c These tests fail on arm-none-eabi. Below is a patch that fixes them. Thanks, Greta gcc/testsuite 2012-05-02 Greta Yorsh greta.yo...@arm.com *

Re: [6/6] Fold prev/next into gimple: do it

2012-05-02 Thread Michael Matz
Hi, On Wed, 2 May 2012, Richard Guenther wrote: Please remove the flags. Okay. can you factor out this idiom to sth like gsi_new_seq_from_stmt () or so? Or gimple_init_seq_pointers? Not necessarily exported. gimple_init_singleton it is. The whole series is now at r187053. Ciao,

Re: User directed Function Multiversioning via Function Overloading (issue5752064)

2012-05-02 Thread Sriraman Tallam
On Wed, May 2, 2012 at 9:05 AM, H.J. Lu hjl.to...@gmail.com wrote: On Wed, May 2, 2012 at 8:08 AM, Sriraman Tallam tmsri...@google.com wrote: On Wed, May 2, 2012 at 6:42 AM, H.J. Lu hjl.to...@gmail.com wrote: On Tue, May 1, 2012 at 7:45 PM, Sriraman Tallam tmsri...@google.com wrote: Hi H.J,  

Re: [PATCH, i386, middle-end, tessuite] Intel TSX's HLE.

2012-05-02 Thread H.J. Lu
On Wed, May 2, 2012 at 8:37 AM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Otherwise, this looks good. Thanks, I've applied inputs! Comitted to MT: http://gcc.gnu.org/ml/gcc-cvs/2012-05/msg00047.html It caused: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53194 -- H.J.

[AVR,4.5,Backport] PR39633, PR45263, PR46779, PR50820

2012-05-02 Thread Georg-Johann Lay
Backported the following wrong-code PRs from 4.6 to 4.5.4: PR target/39633 http://gcc.gnu.org/viewcvs?root=gccview=revrev=187056 PR target/45263 http://gcc.gnu.org/viewcvs?view=revisionrevision=187058 PR target/46779 http://gcc.gnu.org/viewcvs?view=revisionrevision=187055 PR target/50820

Re: User directed Function Multiversioning via Function Overloading (issue5752064)

2012-05-02 Thread H.J. Lu
On Wed, May 2, 2012 at 10:44 AM, Sriraman Tallam tmsri...@google.com wrote: 1.  Since AVX SSE4 SSSE3 SSE3 SSE2 SSE, with foo for AVX and SSE3, on AVX processors, which foo will be selected? foo for AVX will get called since that appears ahead. The dispatching is done in the same order

Re: [RFH / Patch] PR 51222]

2012-05-02 Thread Paolo Carlini
Hi, On 05/01/2012 11:01 PM, Paolo Carlini wrote: Well, not really. The first time walk_template_parms_r is called, t is a SIZEOF_EXPR which remains unhandled, the function returns NULL_TREE. The second time, t is a COMPONENT_REF, thus !TYPE_P (t) is true, TREE_TYPE is NULL, the function

[Patch, testsuite] missing -ftrack-macro-expansion=0 option in gcc.dg/builtin-stringop-chk-1.c

2012-05-02 Thread Greta Yorsh
The test gcc.dg/builtin-stringop-chk-1.c fails on arm-none-eabi because the command line option -ftrack-macro-expansion=0 is missing. This command-line option has recently been added to dg-options directive in this test, but for arm targets the first dg-options directive in the test is

Re: Backported r187026 from branches/google/gcc-4_6 (issue 6148044)

2012-05-02 Thread asharif
On 2012/05/01 22:51:22, jingyu wrote: 1) Please add an description entry to libgcc/ChangeLog.google-4_6 Done. 2) Your gcc/ChangeLog.google-4_6 change reverts someone else's change. Please update it and also update the time of your entry. Done. 3) Please list in the Description field

Re: [patch] Update DWARF codes for Fission

2012-05-02 Thread Cary Coutant
This patch to include/dwarf2.def updates the DW_FORM and DW_AT codes for the Fission extensions. We've eliminated DW_FORM_GNU_ref_index, and replaced DW_AT_GNU_ref_base with DW_AT_GNU_ranges_base. (The wiki page at http://gcc.gnu.org/wiki/DebugFission has been updated.) OK for binutils and

Re: No documentation of -fsched-pressure-algorithm

2012-05-02 Thread Richard Sandiford
Ian Lance Taylor i...@google.com writes: Richard Sandiford rdsandif...@googlemail.com writes: Well, given the replies from you, Ian and Vlad (when reviewing the patch), I feel once again in a minority of one here :-) but... I just don't think we should be advertising this sort of stuff to

Re: PR 53115

2012-05-02 Thread François Dumont
On 05/02/2012 06:23 PM, H.J. Lu wrote: On Tue, May 1, 2012 at 1:23 PM, François Dumontfrs.dum...@gmail.com wrote: unordered_multilmap test added, attached patch applied to 4.7 branch and trunk. This bug was not so difficult to fix. It would even have been quite easy to detect with a good test

[patch] For alpha-vms, unset flag_jump_tables if flag_pic is nonzero

2012-05-02 Thread Steven Bosscher
Hello, alpha-vms is the only target that does not define ASM_OUTPUT_ADDR_DIFF_ELT. That makes the code in stmt.c to handle this case alpha-vms specific. But there is a better way to handle this: Just mimic -fno-jump-tables if flag_pic is nonzero. Tested by building a cross-compiler for

Re: [PING] iwMMXt patches

2012-05-02 Thread Matt Turner
On Tue, Apr 17, 2012 at 4:17 PM, Matt Turner matts...@gmail.com wrote: Are these patches ready to go in? It looks like they were ack'd. http://gcc.gnu.org/ml/gcc-patches/2011-10/msg01815.html http://gcc.gnu.org/ml/gcc-patches/2011-10/msg01817.html

Re: [Patch, libfortran] Fix handling of temporary files

2012-05-02 Thread Janne Blomqvist
PING #2 On Thu, Apr 26, 2012 at 12:19 AM, Janne Blomqvist blomqvist.ja...@gmail.com wrote: PING! On Thu, Apr 19, 2012 at 01:18, Janne Blomqvist blomqvist.ja...@gmail.com wrote: Hi, the attached patch implements some fixes for handling STATUS=SCRATCH files. - Currently, we check

Re: [Patch, fortran] PR 49010/24518 MOD/MODULO fixes, take 2

2012-05-02 Thread Janne Blomqvist
PING #2 On Thu, Apr 26, 2012 at 12:20 AM, Janne Blomqvist blomqvist.ja...@gmail.com wrote: PING! On Thu, Apr 19, 2012 at 00:46, Janne Blomqvist blomqvist.ja...@gmail.com wrote: Hi, the attached patch implements a few fixes and cleanups for the MOD and MODULO intrinsics. - When the

Re: [Patch, fortran] PR 52428 Reading of large negative values and range checking

2012-05-02 Thread Janne Blomqvist
PING On Thu, Apr 26, 2012 at 12:08 AM, Janne Blomqvist blomqvist.ja...@gmail.com wrote: Hi, currently when -frange-check is enabled, we check for overflow when doing a formatted read of an integer value. This check, however, is against the Fortran numerical model (see 13.4 in F2008), which

Re: [patch] skip tpf configure tests

2012-05-02 Thread DJ Delorie
* crossconfig.m4: Since we know that all TPF builds are cross- builds and cannot run configuration-time link tests, do not allow it; just go with known supported linker options. * configure: Regenerate (called as GLIBCXX_CROSSCONFIG). OK Thanks! Committed.

Re: [PATCH] Fix overzealous DSE on sparc

2012-05-02 Thread Richard Sandiford
David Miller da...@davemloft.net writes: On targets such as sparc, where ARG_POINTER_REGNUM == FRAME_POINTER_REGNUM, some of the invariants currently built into DSE simply do not hold. Unlike how DSE assumes, we will in fact see stores to frame pointer relative addresses for setting up

Re: [6/6] Fold prev/next into gimple: do it

2012-05-02 Thread H.J. Lu
On Wed, May 2, 2012 at 10:25 AM, Michael Matz m...@suse.de wrote: Hi, On Wed, 2 May 2012, Richard Guenther wrote: Please remove the flags. Okay. can you factor out this idiom to sth like gsi_new_seq_from_stmt () or so? Or gimple_init_seq_pointers?  Not necessarily exported.

RFC: PATCH to correct the scopes of various types

2012-05-02 Thread Jason Merrill
I've been working on a patch to split out the DWARF debug info for COMDAT functions into COMDAT CUs to go in the same group. One problem I ran into was that modified versions of function-local types (pointers, cv-qualified variants, etc) were ending up at global scope. Separately, Tom Tromey

RFC: PATCH to allow symbolic and sig8 DWARF references in the same compile

2012-05-02 Thread Jason Merrill
Here's my patch to allow -feliminate-dwarf2-dups and -fdebug-types-section to coexist, so that I can then use symbolic references for my function COMDAT work. Does this look reasonable to you? Tested x86_64-pc-linux-gnu. commit fd1868de6f3793901b8ee5bd9d60ce01643a6d78 Author: Jason Merrill

Re: Backported r187026 from branches/google/gcc-4_6 (issue 6148044)

2012-05-02 Thread Jing Yu
LGTM On Wed, May 2, 2012 at 11:24 AM, asha...@chromium.org wrote: On 2012/05/01 22:51:22, jingyu wrote: 1) Please add an description entry to libgcc/ChangeLog.google-4_6 Done. 2) Your gcc/ChangeLog.google-4_6 change reverts someone else's change. Please update it and also update

Re: RFC: PATCH to correct the scopes of various types

2012-05-02 Thread Richard Henderson
On 05/02/2012 01:08 PM, Jason Merrill wrote: * dwarf2out.c (modified_type_die): Use scope_die_for. (gen_type_die_with_usage, dwarf2out_finish): Likewise. (uses_local_type_r, uses_local_type): New. (scope_die_for): Keep a type that uses a local type in local scope.

Re: Fix find_moveable_pseudos, PR52997

2012-05-02 Thread Ulrich Weigand
Bernd Schmidt wrote: On 04/27/2012 06:25 PM, Ulrich Weigand wrote: Bernd Schmidt wrote: We're creating new pseudos, and while we're resizing some data structures, we aren't doing it for everything. @@ -3983,7 +3983,8 @@ find_moveable_pseudos (void) last_moveable_pseudo =

[commit] Enable -fsched-pressure by default on s390

2012-05-02 Thread Ulrich Weigand
Richard Sandiford wrote: Ulrich Weigand uweig...@de.ibm.com writes: Richard Sandiford wrote: Vladimir Makarov vmaka...@redhat.com writes: Taking your results for S390 and ARM with Neon into account, I guess it should be included and probably made by default for these 2 targets (for

Re: RFC: PATCH to allow symbolic and sig8 DWARF references in the same compile

2012-05-02 Thread Cary Coutant
Here's my patch to allow -feliminate-dwarf2-dups and -fdebug-types-section to coexist, so that I can then use symbolic references for my function COMDAT work.  Does this look reasonable to you? Yep, looks good to me. -cary

Re: [patch] For alpha-vms, unset flag_jump_tables if flag_pic is nonzero

2012-05-02 Thread Richard Henderson
On 05/02/2012 12:26 PM, Steven Bosscher wrote: * config/alpha/vms.h (SUBTARGET_OVERRIDE_OPTIONS): For pic code, unset flag_jump_tables. * stmt.c (expand_case): Remove special flag_pic case conditional on ASM_OUTPUT_ADDR_DIFF_ELT not being defined. Ok. I had

[i386] access subvectors

2012-05-02 Thread Marc Glisse
Hello, I definitely don't expect the attached patch to be accepted, but I would like some advice on the direction to go, and a patch that passes the testsuite and does the optimization I want on a couple testcases seems like it may help start the conversation. This is the first time I even

Re: [Patch, testsuite] missing -ftrack-macro-expansion=0 option in gcc.dg/builtin-stringop-chk-1.c

2012-05-02 Thread Gabriel Dos Reis
On Wed, May 2, 2012 at 1:21 PM, Greta Yorsh greta.yo...@arm.com wrote: The test gcc.dg/builtin-stringop-chk-1.c fails on arm-none-eabi because the command line option -ftrack-macro-expansion=0 is missing. This command-line option has recently been added to dg-options directive in this test,

[v3] update outdated allocator docs

2012-05-02 Thread Jonathan Wakely
Several places in the manual still imply mt_allocator is the default or refer to GLIBCXX_FORCE_NEW as affecting the default allocator, which hasn't been true since GCC 3.3! * doc/xml/faq.xml: Update outdated allocator documentation. * doc/xml/manual/allocator.xml: Likewise.

Re: [PATCH] Fix overzealous DSE on sparc

2012-05-02 Thread David Miller
From: Richard Sandiford rdsandif...@googlemail.com Date: Wed, 02 May 2012 20:37:58 +0100 I think the problem is in the way sparc.c:emit_soft_tfmode_libcall reuses existing MEMs when passing arguments on the stack: if (GET_CODE (this_arg) == MEM ! force_stack_temp)

[C++ Patch] PR 53186

2012-05-02 Thread Paolo Carlini
Hi, Vincenzo reported today that the work done by Roberto on devirtualizing final methods, doesn't cover operators (of all sorts). Thus I prepared the below, which passes the testsuite on x86_64-linux. Is it Ok? Thanks, Paolo. / /cp 2012-05-03 Paolo Carlini

Re: [PATCH] Fix overzealous DSE on sparc

2012-05-02 Thread David Miller
From: Richard Sandiford rdsandif...@googlemail.com Date: Wed, 02 May 2012 20:37:58 +0100 I think the DSE assuption is fair though. If you reuse MEMs like this, then they're no longer just serving the purpose described by MEM_EXPR. I think what Sparc does is fair, so if you are going to

RE: [PATCH, reload] Fix bug pr52804, RELOAD pass reloads wrong register on ARM for cortex-m0

2012-05-02 Thread Bin Cheng
-Original Message- From: Ulrich Weigand [mailto:uweig...@de.ibm.com] Sent: Thursday, May 03, 2012 12:05 AM To: Bin Cheng Cc: gcc-patches@gcc.gnu.org Subject: Re: [PATCH, reload] Fix bug pr52804, RELOAD pass reloads wrong register on ARM for cortex-m0 Bin Cheng wrote: In

Restore bootstrap (PR53197)

2012-05-02 Thread Michael Matz
Hi, my gimple_seq reshuffling broke boostrap on some machines. In particular on those for which contrib/compare-debug works, meaning that bootstrap-debug is selected as default BUILD_CONFIG (none of my machines pass the respective configure.ac test, and hence I wasn't seeing the problem

  1   2   >