Re: [PR49888, VTA] don't keep VALUEs bound to modified MEMs

2012-06-28 Thread Alexandre Oliva
On Jun 27, 2012, Richard Henderson r...@redhat.com wrote: On 06/26/2012 01:54 PM, Alexandre Oliva wrote: + track_stack_pointer (dst, src1, src2); Why does this function return a value then? During testing, I used an assert on the return value to catch cases that couldn't be handled. The

Re: [testsuite] don't use lto plugin if it doesn't work

2012-06-28 Thread Alexandre Oliva
On Jun 27, 2012, Mike Stump mikest...@comcast.net wrote: On Jun 27, 2012, at 2:07 AM, Alexandre Oliva wrote: Why? We don't demand a working plugin. Indeed, we disable the use of the plugin if we find a linker that doesn't support it. We just don't account for the possibility of finding a

Re: [PATCH] Add MULT_HIGHPART_EXPR

2012-06-28 Thread Jakub Jelinek
On Wed, Jun 27, 2012 at 02:37:08PM -0700, Richard Henderson wrote: I was sitting on this patch until I got around to fixing up Jakub's existing vector divmod code to use it. But seeing as how he's adding more uses, I think it's better to get it in earlier. Tested via a patch sent under

Re: [testsuite] don't use lto plugin if it doesn't work

2012-06-28 Thread Jakub Jelinek
On Thu, Jun 28, 2012 at 04:16:55AM -0300, Alexandre Oliva wrote: On Jun 27, 2012, Mike Stump mikest...@comcast.net wrote: On Jun 27, 2012, at 2:07 AM, Alexandre Oliva wrote: Why? We don't demand a working plugin. Indeed, we disable the use of the plugin if we find a linker that doesn't

Re: [PATCH][configure] Make sure CFLAGS_FOR_TARGET And CXXFLAGS_FOR_TARGET contain -O2

2012-06-28 Thread Alexandre Oliva
On Jun 27, 2012, Christophe Lyon christophe.l...@st.com wrote: I looked at the patch in there, and I'm afraid I don't understand how it achieves the ChangeLog-suggested purpose of ensuring -O2 makes to C*FLAGS_FOR_TARGET, when all it appears to do is to prepend -g. Can you please clarify?

[Patch, Fortran] Handle C_F_POINTER with a noncontiguous SHAPE=

2012-06-28 Thread Tobias Burnus
This patch generates inline code for C_F_POINTER with an array argument. One reason is that GCC didn't handle SHAPE= arguments which were noncontiguous. However, the real motivation is the fortran-dev branch with the new array-descriptor: C_F_POINTER needs then to set the stride multiplier,

Re: [onlinedocs]: No more automatic rebuilt?

2012-06-28 Thread Andreas Schwab
libgomp.texi is still using gpl.texi, although libgomp has been relicensed to GPLv3 in 2009. OK? (This is the last use of gpl.texi in the gcc sources. Perhaps it should be removed and gpl_v3.texi renamed back to gpl.texi?) Andreas. * libgomp.texi: Include gpl_v3.texi instead of

Re: [onlinedocs]: No more automatic rebuilt?

2012-06-28 Thread Jakub Jelinek
On Thu, Jun 28, 2012 at 10:18:49AM +0200, Andreas Schwab wrote: libgomp.texi is still using gpl.texi, although libgomp has been relicensed to GPLv3 in 2009. OK? Yes. * libgomp.texi: Include gpl_v3.texi instead of gpl.texi. Jakub

Re: [RFA] Enable dump-noaddr test to work in out of build tree testing

2012-06-28 Thread Matthew Gretton-Dann
On 27/06/12 21:35, Andrew Pinski wrote: On Wed, Jun 27, 2012 at 3:33 AM, Matthew Gretton-Dann matthew.gretton-d...@arm.com wrote: All, This patch enables the dump-noaddr test to work in out-of-build-tree testing. [snip] I created a much simpler patch which I have been meaning to submit. I

RE: [PATCH] Disable loop2_invariant for -Os

