Re: [Patch, libgfortran] Pass mode in "open" for O_CREAT and on VxWorks

2012-05-15 Thread Janne Blomqvist
On Tue, May 15, 2012 at 12:03 PM, Tobias Burnus wrote: > Dear all, > > the motivation for the following is rbmj's patch for libstdc++ on VxWorks, > cf. http://gcc.gnu.org/ml/gcc-patches/2012-05/msg00073.html > > Note that gfortan is used on VxWorks as the following email proves: > http://gcc.gnu.o

Re: ping: Use "sed -n …" instead of "sed s/…/p -e d" in s-header-vars

2012-05-15 Thread Paolo Bonzini
Il 15/05/2012 16:18, Olivier Hainque ha scritto: > rm -f tmp-header-vars > ! # The first sed gets the list "header variables" as the list variables > ! # assigned in Makefile and having _H at the end of the name. "sed -n" > proved > ! # more portable than a trailing "-e d" to filter out the

Re: patch for PR53125

2012-05-15 Thread Paolo Bonzini
> Now with patch > > On Fri, May 11, 2012 at 8:42 PM, Steven Bosscher > wrote: > 2012-05-10 Vladimir Makarov > > PR rtl-optimization/53125 > * ira.c (ira): Call find_moveable_pseudos and > move_unallocated_pseudos if only ira_conflicts_p is true. >

ping: Fix gcc.dg/lower-subreg-1.c failure (was: [C Patch]: pr52543)

2012-05-15 Thread Hans-Peter Nilsson
> From: Hans-Peter Nilsson > Date: Wed, 9 May 2012 08:02:25 +0200 Ping. I missed the PR number decoration on the ChangeLog entry: PR rtl-optimization/53176 > * rtlanal.c (rtx_cost): Adjust default cost for X with a > UNITS_PER_WORD factor for all X according to the size of >

libgo patch committed: Fix handling of LineSetFile in DWARF line info

2012-05-15 Thread Ian Lance Taylor
This libgo patch fixes the handling of LineSetFile aka DW_LNS_set_file in DWARF line info. The DWARF number is one-based, but I am using it to index into a zero-based array. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline and 4.7 branch. Ian diff -r dd8c9c3

Re: Make vectoriser use operand_equal_p to compare base addresses

2012-05-15 Thread H.J. Lu
On Tue, Apr 19, 2011 at 7:59 AM, Richard Sandiford wrote: > In the attached testcase, we treat: > >    sum += x[i][0] * x[i][0] + x[i][1] * x[i][1]; > > as being two independent strided loads: x[i][0] and x[i][1]. > On targets with appropriate support, we therefore use two interleaved > loads rath

Re: [C++ Patch] PR 44516

2012-05-15 Thread Jason Merrill
On 05/15/2012 07:56 PM, Paolo Carlini wrote: But, speaking of incremental work: what if, post the build_min_nt_loc chunk, we handle build_min_non_dep and build_min in a case by case way? Thus we keep around the non-_loc variant and gradually replace each call? With testcases (small!) checking col

PING: [PATCH] Fix PR53217

2012-05-15 Thread William J. Schmidt
Ping. Thanks, Bill On Tue, 2012-05-08 at 22:04 -0500, William J. Schmidt wrote: > This fixes another statement-placement issue when reassociating > expressions with repeated factors. Multiplies feeding into > __builtin_powi calls were not getting placed properly ahead of them in > some cases. >

Re: rs6000 toc reference rtl again

2012-05-15 Thread Alan Modra
On Tue, May 01, 2012 at 01:47:51PM +0930, Alan Modra wrote: > This revision splits the medium/large code model toc reference after > reload. Ping http://gcc.gnu.org/ml/gcc-patches/2012-05/msg6.html -- Alan Modra Australia Development Lab, IBM

Re: gnu-tm: Dont allow assigning transaction_unsafe functions to transaction_safe function pointers (issue6198054)

2012-05-15 Thread Dave Boutcher
On Tue, May 15, 2012 at 4:16 PM, Patrick Marlier wrote: > Follow-up of Dave's patch. I would prefer to see such checks in trans-mem.c > as follows. > In a transaction, a function pointer can be declared and assigned but there > is no check that the function pointer is transaction_safe. So at runti

Re: gnu-tm: Dont allow assigning transaction_unsafe functions to transaction_safe function pointers (issue6198054)

2012-05-15 Thread Dave Boutcher
On Tue, May 15, 2012 at 11:23 AM, Torvald Riegel wrote: > > On Tue, 2012-05-08 at 18:02 -0500, Dave Boutcher wrote: > > Without this patch it is perfectly fine to assign non-transaction_safe > > functions to function pointers marked as transaction_safe. Unpleasantness > > happens at run time. > >

[PATCH] Fix var tracking ICE due to reorg.

2012-05-15 Thread David Miller
How bugs like this were not hit earlier, I'll never understand :-) If during reorg we delete a code label, and as a result we decide to delete all the code following that label, we hit this condition in jump.c:delete_related_insns(): if (was_code_label && prev && BARRIER_P (prev))

