[patch][RFA] Move the C front end to gcc/c/

2012-06-28 Thread Steven Bosscher
On Thu, Jun 21, 2012 at 4:51 PM, Joseph S. Myers wrote: > On Wed, 20 Jun 2012, Steven Bosscher wrote: > >> I'm posting this as an RFC: Does this look like the right approach? >> Have I overlooked other things than just documentation updates? I hope >> this would not cause too much trouble for bran

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 reaso

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 the

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: [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.

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 technolo

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 eith

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] 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.

[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] 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] 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 pl

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

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 th

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 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 properly?

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

2012-06-28 Thread Steven Bosscher
On Tue, May 22, 2012 at 3:45 AM, rbmj 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 powerpc-*-vxworks. > > A

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

2012-06-28 Thread Alexandre Oliva
On Jun 28, 2012, Mike Stump wrote: > On Jun 28, 2012, at 4:39 AM, Alexandre Oliva 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 it's built? > See your point bel

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

2012-06-28 Thread Lawrence Crowl
On 6/27/12, Lawrence Crowl 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 this patch? BT

Re: [PATCH] Add MULT_HIGHPART_EXPR

2012-06-28 Thread Bernhard Reutner-Fischer
On Thu, Jun 28, 2012 at 04:05:58PM +0200, Jakub Jelinek wrote: >On Thu, Jun 28, 2012@09:17:55AM +0200, Jakub Jelinek wrote: >> I'll look@using MULT_HIGHPART_EXPR in the pattern recognizer and >> vectorizing it as either of the sequences next. > >And here is corresponding pattern recognizer and vect

[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 reca

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

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 wrote: > On 28/06/12 14:38, Mike Stump wrote: >> >> On Jun 28, 2012, at 1:28 AM, Matthew Gretton-Dann > >> wrote: >>> >>> On 27/06/12 21:35, Andrew Pinski wrote: On Wed, Jun 27, 2012 at 3:33 AM, Matthew Gretton-Dann wrote:

[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__lanes on the same lines of the ARM backend. The test case that caused the issue was: v

[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 bootstra

[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 nam

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 >

Re: [PATCH] Add MULT_HIGHPART_EXPR

2012-06-28 Thread Richard Henderson
On 2012-06-28 00:17, Jakub Jelinek wrote: > 2012-06-28 Jakub Jelinek > > PR tree-optimization/53645 > * tree-vect-generic.c (expand_vector_divmod): Use MULT_HIGHPART_EXPR > instead of VEC_WIDEN_MULT_{HI,LO}_EXPR followed by VEC_PERM_EXPR > if possible. > > * gcc.c

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

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 > > gcc/ > * config/aar

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" { xfai

Re: [PATCH] Add MULT_HIGHPART_EXPR

2012-06-28 Thread H.J. Lu
On Thu, Jun 28, 2012 at 8:57 AM, 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 when t

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) compare

[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 Makarov * lra.c (check_rtl): Add arg to insn_invalid_p call. * lra-assigns.c (init_regno_assign_info): Use ira_class_hard_re

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 [mailto:gcc-patch

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: [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: [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: [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 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 like tha

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

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

2012-06-28 Thread Joseph S. Myers
On Wed, 27 Jun 2012, Lawrence Crowl wrote: > >> +Namespaces > >> + > >> + > >> +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 map to nes

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 i686

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 wrote: > On Jun 28, 2012, Jakub Jelinek 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 becau

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 wrote: > On Jun 27, 2012, Mike Stump 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 >>

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 : 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 this code only e

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 > wrote: On 27/06/12 21:35, Andrew Pinski wrote: On Wed, Jun 27, 2012 at 3:33 AM, Matthew Gretton-Dann wrote: All, This patch enables the dump-noaddr test to work in out-of-build-tree testing. [snip]

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 wrote: > On 27/06/12 21:35, Andrew Pinski wrote: >> On Wed, Jun 27, 2012 at 3:33 AM, Matthew Gretton-Dann >> wrote: >>> All, >>> >>> This patch enables the dump-noaddr test to work in out-of-build-tree >>> testing. > [snip] >> >> I created a mu

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 : > 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: [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 Michael Matz Tobias Grosser Sebastian Pop config/ * cloog.m4: Set up to work against ISL only. * isl.m4: New file. * Makefile.def: Add ISL host module, remove

[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 th

[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 (avr_hard_re

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 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 > * configure.ac (CFLAGS_FOR_TARGET, CXXFLAGS_FOR_TARGET): Make sure > they contain -O2. > * configure: Reg

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. >> >>>                 (LTUGEU:SI (reg: CC_REGNUM) (const_int 0]   "TARGET_32BIT" -  "adc%?\\t%0, %1, %2" +  "@ +  adc%?\\t%0, %1, %2 +  sbc%?\\t%

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 wrote: > On Jun 28, 2012, Jakub Jelinek 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 be

Re: [onlinedocs]: No more automatic rebuilt?

2012-06-28 Thread Andreas Schwab
Gerald Pfeifer 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 a/gcc/doc/include/gpl.texi b/gcc/do

[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 PR middle-end/53790 * expr.c (expand_expr_real_1): Verify if the type i

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

2012-06-28 Thread Alexandre Oliva
On Jun 28, 2012, Jakub Jelinek 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 able to run t

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 wrote: > On 28 June 2012 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 >> wrote: >>> >>> On 8 June 2012 10:12, Carrot Wei wrote

Re: [patch] support for multiarch systems

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

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 Matthias Klose
On 28.06.2012 12:01, Thomas Schwinge wrote: > Hi! > > On Mon, 25 Jun 2012 18:19:26 +0200, Matthias Klose > 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, teste

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 firs

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 (

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

2012-06-28 Thread Eric Botcazou
> 2012-06-26 Yannick Moy > > * 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.

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, ). I have been making some progress - I'm attaching below what I have so far in my

Re: [patch] support for multiarch systems

2012-06-28 Thread Thomas Schwinge
Hi! On Mon, 25 Jun 2012 18:19:26 +0200, Matthias Klose 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. > 2

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 > wrote: >> >> On 8 June 2012 10:12, Carrot Wei wrote: >>> Hi >>> >>> In rtl expression, substract a constant c is expressed as

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

2012-06-28 Thread Iain Buclaw
On 27 June 2012 19:17, Mike Stump 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 my part. > > Why

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 wrote: > Hi Ramana > > Thanks for the review, please see my inlined comments. > > On Thu, Jun 28, 2012 at 12:02 AM, Ramana Radhakrishnan > wrote: >> >> On 8 June 2012 10:12, Carrot Wei wrote: >> > Hi >> > >> > In rtl expression, substract a constant c is express

Re: [PATCH] Disable loop2_invariant for -Os

2012-06-28 Thread Richard Guenther
On Thu, Jun 28, 2012 at 10:33 AM, Zhenqiang Chen 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 >>>  gate_rtl_move_loo

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 you

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 wrote: > > On 8 June 2012 10:12, Carrot Wei wrote: > > Hi > > > > In rtl expression, substract a constant c is expressed as add a value -c, > > so it > > is alse processed by

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 testcas

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 > wrote: >> Hi, >> >> In general, inva

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

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 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 attached it for reference. T

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: [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 gpl.tex

[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, b

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 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? > With more cont

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 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.

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 und

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

2012-06-28 Thread Alexandre Oliva
On Jun 27, 2012, Mike Stump 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 linker that supp

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

2012-06-28 Thread Alexandre Oliva
On Jun 27, 2012, Richard Henderson 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 comments befor