2012-06-28 Thread Zhenqiang Chen
diff --git a/gcc/loop-init.c b/gcc/loop-init.c index 03f8f61..5d8cf73 100644 --- a/gcc/loop-init.c +++ b/gcc/loop-init.c @@ -273,6 +273,12 @@ struct rtl_opt_pass pass_rtl_loop_done =  static bool  gate_rtl_move_loop_invariants (void)  { +  /* In general, invariant motion can not reduce

RE: [PATCH] Disable loop2_invariant for -Os

2012-06-28 Thread Zhenqiang Chen
-Original Message- From: Steven Bosscher [mailto:stevenb@gmail.com] Sent: 2012年6月27日 16:54 To: Zhenqiang Chen Cc: gcc-patches@gcc.gnu.org Subject: Re: [PATCH] Disable loop2_invariant for -Os On Wed, Jun 27, 2012 at 10:40 AM, Zhenqiang Chen zhenqiang.c...@arm.com wrote: Hi, In

Re: [PATCH] Move Graphite from using PPL over to ISL

2012-06-28 Thread Tobias Grosser
On 06/27/2012 05:06 PM, Richard Guenther wrote: This merges from the graphite branch the move of PPL to ISL, and completes it where it was lacking - thanks to Micha. It leaves unmerged the addition of a pluto-like ISL optimizer as well as a bugfix for stride 1 which did not come with a

Re: [ARM Patch 1/n] PR53447: optimizations of 64bit ALU operation with constant

2012-06-28 Thread Carrot Wei
Hi Ramana Thanks for the review, please see my inlined comments. On Thu, Jun 28, 2012 at 12:02 AM, Ramana Radhakrishnan ramana.radhakrish...@linaro.org wrote: On 8 June 2012 10:12, Carrot Wei car...@google.com wrote: Hi In rtl expression, substract a constant c is expressed as add a

Re: [PATCH][configure] Make sure CFLAGS_FOR_TARGET And CXXFLAGS_FOR_TARGET contain -O2

2012-06-28 Thread Christophe Lyon
On 28.06.2012 09:32, Alexandre Oliva wrote: I suggest changing both occurrences of $CFLAGS within the case statements, then; the more uniform logic is more appealing to me. Patch approved with these changes. Thanks, Thanks; here is an updated version taking your comment into account. Can

Re: [PATCH] Disable loop2_invariant for -Os

2012-06-28 Thread Richard Guenther
On Thu, Jun 28, 2012 at 10:33 AM, Zhenqiang Chen zhenqiang.c...@arm.com wrote: diff --git a/gcc/loop-init.c b/gcc/loop-init.c index 03f8f61..5d8cf73 100644 --- a/gcc/loop-init.c +++ b/gcc/loop-init.c @@ -273,6 +273,12 @@ struct rtl_opt_pass pass_rtl_loop_done =  static bool  

Re: [ARM Patch 1/n] PR53447: optimizations of 64bit ALU operation with constant

2012-06-28 Thread Ramana Radhakrishnan
On 28 June 2012 10:03, Carrot Wei car...@google.com wrote: Hi Ramana Thanks for the review, please see my inlined comments. On Thu, Jun 28, 2012 at 12:02 AM, Ramana Radhakrishnan ramana.radhakrish...@linaro.org wrote: On 8 June 2012 10:12, Carrot Wei car...@google.com wrote: Hi In

Re: [PATCH, gdc] - Merging gdc (GNU D Compiler) into gcc

2012-06-28 Thread Iain Buclaw
On 27 June 2012 19:17, Mike Stump mikest...@comcast.net wrote: On Jun 27, 2012, at 7:45 AM, Iain Buclaw wrote: I do have a question though, what is available for the transition of development from git to svn?  Other than a lot of ready and getting used to the various switches and commands on

Re: [ARM Patch 1/n] PR53447: optimizations of 64bit ALU operation with constant

2012-06-28 Thread Richard Earnshaw
On 28/06/12 10:03, Carrot Wei wrote: Hi Ramana Thanks for the review, please see my inlined comments. On Thu, Jun 28, 2012 at 12:02 AM, Ramana Radhakrishnan ramana.radhakrish...@linaro.org wrote: On 8 June 2012 10:12, Carrot Wei car...@google.com wrote: Hi In rtl expression, substract

Re: [patch] support for multiarch systems

2012-06-28 Thread Thomas Schwinge
Hi! On Mon, 25 Jun 2012 18:19:26 +0200, Matthias Klose d...@ubuntu.com wrote: On 25.06.2012 15:56, Joseph S. Myers wrote: On Mon, 25 Jun 2012, Matthias Klose wrote: Please find attached the patch updated for trunk 20120625, x86 only, tested on x86-linux-gnu, KFreeBSD and the Hurd.

Re: [C++ RFC / Patch] PR 51213 (access control under SFINAE)

2012-06-28 Thread Paolo Carlini
On 06/15/2012 04:27 PM, Paolo Carlini wrote: Hi, as I mentioned a few days ago, I'm working on implementing this feature, which I personally consider rather high priority, from the library point of view too (eg, type_traits). I have been making some progress - I'm attaching below what I have

Re: [Ada] Attribute 'Old should only be used in postconditions

2012-06-28 Thread Eric Botcazou
2012-06-26 Yannick Moy m...@adacore.com * sem_attr.adb (Analyze_Attribute): Detect if 'Old is used outside a postcondition, and issue an error in such a case. This has introduced the following failures in the gnat.dg testsuite: FAIL: gnat.dg/deep_old.adb (test for excess

Re: [Ada] Attribute 'Old should only be used in postconditions

2012-06-28 Thread Arnaud Charlet
* sem_attr.adb (Analyze_Attribute): Detect if 'Old is used outside a postcondition, and issue an error in such a case. This has introduced the following failures in the gnat.dg testsuite: FAIL: gnat.dg/deep_old.adb (test for excess errors) FAIL: gnat.dg/old_errors.adb (test for

Re: [PATCH] Add generic vector lowering for integer division and modulus (PR tree-optimization/53645)

2012-06-28 Thread Richard Guenther
On Wed, 27 Jun 2012, Jakub Jelinek wrote: Hi! This patch makes veclower2 attempt to emit integer division/modulus of vectors by constants using vector multiplication, shifts or masking. It is somewhat similar to the vect_recog_divmod_pattern, but it needs to analyze everything first, see

Re: [patch] support for multiarch systems

2012-06-28 Thread Matthias Klose
On 28.06.2012 12:01, Thomas Schwinge wrote: Hi! On Mon, 25 Jun 2012 18:19:26 +0200, Matthias Klose d...@ubuntu.com wrote: On 25.06.2012 15:56, Joseph S. Myers wrote: On Mon, 25 Jun 2012, Matthias Klose wrote: Please find attached the patch updated for trunk 20120625, x86 only, tested on

Re: [onlinedocs]: No more automatic rebuilt?

2012-06-28 Thread Gerald Pfeifer
On Thu, 28 Jun 2012, Andreas Schwab wrote: libgomp.texi is still using gpl.texi, although libgomp has been relicensed to GPLv3 in 2009. OK? Looks good, thank you. (This is the last use of gpl.texi in the gcc sources. Perhaps it should be removed and gpl_v3.texi renamed back to gpl.texi?)

Re: [patch] support for multiarch systems

2012-06-28 Thread Thomas Schwinge
Hi! On Thu, 28 Jun 2012 12:42:23 +0200, Matthias Klose d...@ubuntu.com wrote: On 28.06.2012 12:01, Thomas Schwinge wrote: On Mon, 25 Jun 2012 18:19:26 +0200, Matthias Klose d...@ubuntu.com wrote: On 25.06.2012 15:56, Joseph S. Myers wrote: On Mon, 25 Jun 2012, Matthias Klose wrote:

Re: [ARM Patch 1/n] PR53447: optimizations of 64bit ALU operation with constant

2012-06-28 Thread Carrot Wei
On Thu, Jun 28, 2012 at 5:37 PM, Ramana Radhakrishnan ramana.radhakrish...@linaro.org wrote: On 28 June 2012 10:03, Carrot Wei car...@google.com wrote: Hi Ramana Thanks for the review, please see my inlined comments. On Thu, Jun 28, 2012 at 12:02 AM, Ramana Radhakrishnan

Re: [testsuite] don't use lto plugin if it doesn't work

2012-06-28 Thread Alexandre Oliva
On Jun 28, 2012, Jakub Jelinek ja...@redhat.com wrote: On Thu, Jun 28, 2012 at 04:16:55AM -0300, Alexandre Oliva wrote: I'd very be surprised if I asked for an i686 native build to package and install elsewhere, and didn't get a plugin just because the build-time linker wouldn't have been

[PATCH] Fix PR53790

2012-06-28 Thread Richard Guenther
This fixes PR53790 - with MEM_REF you can get base decls of incomplete type. Deal with that. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied everywhere. Richard. 2012-06-28 Richard Guenther rguent...@suse.de PR middle-end/53790 * expr.c (expand_expr_real_1):

Re: [onlinedocs]: No more automatic rebuilt?

2012-06-28 Thread Andreas Schwab
Gerald Pfeifer ger...@pfeifer.com writes: If it's not used any more, yes, please go ahead an remove it. Done as this, tested with make info. Andreas. * doc/include/gpl.texi: Remove. * doc/sourcebuild.texi (Texinfo Manuals): Don't mention gpl.texi. diff --git

Re: [testsuite] don't use lto plugin if it doesn't work

2012-06-28 Thread Richard Guenther
On Thu, Jun 28, 2012 at 1:39 PM, Alexandre Oliva aol...@redhat.com wrote: On Jun 28, 2012, Jakub Jelinek ja...@redhat.com wrote: On Thu, Jun 28, 2012 at 04:16:55AM -0300, Alexandre Oliva wrote: I'd very be surprised if I asked for an i686 native build to package and install elsewhere, and

Re: [ARM Patch 1/n] PR53447: optimizations of 64bit ALU operation with constant

2012-06-28 Thread Ramana Radhakrishnan
 subs -lo ; sbc ~hi - lower negative, upper negative  subs -lo ; adc hi  - lower negative, upper positive Yes. snip                 (LTUGEU:SI (reg:cnb CC_REGNUM) (const_int 0]   TARGET_32BIT -  adc%?\\t%0, %1, %2 +  @ +  adc%?\\t%0, %1, %2 +  sbc%?\\t%0, %1, %#n2 Since

Re: [PATCH][configure] Make sure CFLAGS_FOR_TARGET And CXXFLAGS_FOR_TARGET contain -O2

2012-06-28 Thread Alexandre Oliva
On Jun 28, 2012, Christophe Lyon christophe.l...@st.com wrote: Can you commit it for me (I don't have write access). Done, GCC SVN and src CVS trees. Thanks! 2012-06-28 Christophe Lyon christophe.l...@st.com * configure.ac (CFLAGS_FOR_TARGET, CXXFLAGS_FOR_TARGET): Make sure

[patch]: Fix PR53595 (hard_regno_call_part_clobbered called with invalid regno)

2012-06-28 Thread Georg-Johann Lay
This patch returns false in HARD_REGNO_CALL_PART_CLOBBERED if !HARD_REGNO_MODE_OK. Returning true for such registers might lead to performance degradation that eat up all performance gained from 4.6 to 4.7 for example. Ok to apply? Johann PR 53595 * config/avr/avr.c

[PATCH] MIPS/libgcc: Add soft-fp support for SDE bare-iron targets

2012-06-28 Thread Maciej W. Rozycki
Hello, This change adds soft-fp support for SDE bare-iron targets. The settings have been mostly based on the version already present in glibc, except that the ABI variations have been merged into a single file and conditionalised on preprocessor macros (and the file reformatted to follow

Re: [PATCH] Move Graphite from using PPL over to ISL

2012-06-28 Thread Diego Novillo
On 12-06-27 11:06 , Richard Guenther wrote: 2012-06-27 Richard Guenther rguent...@suse.de Michael Matz m...@suse.de Tobias Grosser tob...@grosser.es Sebastian Pop seb...@gmail.com config/ * cloog.m4: Set up to work against ISL only. * isl.m4:

Re: [patch]: Fix PR53595 (hard_regno_call_part_clobbered called with invalid regno)

2012-06-28 Thread Denis Chertykov
2012/6/28 Georg-Johann Lay a...@gjlay.de: This patch returns false in HARD_REGNO_CALL_PART_CLOBBERED if !HARD_REGNO_MODE_OK. Returning true for such registers might lead to performance degradation that eat up all performance gained from 4.6 to 4.7 for example. Ok to apply? Johann      

Re: [RFA] Enable dump-noaddr test to work in out of build tree testing

2012-06-28 Thread Mike Stump
On Jun 28, 2012, at 1:28 AM, Matthew Gretton-Dann matthew.gretton-d...@arm.com wrote: On 27/06/12 21:35, Andrew Pinski wrote: On Wed, Jun 27, 2012 at 3:33 AM, Matthew Gretton-Dann matthew.gretton-d...@arm.com wrote: All, This patch enables the dump-noaddr test to work in out-of-build-tree

Re: [RFA] Enable dump-noaddr test to work in out of build tree testing

2012-06-28 Thread Matthew Gretton-Dann
On 28/06/12 14:38, Mike Stump wrote: On Jun 28, 2012, at 1:28 AM, Matthew Gretton-Dann matthew.gretton-d...@arm.com wrote: On 27/06/12 21:35, Andrew Pinski wrote: On Wed, Jun 27, 2012 at 3:33 AM, Matthew Gretton-Dann matthew.gretton-d...@arm.com wrote: All, This patch enables the

Re: [PATCH] MIPS/libgcc: Add soft-fp support for SDE bare-iron targets

2012-06-28 Thread Joseph S. Myers
On Thu, 28 Jun 2012, Maciej W. Rozycki wrote: * config/mips/sfp-machine.h: New file. * config.host mips*-sde-elf*: Enable soft-fp. The compiler uses MIPS NaN conventions on MIPS; fp-bit knows about those but soft-fp does not. Are you not concerned about that regression? (Is

Re: [testsuite] don't use lto plugin if it doesn't work

2012-06-28 Thread Mike Stump
On Jun 28, 2012, at 12:16 AM, Alexandre Oliva aol...@redhat.com wrote: On Jun 27, 2012, Mike Stump mikest...@comcast.net wrote: On Jun 27, 2012, at 2:07 AM, Alexandre Oliva wrote: Why? We don't demand a working plugin. Indeed, we disable the use of the plugin if we find a linker that doesn't

Re: [testsuite] don't use lto plugin if it doesn't work

2012-06-28 Thread Mike Stump
On Jun 28, 2012, at 4:39 AM, Alexandre Oliva aol...@redhat.com wrote: On Jun 28, 2012, Jakub Jelinek ja...@redhat.com wrote: On Thu, Jun 28, 2012 at 04:16:55AM -0300, Alexandre Oliva wrote: I'd very be surprised if I asked for an i686 native build to package and install elsewhere, and didn't

Re: [PATCH] Add MULT_HIGHPART_EXPR

2012-06-28 Thread Jakub Jelinek
On Thu, Jun 28, 2012 at 09:17:55AM +0200, Jakub Jelinek wrote: I'll look at using MULT_HIGHPART_EXPR in the pattern recognizer and vectorizing it as either of the sequences next. And here is corresponding pattern recognizer and vectorizer patch. Bootstrapped/regtested on x86_64-linux and

Re: [wwwdocs] Update coding conventions for C++

2012-06-28 Thread Joseph S. Myers
On Wed, 27 Jun 2012, Lawrence Crowl wrote: +h4a name=Namespace_UseNamespaces/a/h4 + +p +Namespaces are encouraged. +All separable libraries should have a unique global namespace. +All individual tools should have a unique global namespace. +Nested include directories names should

Re: [testsuite] don't use lto plugin if it doesn't work

2012-06-28 Thread Jakub Jelinek
On Thu, Jun 28, 2012 at 07:03:37AM -0700, Mike Stump wrote: Also, this scenario of silently deciding whether or not to use the linker plugin could bring us to different test results for the same command lines. I don't like that. Right, which is why the static configuration of the host

Re: [testsuite] don't use lto plugin if it doesn't work

2012-06-28 Thread Richard Guenther
On Thu, Jun 28, 2012 at 4:08 PM, Jakub Jelinek ja...@redhat.com wrote: On Thu, Jun 28, 2012 at 07:03:37AM -0700, Mike Stump wrote: Also, this scenario of silently deciding whether or not to use the linker plugin could bring us to different test results for the same command lines.  I don't

Re: [Ada] Attribute 'Old should only be used in postconditions

2012-06-28 Thread Eric Botcazou
Probably suppress both, since they no longer make sense (they are testing an early implementation of 'Old, before 'Old was standardized in Ada 2012). I'll take care of it. Thanks! -- Eric Botcazou

Re: [Ada] Attribute 'Old should only be used in postconditions

2012-06-28 Thread Arnaud Charlet
Probably suppress both, since they no longer make sense (they are testing an early implementation of 'Old, before 'Old was standardized in Ada 2012). I'll take care of it. Thanks! Sure, done for the record (revision 189042).

Re: [PATCH] Add MULT_HIGHPART_EXPR

2012-06-28 Thread Richard Henderson
On 2012-06-28 07:05, Jakub Jelinek wrote: Unfortunately the addition of the builtin_mul_widen_* hooks on i?86 seems to pessimize the generated code for gcc.dg/vect/pr51581-3.c testcase (at least with -O3 -mavx) compared to when the hooks aren't present, because i?86 has more natural support

Re: [PATCH, GCC][AArch64] Use Enums for code models option selection

2012-06-28 Thread Tejas Belagod
Tejas Belagod wrote: Marcus Shawcroft wrote: On 13/06/12 14:38, Sofiane Naci wrote: Hi, I discovered a bug in my previous patch, so I attach a new one. The ChangeLog hasn't changed. OK to commit? Thanks Sofiane -Original Message- From: gcc-patches-ow...@gcc.gnu.org

[lra] trunk merged into the branch

2012-06-28 Thread Vladimir Makarov
I merged trunk at 188913 into lra branch. Some changes were required to make lra branch bootstrapped on x86/x86-64 and ppc. 2012-06-23 Vladimir Makarovvmaka...@redhat.com * lra.c (check_rtl): Add arg to insn_invalid_p call. * lra-assigns.c (init_regno_assign_info): Use

Re: [PATCH] Add MULT_HIGHPART_EXPR

2012-06-28 Thread Jakub Jelinek
On Thu, Jun 28, 2012 at 08:57:23AM -0700, Richard Henderson wrote: On 2012-06-28 07:05, Jakub Jelinek wrote: Unfortunately the addition of the builtin_mul_widen_* hooks on i?86 seems to pessimize the generated code for gcc.dg/vect/pr51581-3.c testcase (at least with -O3 -mavx) compared to

Re: [PATCH] Add MULT_HIGHPART_EXPR

2012-06-28 Thread H.J. Lu
On Thu, Jun 28, 2012 at 8:57 AM, Richard Henderson r...@redhat.com wrote: On 2012-06-28 07:05, Jakub Jelinek wrote: Unfortunately the addition of the builtin_mul_widen_* hooks on i?86 seems to pessimize the generated code for gcc.dg/vect/pr51581-3.c testcase (at least with -O3 -mavx) compared

Re: [testsuite] gcc.dg/vect/vect-50.c: combine two scans

2012-06-28 Thread Janis Johnson
On 06/27/2012 05:05 PM, Mike Stump wrote: On Jun 27, 2012, at 3:36 PM, Janis Johnson wrote: These scans from gcc.dg/vect/vect-50.c, and others similar to them in other vect tests, hurt my brain: /* { dg-final { scan-tree-dump-times Vectorizing an unaligned access 2 vect { xfail {

Re: [PATCH, GCC][AArch64] Use Enums for code models option selection

2012-06-28 Thread Richard Earnshaw
On 28/06/12 16:58, Tejas Belagod wrote: Sorry, I broke the build when I applied this patch. Attached is a patch that fixes this. Build and regressions are happy. OK to commit? Thanks, Tejas Belagod. ARM. Changelog 2012-06-28 Tejas Belagod tejas.bela...@arm.com gcc/ *

Re: [PATCH] Add MULT_HIGHPART_EXPR

2012-06-28 Thread Richard Henderson
On 2012-06-28 09:20, Jakub Jelinek wrote: Perhaps the problem is then that the permutation is much more expensive for even/odd. With even/odd the f2 routine is: ... vpshufb %xmm2, %xmm5, %xmm5 vpshufb %xmm1, %xmm4, %xmm4 vpor%xmm4, %xmm5, %xmm4 ... and with lo/hi

Re: [PATCH] Add MULT_HIGHPART_EXPR

2012-06-28 Thread Richard Henderson
On 2012-06-28 07:05, Jakub Jelinek wrote: PR tree-optimization/51581 * tree-vect-stmts.c (permute_vec_elements): Add forward decl. (vectorizable_operation): Handle vectorization of MULT_HIGHPART_EXPR also using VEC_WIDEN_MULT_*_EXPR or builtin_mul_widen_* plus

[C++ Pubnames Patch] Anonymous namespaces enclosed in named namespaces. (issue6343052)

2012-06-28 Thread Sterling Augustine
The enclosed patch adds a fix for the pubnames anonymous namespaces contained within named namespaces, and adds an extensive test for the various pubnames. The bug is that when printing at verbosity level 1, and lang_decl_name sees a namespace decl in not in the global namespace, it prints the

[lra] a patch to fix last testsuite regression on x86/x86-64

2012-06-28 Thread Vladimir Makarov
The following patch fixes last GCC testsuite regression (in comparison with reload) on x86/x86-64 after last merge of trunk into lra. The patch actually implements recent Bernd's optimization (restoring an argument pseudo value from the call result) in LRA. The patch was successfully

[PATCH][RFC, Reload]. Reload bug?

2012-06-28 Thread Tejas Belagod
Hi, Attached is a fix for what seems to be a reload bug while handling subreg(mem...). I ran into this problem while implementing support for struct load/store in AArch64 using the standard patterns vec_loadstore_laneslarge_int_modevec_mode on the same lines of the ARM backend. The test

Re: [RFA] Enable dump-noaddr test to work in out of build tree testing

2012-06-28 Thread Andrew Pinski
On Thu, Jun 28, 2012 at 6:50 AM, Matthew Gretton-Dann matthew.gretton-d...@arm.com wrote: On 28/06/12 14:38, Mike Stump wrote: On Jun 28, 2012, at 1:28 AM, Matthew Gretton-Dann matthew.gretton-d...@arm.com wrote: On 27/06/12 21:35, Andrew Pinski wrote: On Wed, Jun 27, 2012 at 3:33 AM,

Re: [RFA] Enable dump-noaddr test to work in out of build tree testing

2012-06-28 Thread Mike Stump
On Jun 28, 2012, at 11:42 AM, Andrew Pinski wrote: Both will work in the case of build==host. I don't even know if we really support build!=host testing at all. Sure... works just fine, last I knew. Generally easy enough to fixup, if people get it wrong. I have never seen it done and I

[PATCH] Fix PR46556 (straight-line strength reduction, part 2)

2012-06-28 Thread William J. Schmidt
Here's a relatively small piece of strength reduction that solves that pesky addressing bug that got me looking at this in the first place... The main part of the code is the stuff that was reviewed last year, but which needed to find a good home. So hopefully that's in pretty good shape. I

Re: [wwwdocs] Update coding conventions for C++

2012-06-28 Thread Lawrence Crowl
On 6/27/12, Lawrence Crowl cr...@google.com wrote: ..., does anyone object to removing the permission to use C++ streams? Having heard no objection, I removed the permission. The following patch is the current state of the changes. Since the discussion appears to have died down, can I commit

Re: [testsuite] don't use lto plugin if it doesn't work

2012-06-28 Thread Alexandre Oliva
On Jun 28, 2012, Mike Stump mikest...@comcast.net wrote: On Jun 28, 2012, at 4:39 AM, Alexandre Oliva aol...@redhat.com wrote: That still doesn't sound right to me: why should the compiler refrain from using a perfectly functional linker plugin on the machine where it's installed (not where

Re: [Patch, libgfortran] Add FPU Support for powerpc

2012-06-28 Thread Steven Bosscher
On Tue, May 22, 2012 at 3:45 AM, rbmj r...@verizon.net wrote: Hi everyone, This patch adds FPU support for powerpc on platforms that do not have glibc.  It is basically the same code as glibc has.  The motivation for this was that right now there is no fpu-target.h that works for

Re: [PATCH] gfortran testsuite: implicitly cleanup-modules

2012-06-28 Thread Bernhard Reutner-Fischer
Rehi Janis, Good to see you active again :) Perhaps you want to pursue this? We'd need to suggest this to dejagnu, have it in a release and bump the minimum required deja version of gcc. So it may take time but IMO would be a worthwhile cleanup. Or do you see a better way to handle this

Fwd: [Bug debug/53754] [4.8 Regression][lto] ICE in lhd_decl_printable_name, at langhooks.c:222 (with -g)

2012-06-28 Thread Cary Coutant
[resending in plain text. Sorry, gmail defaulted to HTML.] Ping. I'm not looking for commit approval yet, just advice on how thorough we need to be to support -g and LTO together. (What's the right way to send a patch to fix a PR? I'm not even sure whether you were cc'ed on my response.) -cary

Re: [PATCH] gfortran testsuite: implicitly cleanup-modules

2012-06-28 Thread Mike Stump
On Jun 28, 2012, at 3:27 PM, Bernhard Reutner-Fischer wrote: Perhaps you want to pursue this? We'd need to suggest this to dejagnu, Actually, we have the technology, so that isn't necessary. :-) You can install replacements for any procs you want, not pretty, but... it does work. I think

Re: [PATCH] gfortran testsuite: implicitly cleanup-modules

2012-06-28 Thread Bernhard Reutner-Fischer
On Thu, Jun 28, 2012 at 04:43:05PM -0700, Mike Stump wrote: On Jun 28, 2012, at 3:27 PM, Bernhard Reutner-Fischer wrote: Perhaps you want to pursue this? We'd need to suggest this to dejagnu, Actually, we have the technology, so that isn't necessary. :-) You can install replacements for any

[testsuite] gcc.dg/Wstrict-aliasing-converted-assigned.c: fix dg-message errors

2012-06-28 Thread Janis Johnson
Test gcc.dg/Wstrict-aliasing-converted-assigned.c uses a combination of target and xfail selectors in a way that would be nice if it worked, but it doesn't. Unfortunately the local code to override dg-error and friends ignores errors, so directives with errors have been silently skipped. I plan

[testsuite] add required comments to dg-message directives in g++.dg

2012-06-28 Thread Janis Johnson
Several tests in g++.dg use dg-message with a target list and line number but without the comment field, which is required when those additional arguments are used. The local replacement of dg-message silently ignores errors (something I plan to fix), so the checks have been ignored. Unprocessed

[testsuite] g++.dg/cpp0x/nullptr19.c: remove duplicate dg-message

2012-06-28 Thread Janis Johnson
Test g++.dg/cpp0x/nullptr19.c contains the following: char* k( char* ); /* { dg-message note } { dg-message note } */ nullptr_t k( nullptr_t ); /* { dg-message note } { dg-message note } */ Having two test directives on a line should have resulted in an ERROR but the local

Re: [testsuite] g++.dg/cpp0x/nullptr19.c: remove duplicate dg-message

2012-06-28 Thread Mike Stump
On Jun 28, 2012, at 5:57 PM, Janis Johnson wrote: Test g++.dg/cpp0x/nullptr19.c contains the following: OK for trunk? Ok.

Re: [testsuite] add required comments to dg-message directives in g++.dg

2012-06-28 Thread Mike Stump
On Jun 28, 2012, at 5:56 PM, Janis Johnson wrote: Several tests in g++.dg use dg-message with a target list and line number but without the comment field, which is required when those additional arguments are used. OK for trunk? Ok.

Re: [testsuite] gcc.dg/Wstrict-aliasing-converted-assigned.c: fix dg-message errors

2012-06-28 Thread Mike Stump
On Jun 28, 2012, at 5:55 PM, Janis Johnson wrote: Test gcc.dg/Wstrict-aliasing-converted-assigned.c uses a combination of target and xfail selectors in a way that would be nice if it worked, OK for trunk? Ok. I prefer no spacing between the comment and the dg-message lines... ok either

Re: [PATCH] gfortran testsuite: implicitly cleanup-modules

2012-06-28 Thread Mike Stump
On Jun 28, 2012, at 5:15 PM, Bernhard Reutner-Fischer wrote: On Thu, Jun 28, 2012 at 04:43:05PM -0700, Mike Stump wrote: On Jun 28, 2012, at 3:27 PM, Bernhard Reutner-Fischer wrote: Perhaps you want to pursue this? We'd need to suggest this to dejagnu, Actually, we have the technology, so

Re: [testsuite] gcc.dg/vect/vect-50.c: combine two scans

2012-06-28 Thread Mike Stump
On Jun 28, 2012, at 10:26 AM, Janis Johnson wrote: No, there is no way to combine target and xfail, Ah... Grrr I hate non-composability. Given that, I think the original patch is fine, subject of course to the wants and wishes of vect people.

Ping: Reorganized documentation for warnings -- attempt 2

2012-06-28 Thread David Stone
http://gcc.gnu.org/ml/gcc-patches/2012-06/msg01208.html

Re: New option to turn off stack reuse for temporaries

2012-06-28 Thread Xinliang David Li
(re-post in plain text) Moving this to cfgexpand time is simple and it can also be extended to handle scoped variables. However Jakub raised a good point about this being too late as stack space overlay is not the only way to cause trouble when the lifetime of a stack object is extended beyond

Re: [PATCH] Add MULT_HIGHPART_EXPR

2012-06-28 Thread Jakub Jelinek
On Fri, Jun 29, 2012 at 12:00:10AM +0200, Bernhard Reutner-Fischer wrote: Really both HI? If so optab2 could be removed from that fn altogether.. Of course, thanks for pointing that out. I've additionally added a result mode check (similar to what supportable_widening_operation does). The