Re: remove TARGET_E500 and factorize SPE defaults computation in powerpc ports

2012-05-15 Thread David Edelsohn
On Tue, May 15, 2012 at 9:47 AM, Olivier Hainque wrote: >        config/rs6000: > >        * rs6000-opts.h (enum processor_type): Add PROCESSOR_PPC8548. >        * rs6000-cpus.def: Reference it for cpu="8548". >        * rs6000.md (cpu attribute definition): Add ppc8548. >        * 8540.md: indic

[google/gcc-4_6_3-mobile] Port r187569 from google/gcc-4_6 branch (issue 6210060)

2012-05-15 Thread shenhan
Reviewers: asharif1, jingyu, Message: Hi Amhad and Jing, could you take a look at this CL at http://codereview.appspot.com/6210060/ Thanks, -Han Description: Ported r187569 from google/gcc-4_6 branch. 2012-05-11 Han Shen Backport from mainline. 2012-05-01 Richard Earnshaw

Another merge from gcc-4_7-branch to gccgo branch

2012-05-15 Thread Ian Lance Taylor
And I've committed yet another merge from gcc-4_7-branch to gccgo branch, this time merging revision 187575. Ian

Go patch committed: Fix names for types defined in functions

2012-05-15 Thread Ian Lance Taylor
This patch to the Go frontend fixes the names used for types defined in functions. Without this patch the names would use whatever appeared in the -fgo-pkgpath or -fgo-prefix strings, which were not required to be valid assembler identifier characters. There is no need to use the pkgpath/prefix s

Re: [C++ Patch] PR 44516

2012-05-15 Thread Paolo Carlini
On 05/16/2012 01:56 AM, Paolo Carlini wrote: But, speaking of incremental work: what if, post the build_min_nt_loc chunk, we handle build_min_non_dep and build_min in a case by case way? Thus we keep around the non-_loc variant and gradually replace each call? With testcases (small!) checking c

[RFA:] doc: TARGET_LEGITIMIZE_ADDRESS needs to be defined for native TLS

2012-05-15 Thread Hans-Peter Nilsson
An old patch I finally came around to submit. Verified that the DVI and info output looks ok. Ok to commit with inherent relicensing and whatever? gcc: * doc/tm.texi.in (Addressing Modes) : Mention that this hook needs to be defined for native TLS. * doc/tm.texi: Regenerat

Re: [C++ Patch] PR 44516

2012-05-15 Thread Paolo Carlini
Hi, On 05/16/2012 01:36 AM, Jason Merrill wrote: On 05/15/2012 06:44 PM, Paolo Carlini wrote: I can't change the build_min_non_dep_loc call in build_x_binary_op to simply use loc, that causes the diagnostic regression I mentioned before. Sorry, I wasn't clear in my previous message, it happens

Committed: fix source-code reference in SIZE_TYPE docs (was: How do I set SIG_ATOMIC_TYPE to a variant of a C-type?)

2012-05-15 Thread Hans-Peter Nilsson
(Originally a post to gcc@. I don't know why I bother posting there; it usually ends up with a patch. :) > From: Hans-Peter Nilsson > Date: Tue, 15 May 2012 23:44:49 +0200 > > From: "Joseph S. Myers" > > Date: Tue, 15 May 2012 22:06:03 +0200 > > > And as you have discovered, there are rules fo

Re: [Patch / RFC] PR 39681

2012-05-15 Thread Jason Merrill
OK. Jason

Re: [C++ Patch] PR 44516

2012-05-15 Thread Jason Merrill
On 05/15/2012 06:44 PM, Paolo Carlini wrote: I can't change the build_min_non_dep_loc call in build_x_binary_op to simply use loc, that causes the diagnostic regression I mentioned before. Sorry, I wasn't clear in my previous message, it happens when one "naturally" just passes loc, not the EXPR_

Re: Turn check macros into functions. (issue6188088)

2012-05-15 Thread Diego Novillo
On 12-05-15 13:19 , Lawrence Crowl wrote: The gcc source uses several constructs that GDB does not understand. This patch corrects some of them. It affects only compilers built with ENABLE_TREE_CHECKING, and hence release compilers are unaffected. In particular, I change the implementation of C

Merge from gcc-4_7-branch to gccgo branch

2012-05-15 Thread Ian Lance Taylor
I have done yet another merge from gcc-4_7-branch to gccgo branch, this time from revision 187566. Ian

Go patch committed: Fix taking address of constant outside function

2012-05-15 Thread Ian Lance Taylor
This patch to the Go frontend extends the code I committed earlier today for taking the address of a constant to work outside of a function. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline and 4.7 branch. Ian diff -r f37debbb3806 go/expressions.cc --- a/go/exp

Re: [C++ Patch] PR 44516

2012-05-15 Thread Paolo Carlini
Hi, On 05/16/2012 12:11 AM, Jason Merrill wrote: On 05/15/2012 04:56 PM, Paolo Carlini wrote: @@ -2753,7 +2754,8 @@ build_x_indirect_ref (location_t loc, tree expr, r rval = cp_build_indirect_ref (expr, errorstring, complain); if (processing_template_decl&& rval != error_mark_node)

Fix PR ada/52494

2012-05-15 Thread Eric Botcazou
Andris Pavenis devised this patch to fix the build for targets which use the s-taprop-dummy.adb package. Applied on mainline and 4.7 branch. 2012-05-15 Andris Pavenis PR ada/52494 * s-taprop-dummy.adb (Specific): New package. -- Eric Botcazou --- s-taprop-dummy.adb~ 2012-

Go patch committed: Sort array constructors by index

2012-05-15 Thread Ian Lance Taylor
This patch to the Go frontend sorts array constructors by index. That is what the middle-end expects. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline and 4.7 branch. Ian diff -r b4118d35075b go/expressions.cc --- a/go/expressions.cc Tue May 15 12:58:54 2012

Re: [google/google-main] Fix regression - SUBTARGET_EXTRA_SPECS overridden by LINUX_GRTE_EXTRA_SPECS (issue 6016047)

2012-05-15 Thread jingyu
I suspect this patch would change specs of non-i386 platform. For example, LINUX_GRTE_EXTRA_SPECS is not part of SUBTARGET_EXTRA_SPECS for darwin. Can you tell where SUBTARGET_EXTRA_SPECS is firstly defined for chromeos toolchain? On 2012/05/14 18:32:17, shenhan wrote: On 2012/04/12 21:14:29,

Re: [C++ Patch] PR 11586

2012-05-15 Thread Jason Merrill
On 05/15/2012 04:40 PM, Paolo Carlini wrote: However, I just noticed that I somehow missed a small issue in the testsuite (sorry): we have a testcase which is now failing because in templates we don't warn anymore (the warning is produced by warn_for_div_by_zero which also uses c_inhibit_evaluati

Re: [C++ Patch] PR 44516

2012-05-15 Thread Jason Merrill
On 05/15/2012 04:56 PM, Paolo Carlini wrote: @@ -2753,7 +2754,8 @@ build_x_indirect_ref (location_t loc, tree expr, r rval = cp_build_indirect_ref (expr, errorstring, complain); if (processing_template_decl&& rval != error_mark_node) -return build_min_non_dep (INDIRECT_REF, rval,

Re: trans-mem: functions making indirect calls are not transformed (issue6194061)

2012-05-15 Thread Patrick Marlier
Follow-up of Dave's patch with a testcase. Functions with indirect calls must be transactified (transformed). Tested on i686. Ok for trunk? -- Patrick 2012-05-15 Dave Boutcher Patrick Marlier * trans-mem.c (ipa_tm_transform_clone): Transform functions with indir

Re: [google/gcc-4_7] Allow static const floats unless -pedantic is passed (issue6212051)

2012-05-15 Thread Jeffrey Yasskin
Looks good to me. Yay for kicking the can down the road. ;) On Tue, May 15, 2012 at 12:57 PM, Ollie Wild wrote: > To be applied to google/gcc-4_7. > > Allow static const floats unless -pedantic is passed. > > This patch allows us to migrate to C++11 more incrementally, since we can > leave > the

Re: gnu-tm: Dont allow assigning transaction_unsafe functions to transaction_safe function pointers (issue6198054)

2012-05-15 Thread Patrick Marlier
Follow-up of Dave's patch. I would prefer to see such checks in trans-mem.c as follows. In a transaction, a function pointer can be declared and assigned but there is no check that the function pointer is transaction_safe. So at runtime, if the function was unsafe, libitm stops on assert because

Re: Turn check macros into functions. (issue6188088)

2012-05-15 Thread Mike Stump
On May 15, 2012, at 11:23 AM, Steven Bosscher wrote: > On Tue, May 15, 2012 at 7:19 PM, Lawrence Crowl wrote: >> The gcc source uses several constructs that GDB does not understand. >> This patch corrects some of them. It affects only compilers built >> with ENABLE_TREE_CHECKING, and hence releas

Re: find_movable_pseudos vs. split moves (was Re: rx-elf fails after r187015)

2012-05-15 Thread DJ Delorie
> gcc/ > * ira.c (pseudo_move_insn): Delete. > (find_moveable_pseudos): Don't set it. > (move_unallocated_pseudos): Use DF_REG_DEF_CHAIN to find > the definitions of the original pseudo. Delete all of them. I can build with this. Thanks!

Re: [C++ Patch] PR 44516

2012-05-15 Thread Paolo Carlini
Hi, On 05/15/2012 08:53 PM, Jason Merrill wrote: On 05/14/2012 07:17 PM, Paolo Carlini wrote: @@ -5670,7 +5672,8 @@ build_x_compound_expr_from_list (tree list, expr_l return error_mark_node; for (list = TREE_CHAIN (list); list; list = TREE_CHAIN (list)) -expr = build_x_compou

Re: [C++ Patch] PR 11586

2012-05-15 Thread Paolo Carlini
On 05/15/2012 03:14 PM, Jason Merrill wrote: OK. Thanks. I applied the patch. However, I just noticed that I somehow missed a small issue in the testsuite (sorry): we have a testcase which is now failing because in templates we don't warn anymore (the warning is produced by warn_for_div_by_z

Re: find_movable_pseudos vs. split moves (was Re: rx-elf fails after r187015)

2012-05-15 Thread Vladimir Makarov
On 05/15/2012 09:08 AM, Richard Sandiford wrote: DJ Delorie writes: After r187015 (Mar 31), gcc builds for rx-elf are failing with: make[3]: Entering directory `/greed/dj/m32c/gcc/rx-elf-head/rx-elf/64-bit-double/libgcc' # If this is the top-level multilib, build all the other # multilibs. /g

Re: patch for PR53125

2012-05-15 Thread Vladimir Makarov
On 05/15/2012 07:59 AM, Steven Bosscher wrote: On Fri, May 11, 2012 at 8:42 PM, Steven Bosscher wrote: Now with patch On Fri, May 11, 2012 at 8:42 PM, Steven Bosscher wrote: 2012-05-10 Vladimir Makarov PR rtl-optimization/53125 * ira.c (ira): Call find_moveable_pseudos

[Fortran, patch] PR 48831 - Constant expression (PARAMETER array element) rejected as nonconstant

2012-05-15 Thread Alessandro Fanfarillo
Hi All, in attachment there's a patch for PR 48831, it also includes a new test case suggested by Tobias Burnus. The patch is bootstrapped and tested on x86_64-unknown-linux-gnu. Regards. Alessandro Fanfarillo patch.diff Description: Binary data

Merge from gcc-4_7-branch to gccgo branch

2012-05-15 Thread Ian Lance Taylor
I've merged gcc 4.7 branch revision 187554 to the gccgo branch. Ian

Re: [Ada] Restrict propagation of atomicity from objects to types

2012-05-15 Thread Eric Botcazou
> Tested on x86_64-pc-linux-gnu, committed on trunk > > 2012-05-15 Eric Botcazou > > * sem_prag.adb (Process_Atomic_Shared_Volatile): Propagate > atomicity from an object to its underlying type only if it > is composite. Backported to the 4.7 branch after testing on x86/ and P

Go patch committed: Don't try to address of a constant

2012-05-15 Thread Ian Lance Taylor
This patch changes the Go frontend to avoid trying to directly take the address of a constant when generating GENERIC. That actually mostly works--unless the DECL_INITIAL of the constant is another constant. This patch changes the code to use a temporary variable instead. Bootstrapped and ran Go t

[google/gcc-4_7] Allow static const floats unless -pedantic is passed (issue6212051)

2012-05-15 Thread Ollie Wild
To be applied to google/gcc-4_7. Allow static const floats unless -pedantic is passed. This patch allows us to migrate to C++11 more incrementally, since we can leave the static const float initializations in place, flip the switch, and then change it to use constexpr. This is a forward port of

[PATCH] PR rtl-optimization/53352

2012-05-15 Thread Meador Inge
Hi All, As reported in PR rtl-optimization/53352 CSE currently trips up on a paradoxical subreg case. When compiling for ARM GNU/Linux with -O3 the expanded RTL of interest looks like: (insn 12 11 13 3 (set (reg:SI 140) (lshiftrt:SI (reg/v:SI 135 [ tmp1 ]) (const_int 16 [0x10

Re: [PATCH, Android] Stack protector enabling for Android target

2012-05-15 Thread Maxim Kuvyrkov
On 12/05/2012, at 9:03 AM, Igor Zamyatin wrote: > Hi! > > Please look at the modified patch in the attachment. ChangeLog remains the > same. > > Tested in android environment(x86_64-*-linux-android), also > bootstrapped on x86_64-unknown-linux-gnu. > I also started regtesting on linux. Is it ok

Re: [Patch] Use CAN_HAVE_LOCATION_P in EXPR_LOCATION

2012-05-15 Thread Jason Merrill
OK. Jason

Re: Turn check macros into functions. (issue6188088)

2012-05-15 Thread Diego Novillo
On 12-05-15 14:23 , Steven Bosscher wrote: On Tue, May 15, 2012 at 7:19 PM, Lawrence Crowl wrote: The gcc source uses several constructs that GDB does not understand. This patch corrects some of them. It affects only compilers built with ENABLE_TREE_CHECKING, and hence release compilers are un

Re: Build problem with libgo runtime

2012-05-15 Thread Ian Lance Taylor
"William J. Schmidt" writes: > I'm investigating another build failure for Fedora 17 (based on 4.7). > The failing compile from the build log is as follows: > > /bin/sh ./libtool --tag=CC > --mode=compile > /builddir/build/BUILD/gcc-4.7.0-20120504/obj-ppc64-redhat-linux/./gcc/xgcc > -B/builddi

Re: [C++ Patch] PR 44516

2012-05-15 Thread Jason Merrill
On 05/14/2012 07:17 PM, Paolo Carlini wrote: @@ -5670,7 +5672,8 @@ build_x_compound_expr_from_list (tree list, expr_l return error_mark_node; for (list = TREE_CHAIN (list); list; list = TREE_CHAIN (list)) - expr = build_x_compound_expr (expr, TREE_VALUE (list), + expr

Re: [PATCH preprocessor, diagnostics] PR preprocessor/53229 - Fix diagnostics location when pasting tokens

2012-05-15 Thread Tom Tromey
> "Dodji" == Dodji Seketeli writes: Dodji> To properly fix this, I think libcpp should keep the token of the Dodji> pasting operator '##', instead of representing it with flag on the LHS Dodji> operand's token. That way, it could use its location. Originally I had thought that a pasted toke

Re: [PATCH] PR preprocessor/7263 - Avoid pedantic warnings on system headers macro tokens

2012-05-15 Thread Jason Merrill
OK. Jason

Re: Turn check macros into functions. (issue6188088)

2012-05-15 Thread Steven Bosscher
On Tue, May 15, 2012 at 7:19 PM, Lawrence Crowl wrote: > The gcc source uses several constructs that GDB does not understand. > This patch corrects some of them.  It affects only compilers built > with ENABLE_TREE_CHECKING, and hence release compilers are unaffected. > Index: gcc/ChangeLog.cxx-con

Turn check macros into functions. (issue6188088)

2012-05-15 Thread Lawrence Crowl
The gcc source uses several constructs that GDB does not understand. This patch corrects some of them. It affects only compilers built with ENABLE_TREE_CHECKING, and hence release compilers are unaffected. In particular, I change the implementation of CHECK macros using __extension__ into macros

Merge from gcc-4_7-branch to gccgo branch

2012-05-15 Thread Ian Lance Taylor
I have merged gcc-4_7-branch revision 187489 to the gccgo branch. Ian

Re: gnu-tm: Dont allow assigning transaction_unsafe functions to transaction_safe function pointers (issue6198054)

2012-05-15 Thread Torvald Riegel
On Tue, 2012-05-08 at 18:02 -0500, Dave Boutcher wrote: > Without this patch it is perfectly fine to assign non-transaction_safe > functions to function pointers marked as transaction_safe. Unpleasantness > happens at run time. > > e.g. > > __attribute__((transaction_safe)) long (*compare)(int,

Re: PING: PATCH: Backport x32 support to libtool

2012-05-15 Thread Nathanael Nerode (GCC)
On 05/15/2012 11:32 AM, H.J. Lu wrote: > On Tue, May 15, 2012 at 8:06 AM, Nathanael Nerode (GCC) > wrote: >> On 05/11/2012 12:50 PM, H.J. Lu wrote: >>> On Mon, Apr 16, 2012 at 10:47 AM, H.J. Lu wrote: On Tue, Apr 3, 2012 at 7:49 AM, H.J. Lu wrote: > On Thu, Mar 29, 2012 at 7:34 AM, H.J.

Re: PING: PATCH: Backport x32 support to libtool

2012-05-15 Thread H.J. Lu
On Tue, May 15, 2012 at 8:06 AM, Nathanael Nerode (GCC) wrote: > On 05/11/2012 12:50 PM, H.J. Lu wrote: >> On Mon, Apr 16, 2012 at 10:47 AM, H.J. Lu wrote: >>> On Tue, Apr 3, 2012 at 7:49 AM, H.J. Lu wrote: On Thu, Mar 29, 2012 at 7:34 AM, H.J. Lu wrote: > On Sat, Mar 3, 2012 at 9:54 A

Re: PING: PATCH: Backport x32 support to libtool

2012-05-15 Thread Nathanael Nerode (GCC)
On 05/11/2012 12:50 PM, H.J. Lu wrote: > On Mon, Apr 16, 2012 at 10:47 AM, H.J. Lu wrote: >> On Tue, Apr 3, 2012 at 7:49 AM, H.J. Lu wrote: >>> On Thu, Mar 29, 2012 at 7:34 AM, H.J. Lu wrote: On Sat, Mar 3, 2012 at 9:54 AM, H.J. Lu wrote: > Hi, > > This patch backports x32 supp

[Patch] Use CAN_HAVE_LOCATION_P in EXPR_LOCATION

2012-05-15 Thread Paolo Carlini
Hi, this is something I noticed when looking into more EXPR_LOC_OR_HERE uses in the C++ front-end: I don't think we want to crash if EXPR_LOC_OR_HERE is passed a NULL_TREE. Tested x86_64-linux. Thanks, Paolo. /// 2012-05-15 Paolo Carlini * tree.h (EXPR_LOCATION):

PING: Re: [PR tree-optimization/52558]: RFC: questions on store data race

2012-05-15 Thread Aldy Hernandez
PING. Hi. Sorry for the delay. There were various tricky hiccups along the way to bootstrappability and regression cleanliness... On 04/26/12 04:51, Richard Guenther wrote: On Wed, 25 Apr 2012, Aldy Hernandez wrote: On 04/25/12 06:45, Richard Guenther wrote: On Tue, Apr 24, 2012 at 7:43 PM,

ping: Use "sed -n …" instead of "sed s/…/p -e d" in s-header-vars

2012-05-15 Thread Olivier Hainque
Hello, Ping for http://gcc.gnu.org/ml/gcc-patches/2012-04/msg01896.html New patch attached. Following Hans suggestion, I have added a comment to prevent going back to the original scheme and problem in the future. Thanks in advance for your feedback, With Kind Regards, Olivier On Apr 30, 2

Re: [PATCH preprocessor, diagnostics] PR preprocessor/53229 - Fix diagnostics location when pasting tokens

2012-05-15 Thread Dodji Seketeli
Jason Merrill writes: > On 05/15/2012 07:18 AM, Dodji Seketeli wrote: >> +paste_tokens (cpp_reader *pfile, source_location lhs_location, > > If in the long run we want the location passed in to be the ## > location, let's drop the "lhs" from the parameter name. Dropped. > >> - const cpp_to

Re: Use "sed -n …" instead of "sed s/…/p -e d" in s-header-vars

2012-05-15 Thread Olivier Hainque
Hello Hans, On May 15, 2012, at 05:22 , Hans-Peter Nilsson wrote: > >>* Makefile.in (s-header-vars): Resort to -n instead of trailing >>-e d in sed invocation. > > Random non-maintainer comments: I'd suggest adding a nearby > comment to avoid a future edit changing it back. Su

Re: remove TARGET_E500 and factorize SPE defaults computation in powerpc ports

2012-05-15 Thread Olivier Hainque
Hello David, Back on this one after Alan's correction and extra testing of the patch on my side. On May 7, 2012, at 18:59 , David Edelsohn wrote: >>> http://gcc.gnu.org/ml/gcc-patches/2010-08/msg01667.html > > Yes, exactly. If we can work through the fallout, this is exactly the > type of cl

Re: [RFC] PR 51712 -Wtype-limits should not trigger for enum constants

2012-05-15 Thread Gabriel Dos Reis
On Tue, May 15, 2012 at 4:49 AM, Manuel López-Ibáñez wrote: > On 15 May 2012 11:06, Richard Guenther wrote: >> On Mon, May 14, 2012 at 8:47 PM, Joseph S. Myers >> wrote: >>> On Mon, 14 May 2012, Manuel López-Ibáñez wrote: >>> PING: http://gcc.gnu.org/ml/gcc-patches/2012-05/msg00383.html >>>

[PATCH][5/n] Remove mark_symbols_for_renaming

2012-05-15 Thread Richard Guenther
This tackles thunks. Easiest would be to not claim we are in SSA form (we are not) and just go into SSA ... but well. This makes us use helpers properly. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2012-05-15 Richard Guenther * cgraphunit.c (thunk_adjust

Re: [RFC] PR 51712 -Wtype-limits should not trigger for enum constants

2012-05-15 Thread Gabriel Dos Reis
On Tue, May 15, 2012 at 4:06 AM, Richard Guenther wrote: > On Mon, May 14, 2012 at 8:47 PM, Joseph S. Myers > wrote: >> On Mon, 14 May 2012, Manuel López-Ibáñez wrote: >> >>> PING: http://gcc.gnu.org/ml/gcc-patches/2012-05/msg00383.html >> >> The general idea of checking this information in short

Re: [PATCH] Another fix for TARGET_READ_MODIFY_WRITE i386 peepholes (PR target/53358)

2012-05-15 Thread Uros Bizjak
On Tue, May 15, 2012 at 1:47 PM, Jakub Jelinek wrote: > This is a similar problem as PR52086, but back then when looking > at whether other peepholes might need similar treatment I didn't think > about the possibility that addqi_1 pattern would allow non-q constraint > register.  Apparently it do

Re: [PATCH preprocessor, diagnostics] PR preprocessor/53229 - Fix diagnostics location when pasting tokens

2012-05-15 Thread Jason Merrill
On 05/15/2012 07:18 AM, Dodji Seketeli wrote: +paste_tokens (cpp_reader *pfile, source_location lhs_location, If in the long run we want the location passed in to be the ## location, let's drop the "lhs" from the parameter name. - const cpp_token *result = cpp_get_token (pfile); +

Re: [C++ Patch] PR 11586

2012-05-15 Thread Jason Merrill
OK. Jason

find_movable_pseudos vs. split moves (was Re: rx-elf fails after r187015)

2012-05-15 Thread Richard Sandiford
DJ Delorie writes: > After r187015 (Mar 31), gcc builds for rx-elf are failing with: > > make[3]: Entering directory > `/greed/dj/m32c/gcc/rx-elf-head/rx-elf/64-bit-double/libgcc' > # If this is the top-level multilib, build all the other > # multilibs. > /greed/dj/m32c/gcc/rx-elf-head/./gcc/xgcc

[PATCH][1/n] Improve vectorization in PR53355

2012-05-15 Thread Richard Guenther
This is the first patch to make the generated code for the testcase in PR53355 better. It teaches VRP about LSHIFT_EXPRs (albeit only of a very simple form). Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Richard. 2012-05-15 Richard Guenther PR tree-optimization/533

[Ada] Restrict propagation of atomicity from objects to types

2012-05-15 Thread Arnaud Charlet
The front-end uses a trick to accept a pragma Atomic on objects whose type isn't directly suitable for the pragma: if the type is declared in the same unit as the object, the front-end propagates under the hood the atomicity from the object to the type. This can have unwanted effects, most notably

[Ada] Better handling of illegal iterators

2012-05-15 Thread Arnaud Charlet
This patch improves on the error message for a loop iterator where the loop variable is referenced in the name of the iterator. Compiling main.adb must yield: main.adb:26:17: object "X" cannot be used before end of its declaration --- pragma Ada_2012; pragma Warnings (Off); with Ada.Text_IO

[Ada] Imaging of arbitrary precision integers

2012-05-15 Thread Arnaud Charlet
The compiler includes an arbitrary precision integer computation library, which is used to handle numeric literals in source code (which are allowed to be of arbitrary size). This change improves the code responsible for producing a string image of such integers by using a single Euclidian division

[Ada] Couple of small adjustments to cross-unit inlining

2012-05-15 Thread Arnaud Charlet
The first adjustment ensures that the compiler doesn't inline back into the main unit subprograms coming from subunits, since they are already compiled. The second adjustment ensures that the transitive closure algorithm keeps excluding the subprograms not handled by Add_Inlined_Body. Tested on x8

Re: patch for PR53125

2012-05-15 Thread Steven Bosscher
On Fri, May 11, 2012 at 8:42 PM, Steven Bosscher wrote: > Now with patch > > On Fri, May 11, 2012 at 8:42 PM, Steven Bosscher > wrote: > 2012-05-10  Vladimir Makarov > >          PR rtl-optimization/53125 >          * ira.c (ira): Call find_moveable_pseudos and >          mov

[PATCH] Another fix for TARGET_READ_MODIFY_WRITE i386 peepholes (PR target/53358)

2012-05-15 Thread Jakub Jelinek
Hi! This is a similar problem as PR52086, but back then when looking at whether other peepholes might need similar treatment I didn't think about the possibility that addqi_1 pattern would allow non-q constraint register. Apparently it does in some of the alternatives, the insn is then emitted wi

Re: [PATCH preprocessor, diagnostics] PR preprocessor/53229 - Fix diagnostics location when pasting tokens

2012-05-15 Thread Manuel López-Ibáñez
On 15 May 2012 13:18, Dodji Seketeli wrote: > > People seem to like screenshots. > > Thus, after the patch, we now have: > >    $ cc1 -quiet test.c >    test.c: In function 'foo': >    test.c:6:18: error: pasting "." and "i" does not give a valid > preprocessing token >     #define TEST(X) x.##X

[C++ Patch] PR 11586

2012-05-15 Thread Paolo Carlini
Hi, this small patch fixes the problem using the approach indicated by Jason in the audit trail, already used in many other similar cases. Booted and tested x86_64-linux. Thanks, Paolo. / /cp 2012-05-15 Paolo Carlini PR c++/11586 * pt.c (tsubst_cop

[PATCH preprocessor, diagnostics] PR preprocessor/53229 - Fix diagnostics location when pasting tokens

2012-05-15 Thread Dodji Seketeli
Hello, As stated in the audit trail of this problem report, consider this test case: $ cat test.c 1 struct x { 2int i; 3 }; 4 struct x x; 5 6 #define TEST(X) x.##X 7 8 void foo (void)

[PATCH diagnostics] Make unwound macro expansion trace less redundant

2012-05-15 Thread Dodji Seketeli
Hello, As discussed previously, the unwinder for macro expansion is quite verbose [1]. This patch proposes to address that shortcoming. Consider this test case: $ cat -n test.c 1 #define MYMAX(A,B) __extension__ ({ __typeof__(A) __a = (A); \ 2 __typeof__(B) __b

[Ada] Fix cross-unit inlining on internal inline subprograms

2012-05-15 Thread Arnaud Charlet
This restores cross-unit inlining for user-defined internal inline subprograms invoked from public inline subprograms. This also reworks the algorithm used to compute the list of inlined subprograms passed to gigi. The old algorithm attempts to compute a topological order on the list, and thus s

[Ada] Private with-clauses in the context of generic package declarations

2012-05-15 Thread Arnaud Charlet
In ihe context of a generic package declaration, a private with-clause on a child unit implies that the implicit with clauses on its parents are private as well. Previous to this patch, GNAT applied this rule applied only to non- eneric units. Compiling the following must be rejected with: gp

[Ada] Add postconditions to numeric functions

2012-05-15 Thread Arnaud Charlet
Ada RM requires numeric functions to return values in specific ranges, and specific values for some specified argument values. Many of these are now translated as Ada 2012 Post aspects on numeric functions. Tested on x86_64-pc-linux-gnu, committed on trunk 2012-05-15 Yannick Moy * a-n

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

2012-05-15 Thread Eric Botcazou
> I checked in a fix. Thanks! -- Eric Botcazou

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

2012-05-15 Thread Bernd Schmidt
On 05/15/2012 09:12 AM, Eric Botcazou wrote: Ok, that looks like three votes in favour. I've checked in the following variant with a few minor changes. I've added strcpy and memset to the list of functions, and split off a new function in ira-lives. Changes to subdirectories must be documented

[Ada] Interfacing aspects in Ada 2012

2012-05-15 Thread Arnaud Charlet
The aspects Convention, Export, and Import are intended to replace the use of the earlier pragmas by the same names. The additional aspects External_Name and Link_Nmae provide the remaining functionality. which previously was provided by additional pragma arguments. Tested on x86_64-pc-linux-gnu,

Re: [Fortran] Patch ping

2012-05-15 Thread Bernhard Reutner-Fischer
On Fri, May 11, 2012 at 07:34:30PM +0200, Tobias Burnus wrote: >On 18 April 2012 at 18:57, Bernhard Reutner-Fischer wrote: >>On Tue, Apr 17, 2012 at 12:47:48AM +0200, Tobias Burnus wrote: >>>Approved but not yet committed: >>>Bernhard: >>>- [PATCH] gfortran testsuite: implicitly cleanup-modules, pa

[Patch, Fortran] PR51055 - accept non-spec-expr "i" in allocate(character(len=i)::s)

2012-05-15 Thread Tobias Burnus
A rather simple patch. Build and regtested on x86-64-linux. OK for the trunk? I think that is the last patch required for commonly used code. Remaining are issues with array constructors and concatenations - and, of course, deferred-length components. Tobias 2012-05-15 Tobias Burnus PR

[Ada] Get rid of compilation warnings in raise-gcc.c

2012-05-15 Thread Arnaud Charlet
No functional change. Tested on x86_64-pc-linux-gnu, committed on trunk 2012-05-15 Tristan Gingold * raise-gcc.c (db_region_for): Use %p + cast to avoid warnings. (get_region_description_for): Likewise. (db_action_for): Likewise. (get_call_site_action_for): Lik

[Ada] Save occurrence on the stack before calling last chance handler

2012-05-15 Thread Arnaud Charlet
The occurrence was already saved but in dynamically allocated memory. This patch saves it on the stack to avoid the allocation, which useful in case of Storage_Error. No functional change in normal conditions. Tested on x86_64-pc-linux-gnu, committed on trunk 2012-05-15 Tristan Gingold

[Ada] Missing DSA remote access to subprogram rules enforcement

2012-05-15 Thread Arnaud Charlet
This change adds missing code to enforce language restrictions for remote access to subprogram types in the case of 'Unchecked_Access and 'Unrestricted_ Access. The following compilation must be rejected with the given error message: $ gcc -c server_main.adb server_main.adb:11:35: prefix must stat

Re: [Fortran, patch] PR 52158 - Regression on character function with gfortran 4.7

2012-05-15 Thread Alessandro Fanfarillo
2012/5/13 Tobias Burnus : > > Committed as Rev. 187436. Thanks for the patch and congratulation to your > first GCC contribution! Thank you for your priceless support! Alessandro

[v3] PR libstdc++/53339

2012-05-15 Thread Paolo Carlini
Hi, tested x86_64-linux, committed to mainline (see audit trail for details) Thanks, Paolo. / 2012-05-15 Paolo Carlini PR libstdc++/53339 * include/bits/hashtable_policy.h (__detail::_Identity, __detail::_Select1st): Add. (_Map_bas

Re: [RFC] PR 51712 -Wtype-limits should not trigger for enum constants

2012-05-15 Thread Manuel López-Ibáñez
On 15 May 2012 11:06, Richard Guenther wrote: > On Mon, May 14, 2012 at 8:47 PM, Joseph S. Myers > wrote: >> On Mon, 14 May 2012, Manuel López-Ibáñez wrote: >> >>> PING: http://gcc.gnu.org/ml/gcc-patches/2012-05/msg00383.html >> >> The general idea of checking this information in shorten_compare

[Ada] Give full msg for finalize/adjust only if -gnateE.

2012-05-15 Thread Arnaud Charlet
The circuitery to save the first exception message (reraised as PE) in finalize or adjust operations has a distributed code size impact. This circuitery is now only enabled when then switch -gnateE is specified. Tested on x86_64-pc-linux-gnu, committed on trunk 2012-05-15 Tristan Gingold

[Ada] Simplify binder code

2012-05-15 Thread Arnaud Charlet
This patch simply moves code from the binder generated file to a-except, to slightly simplify the binder. Preliminary work. No testcase as there is no functionnal change. Tested on x86_64-pc-linux-gnu, committed on trunk 2012-05-15 Tristan Gingold * bindgen.adb (Gen_Header): Remove c

  1   2   >