[Ada] Adjust Esize processing in gigi

2012-06-11 Thread Eric Botcazou
This makes it so that the size of types (Esize) is retrieved early and properly capped for elementary types, in order to avoid obscure failures later on for huge sizes. Nothing is changed for composite types. Tested on i586-suse-linux, applied on the mainline. 2012-06-11 Eric Botcazou

[Ada] Fix link error on conditional expression

2012-06-11 Thread Eric Botcazou
The executable cannot be linked because of a duplicated symbol in the object files, which comes from a conditional expression used to initialize a constant that is the upper bound of an index type in an array. Tested on i586-suse-linux, applied on the mainline. 2012-06-11 Eric Botcazou

Re: [PATCH 1/2] SH epilogue unwind, dwarf2 pass changes

2012-06-11 Thread Chung-Lin Tang
Ping? On 2012/6/1 06:24 PM, Chung-Lin Tang wrote: On 12/5/23 1:46 AM, Richard Henderson wrote: On 05/18/12 03:48, Chung-Lin Tang wrote: @@ -2401,6 +2401,7 @@ scan_trace (dw_trace_info *trace) { /* Propagate across fallthru edges. */ dwarf2out_flush_queued_reg_saves (); +

Re: [patch] Do not include rtl.h in tree-phinodes.c

2012-06-11 Thread Richard Guenther
On Wed, Jun 6, 2012 at 5:16 PM, Steven Bosscher stevenb@gmail.com wrote: Hello, tree-phinodes.c includes rtl.h for ... ceil_log2 (?!). Moving ceil_log2 to hwint.c/hwint.h to join its sibling floor_log2 breaks this dependency. Bootstrapped on x86_64-unknown-linux-gnu. OK for trunk? Ok.

Re: [patch] Do not include rtl.h in cfgloop.h

2012-06-11 Thread Richard Guenther
On Wed, Jun 6, 2012 at 5:16 PM, Steven Bosscher stevenb@gmail.com wrote: Hello, cfgloop.h only depends on rtl.h for rtx_code. The patch introduces a new enum and a simple function to convert from one enum to another in a few non-performance critical places. The upside is that this makes

[Ada] Plug hole with constants used by reference

2012-06-11 Thread Eric Botcazou
This aligns the processing of constants used by reference in Identifier_to_gnu with that of regular constants; in particular we stop returning the underlying constant if this is the address of a constant and an lvalue is required. Tested on i586-suse-linux, applied on the mainline. 2012-06-11

[PATCH, GCC DOC][AArch64] Fix layout in AArch64 target-specific documentation

2012-06-11 Thread Sofiane Naci
Hi, This patch fixes several layout, formatting and wording issues in the AArch64 target-specific documentation. Thanks Sofiane - 2012-06-11 Sofiane Naci sofiane.n...@arm.com [AArch64] Fix documentation layout. * doc/invoke.texi: Fix white spaces after dots.

Re: Go patch RFA: Fix taking the address of a field of a local variable

2012-06-11 Thread Richard Guenther
On Thu, Jun 7, 2012 at 10:08 AM, Ian Lance Taylor i...@google.com wrote: This patch to the Go compiler fixes the case of taking the address of a field of a local variable.  Previously taking the address of the field was not treated as taking the address of the variable.  But, of course, it is.

Re: [patch] Remove cfglayout.h, and distribute cfglayout.c code to users

2012-06-11 Thread Richard Guenther
On Thu, Jun 7, 2012 at 10:18 PM, Steven Bosscher stevenb@gmail.com wrote: Hello, The attached patch removes cfglayout.h. It was included in a lot of places that didn't need it, and it didn't define much anyway. While checking the users of cfglayout.h, I noticed that a lot of code in

Re: [patch] Move split_double from final.c to rtlanal.c, do not include output.h in expr.c

2012-06-11 Thread Richard Guenther
On Fri, Jun 8, 2012 at 12:23 AM, Steven Bosscher stevenb@gmail.com wrote: Hello, Another patch to move around some code to break up a strange dependency. expr.c isn't supposed to emit assembly so it shouldn't need output.h, but it needs split_double from final.c. This split_double has

[arm-embedded] Backport mainline r181172, r184089, r184180

2012-06-11 Thread Joey Ye
Committed to ARM/embedded-4_6-branch 2012-06-11 Joey Ye joey...@arm.com Backport r184089,184180 from mainline 2012-02-10 Jan Hubicka j...@suse.cz PR middle-end/48600 * predict.c (predict_paths_for_bb): Prevent looping. (predict_paths_leading_to_edge,

Re: [PATCH, GCC DOC][AArch64] Fix layout in AArch64 target-specific documentation

2012-06-11 Thread Marcus Shawcroft
On 11/06/12 09:54, Sofiane Naci wrote: (-march and -mcpu feature modifiers): New subsubsection. Copy and paste subsub? /Marcus

[Ada] Fix unexpected allocation change with size clause

2012-06-11 Thread Eric Botcazou
The attached packages declare two big objects of the same size, but for the first package the object is allocated statically whereas, for the second one, the object is allocated dynamically because of the size clause. The discrepancy stems from a bit vs byte confusion in a specific place in

Re: [PATCH] Correct cost model for strided loads

2012-06-11 Thread Richard Guenther
On Sun, Jun 10, 2012 at 5:58 PM, William J. Schmidt wschm...@linux.vnet.ibm.com wrote: The fix for PR53331 caused a degradation to 187.facerec on powerpc64-unknown-linux-gnu.  The following simple patch reverses the degradation without otherwise affecting SPEC cpu2000 or cpu2006. Bootstrapped

Re: [Fortran, DRAFT patch] PR 46321 - [OOP] Polymorphic deallocation

2012-06-11 Thread Alessandro Fanfarillo
Thank you for the review, with this patch I get some ICEs during the regstest with: gfortran.dg/coarray/poly_run_3.f90 gfortran.dg/elemental_optional_args_5.f03 gfortran.dg/select_type_26.f03 gfortran.dg/select_type_27.f03 gfortran.dg/class_48.f90 gfortran.dg/class_allocate_10.f03

Re: Wider modes for partial int modes

2012-06-11 Thread Bernd Schmidt
On 06/09/2012 01:58 AM, Mike Stump wrote: On Jun 8, 2012, at 4:11 PM, Mike Stump wrote: On Apr 17, 2012, at 2:08 PM, Bernd Schmidt wrote: This patch enables GET_MODE_WIDER_MODE for MODE_PARTIAL_INT (by setting the wider mode to the one the partial mode is based on), which is useful for the

Re: [Fortran, DRAFT patch] PR 46321 - [OOP] Polymorphic deallocation

2012-06-11 Thread Tobias Burnus
On 06/11/2012 11:24 AM, Alessandro Fanfarillo wrote: gfortran.dg/coarray/poly_run_3.f90 That one fails because I for forgot that se.expr in gfc_trans_deallocate contains the descriptor and not the pointer to the data. That's fixed by: tmp = se.expr; if

Patch ping

2012-06-11 Thread Jakub Jelinek
Hi! http://gcc.gnu.org/ml/gcc-patches/2012-06/msg00105.html - add warning for complit with incomplte type Jakub

Re: [PATCH] Hoist adjacent pointer loads

2012-06-11 Thread Richard Guenther
On Mon, Jun 4, 2012 at 3:45 PM, William J. Schmidt wschm...@linux.vnet.ibm.com wrote: Hi Richard, Here's a revision of the hoist-adjacent-loads patch.  I'm sorry for the delay since the last revision, but my performance testing has been blocked waiting for a fix to PR53487.  I ended up

Re: [PATCH] Add vector cost model density heuristic

2012-06-11 Thread Richard Guenther
On Fri, 8 Jun 2012, William J. Schmidt wrote: This patch adds a heuristic to the vectorizer when estimating the minimum profitable number of iterations. The heuristic is target-dependent, and is currently disabled for all targets except PowerPC. However, the intent is to make it general

[Ping] Fix gcc/gcov.c and libgcc/libgcov.c to fix build on VxWorks

2012-06-11 Thread rbmj
Hi everyone, Ping RE: http://gcc.gnu.org/ml/gcc-patches/2012-06/msg00086.html Thanks, Robert Mason On 06/01/2012 02:43 PM, rbmj wrote: On 06/01/2012 02:40 PM, rbmj wrote: Hi everyone, These fixes are to allow building on vxWorks. Currently there are two issues: 1. VxWorks does not

Re: [Patch] Add AC_ARG_ENABLE for libstdc++-v3

2012-06-11 Thread rbmj
On 05/22/2012 04:37 PM, rbmj wrote: This patch adds an AC_ARG_ENABLE option to build/not build libstdc++-v3. I wrote the patch in order to allow the user to override the automatic disable for libstdc++-v3 for certain targets. Ping^2 on http://gcc.gnu.org/ml/gcc-patches/2012-05/msg01525.html

Re: [Ping] Fix gcc/gcov.c and libgcc/libgcov.c to fix build on VxWorks

2012-06-11 Thread Paolo Bonzini
Il 11/06/2012 13:56, rbmj ha scritto: 1. VxWorks does not have a variadic open - it must receive three arguments. gcc/gcov.c however opens a file for reading and does not pass in a mode argument, which causes an error on vxWorks. This just adds a platform-based ifdef around this. I am

Re: [Patch] Add AC_ARG_ENABLE for libstdc++-v3

2012-06-11 Thread Paolo Bonzini
Il 11/06/2012 13:59, rbmj ha scritto: On 05/22/2012 04:37 PM, rbmj wrote: This patch adds an AC_ARG_ENABLE option to build/not build libstdc++-v3. I wrote the patch in order to allow the user to override the automatic disable for libstdc++-v3 for certain targets. Ping^2 on

[PATCH][C++] Fix PR53605

2012-06-11 Thread Richard Guenther
When I changed empty arrays domain to use a signed sizetype [0, -1] domain mangling of a empty-array-type broke because mangling adds an unsigned one to the signed -1 which causes an ICE (I chose to do that instead of shifting the range to [1, 0] because much more code relies on a zero lower

Re: [PATCH] Hoist adjacent pointer loads

2012-06-11 Thread William J. Schmidt
On Mon, 2012-06-11 at 13:28 +0200, Richard Guenther wrote: On Mon, Jun 4, 2012 at 3:45 PM, William J. Schmidt wschm...@linux.vnet.ibm.com wrote: Hi Richard, Here's a revision of the hoist-adjacent-loads patch. I'm sorry for the delay since the last revision, but my performance testing

Re: [PATCH] Hoist adjacent pointer loads

2012-06-11 Thread Richard Guenther
On Mon, 11 Jun 2012, William J. Schmidt wrote: On Mon, 2012-06-11 at 13:28 +0200, Richard Guenther wrote: On Mon, Jun 4, 2012 at 3:45 PM, William J. Schmidt wschm...@linux.vnet.ibm.com wrote: Hi Richard, Here's a revision of the hoist-adjacent-loads patch. I'm sorry for the

[RFC] Target-specific limits on vector alignment

2012-06-11 Thread Richard Earnshaw
The ARM ABI states that vectors larger than 64 bits in size still have 64-bit alignment; never-the-less, the HW supports alignment hints of up to 128-bits in some cases and will trap in a vector has an alignment that less than the hint. GCC currently hard-codes larger vectors to be aligned by the

Re: [PATCH][C++] Fix PR53605

2012-06-11 Thread Jason Merrill
OK. Jason

Re: [PATCH] Correct cost model for strided loads

2012-06-11 Thread William J. Schmidt
On Mon, 2012-06-11 at 11:15 +0200, Richard Guenther wrote: On Sun, Jun 10, 2012 at 5:58 PM, William J. Schmidt wschm...@linux.vnet.ibm.com wrote: The fix for PR53331 caused a degradation to 187.facerec on powerpc64-unknown-linux-gnu. The following simple patch reverses the degradation

Re: [PATCH] Correct cost model for strided loads

2012-06-11 Thread Richard Guenther
On Mon, 11 Jun 2012, William J. Schmidt wrote: On Mon, 2012-06-11 at 11:15 +0200, Richard Guenther wrote: On Sun, Jun 10, 2012 at 5:58 PM, William J. Schmidt wschm...@linux.vnet.ibm.com wrote: The fix for PR53331 caused a degradation to 187.facerec on powerpc64-unknown-linux-gnu.

[PATCH] Fix PR53470

2012-06-11 Thread Richard Guenther
This fixes PR53470 - I for quite some time wanted to remove that conditional clearing of TYPE_CONTEXT from free_lang_data_in_type but failed to do so because we regress. I've debugged it down to the C frontend having sometimes BLOCK as TYPE_CONTEXT for a type. So, simply replace such BLOCK with

Re: [RFC] Target-specific limits on vector alignment

2012-06-11 Thread Richard Guenther
On Mon, Jun 11, 2012 at 3:16 PM, Richard Earnshaw rearn...@arm.com wrote: The ARM ABI states that vectors larger than 64 bits in size still have 64-bit alignment; never-the-less, the HW supports alignment hints of up to 128-bits in some cases and will trap in a vector has an alignment that

RFA: Fix path based disambiguation in RTL oracle

2012-06-11 Thread Michael Matz
Hi, I haven't yet checked in my patch from last week about not creating alias-set conflicts for stack slot sharing mostly due to the caveat I mentioned. Namely that the RTL disambiguator still uses path-based means (which belong to type-based aliasing) even in the write_dependence tests.

Re: RFA: Fix path based disambiguation in RTL oracle

2012-06-11 Thread Richard Guenther
On Mon, Jun 11, 2012 at 4:19 PM, Michael Matz m...@suse.de wrote: Hi, I haven't yet checked in my patch from last week about not creating alias-set conflicts for stack slot sharing mostly due to the caveat I mentioned.  Namely that the RTL disambiguator still uses path-based means (which

[PATCH][2/n] alias.c TLC

2012-06-11 Thread Richard Guenther
This makes ao_ref_from_mem less conservative if either MEM_OFFSET or MEM_SIZE is not set. From other alias.c code and set_mem_attributes_minus_bitpos one has to conclude that MEM_EXPR is always conservatively correct (it only can cover a larger memory area) and the MEM_OFFSET / MEM_SIZE pair

Re: [PATCH] Add vector cost model density heuristic

2012-06-11 Thread William J. Schmidt
On Mon, 2012-06-11 at 13:40 +0200, Richard Guenther wrote: On Fri, 8 Jun 2012, William J. Schmidt wrote: This patch adds a heuristic to the vectorizer when estimating the minimum profitable number of iterations. The heuristic is target-dependent, and is currently disabled for all targets

Re: [RFC] Target-specific limits on vector alignment

2012-06-11 Thread Richard Earnshaw
On 11/06/12 15:17, Richard Guenther wrote: On Mon, Jun 11, 2012 at 3:16 PM, Richard Earnshaw rearn...@arm.com wrote: The ARM ABI states that vectors larger than 64 bits in size still have 64-bit alignment; never-the-less, the HW supports alignment hints of up to 128-bits in some cases and will

Re: [PATCH] Add vector cost model density heuristic

2012-06-11 Thread Richard Guenther
On Mon, 11 Jun 2012, William J. Schmidt wrote: On Mon, 2012-06-11 at 13:40 +0200, Richard Guenther wrote: On Fri, 8 Jun 2012, William J. Schmidt wrote: This patch adds a heuristic to the vectorizer when estimating the minimum profitable number of iterations. The heuristic is

Re: [PATCH] Add vector cost model density heuristic

2012-06-11 Thread Richard Guenther
On Mon, 11 Jun 2012, David Edelsohn wrote: On Mon, Jun 11, 2012 at 7:40 AM, Richard Guenther rguent...@suse.de wrote: Hmm.  I don't like this patch or its general idea too much.  Instead I'd like us to move more of the cost model detail to the target, giving it a chance to look at the

[AARCH64][PATCH] Fix legitimate address checking for TImode and TFmode

2012-06-11 Thread Marcus Shawcroft
This patch fixes the behaviour of legitimate address checking for TImode and TFmode values. The current implementation does not correctly distinguish between the address modes available for ldp/stp X,X versus ldr/str Q. /Marcus ChangeLog: * config/aarch64/aarch64.c

[Ada] fix use of PICFLAG for Ada library variants

2012-06-11 Thread Olivier Hainque
Hello, We used to have in gcc-interface/Makefile.in: GNATLIBCFLAGS_FOR_C = -W -Wall $(GNATLIBCFLAGS) $(TARGET_LIBGCC2_CFLAGS) ... As part of the libgcc move to toplevel,

Re: [PATCH] Add vector cost model density heuristic

2012-06-11 Thread David Edelsohn
On Mon, Jun 11, 2012 at 10:48 AM, Richard Guenther rguent...@suse.de wrote: On Mon, 11 Jun 2012, David Edelsohn wrote: On Mon, Jun 11, 2012 at 7:40 AM, Richard Guenther rguent...@suse.de wrote: Hmm.  I don't like this patch or its general idea too much.  Instead I'd like us to move more of

Re: [RFC] Target-specific limits on vector alignment

2012-06-11 Thread Richard Guenther
On Mon, Jun 11, 2012 at 4:38 PM, Richard Earnshaw rearn...@arm.com wrote: On 11/06/12 15:17, Richard Guenther wrote: On Mon, Jun 11, 2012 at 3:16 PM, Richard Earnshaw rearn...@arm.com wrote: The ARM ABI states that vectors larger than 64 bits in size still have 64-bit alignment;

[PATCH, GCC][AArch64] Remove support for __float128

2012-06-11 Thread Sofiane Naci
Hi, This patch removes __float128 support in the AArch64 port. Thanks Sofiane - gcc/ChangeLog: 2012-06-11 Sofiane Naci sofiane.n...@arm.com [AArch64] Remove __float128 support. * config/aarch64/aarch64.c (aarch64_mangle_type): Remove function.

Re: [PATCH] Add vector cost model density heuristic

2012-06-11 Thread Richard Guenther
On Mon, 11 Jun 2012, David Edelsohn wrote: On Mon, Jun 11, 2012 at 10:48 AM, Richard Guenther rguent...@suse.de wrote: On Mon, 11 Jun 2012, David Edelsohn wrote: On Mon, Jun 11, 2012 at 7:40 AM, Richard Guenther rguent...@suse.de wrote: Hmm.  I don't like this patch or its general

Re: [PATCH] Add vector cost model density heuristic

2012-06-11 Thread Richard Guenther
On Mon, 11 Jun 2012, Richard Guenther wrote: On Mon, 11 Jun 2012, William J. Schmidt wrote: On Mon, 2012-06-11 at 13:40 +0200, Richard Guenther wrote: On Fri, 8 Jun 2012, William J. Schmidt wrote: This patch adds a heuristic to the vectorizer when estimating the minimum

Re: [PATCH] Correct cost model for strided loads

2012-06-11 Thread William J. Schmidt
On Mon, 2012-06-11 at 16:10 +0200, Richard Guenther wrote: On Mon, 11 Jun 2012, William J. Schmidt wrote: On Mon, 2012-06-11 at 11:15 +0200, Richard Guenther wrote: On Sun, Jun 10, 2012 at 5:58 PM, William J. Schmidt wschm...@linux.vnet.ibm.com wrote: The fix for PR53331

Re: [PATCH] Add vector cost model density heuristic

2012-06-11 Thread David Edelsohn
On Mon, Jun 11, 2012 at 10:55 AM, Richard Guenther rguent...@suse.de wrote: Well, they are at least magic numbers and heuristics that apply generally and not only to the single issue in sphinx.  And in fact how it works for sphinx _is_ magic. Second, I suggest that you need to rephrase I can

Re: [PATCH] Add vector cost model density heuristic

2012-06-11 Thread William J. Schmidt
On Mon, 2012-06-11 at 16:58 +0200, Richard Guenther wrote: On Mon, 11 Jun 2012, Richard Guenther wrote: On Mon, 11 Jun 2012, William J. Schmidt wrote: On Mon, 2012-06-11 at 13:40 +0200, Richard Guenther wrote: On Fri, 8 Jun 2012, William J. Schmidt wrote: This patch adds a

Re: [RFC] Target-specific limits on vector alignment

2012-06-11 Thread Richard Earnshaw
On 11/06/12 15:53, Richard Guenther wrote: On Mon, Jun 11, 2012 at 4:38 PM, Richard Earnshaw rearn...@arm.com wrote: On 11/06/12 15:17, Richard Guenther wrote: On Mon, Jun 11, 2012 at 3:16 PM, Richard Earnshaw rearn...@arm.com wrote: The ARM ABI states that vectors larger than 64 bits in size

Re: [PATCH] Add vector cost model density heuristic

2012-06-11 Thread William J. Schmidt
On Mon, 2012-06-11 at 11:09 -0400, David Edelsohn wrote: On Mon, Jun 11, 2012 at 10:55 AM, Richard Guenther rguent...@suse.de wrote: Well, they are at least magic numbers and heuristics that apply generally and not only to the single issue in sphinx. And in fact how it works for sphinx

Re: [C++] Reject variably modified types in operator new

2012-06-11 Thread Florian Weimer
On 06/01/2012 02:09 PM, Florian Weimer wrote: On 06/01/2012 11:00 AM, Florian Weimer wrote: I'll try to warn about this case and make the transformation to the proper operator new[] call. Here's the version. I've added a warning for the ill-formed code. The only remaining glitch is in

Re: [PATCH] don't presume undelegitimized UNSPEC_TLS SYMBOL_REF is a decl

2012-06-11 Thread Roland McGrath
ping? On Wed, Jun 6, 2012 at 2:04 PM, Roland McGrath mcgra...@google.com wrote: cf this change:        2010-11-19  Jakub Jelinek  ja...@redhat.com                PR target/45870                * dwarf2out.c (const_ok_for_output_1): Don't complain about                non-delegitimized TLS

Re: RFA: Alternative iterator implementation

2012-06-11 Thread Tejas Belagod
Richard Sandiford wrote: As discussed in the context of the AARCH64 submission, this patch rewrites the iterator handling in read-rtl.c so that we record iterator positions using an on-the-side VEC rather than placeholder modes and codes. We then substitute in-place for each sequence of

Re: [PATCH] Hoist adjacent pointer loads

2012-06-11 Thread William J. Schmidt
On Mon, 2012-06-11 at 14:59 +0200, Richard Guenther wrote: On Mon, 11 Jun 2012, William J. Schmidt wrote: On Mon, 2012-06-11 at 13:28 +0200, Richard Guenther wrote: On Mon, Jun 4, 2012 at 3:45 PM, William J. Schmidt wschm...@linux.vnet.ibm.com wrote: Hi Richard, Here's a

Re: [PATCH] Hoist adjacent pointer loads

2012-06-11 Thread William J. Schmidt
On Mon, 2012-06-11 at 12:11 -0500, William J. Schmidt wrote: I found this parameter that seems to correspond to well-predicted conditional jumps: /* When branch is predicted to be taken with probability lower than this threshold (in percent), then it is considered well predictable. */

Re: [PATCH 1/2] SH epilogue unwind, dwarf2 pass changes

2012-06-11 Thread Richard Henderson
On 2012-06-11 01:34, Chung-Lin Tang wrote: Ping? On 2012/6/1 06:24 PM, Chung-Lin Tang wrote: On 12/5/23 1:46 AM, Richard Henderson wrote: On 05/18/12 03:48, Chung-Lin Tang wrote: @@ -2401,6 +2401,7 @@ scan_trace (dw_trace_info *trace) { /* Propagate across fallthru edges. */

Re: [PATCH] Add vector cost model density heuristic

2012-06-11 Thread Richard Guenther
On Mon, Jun 11, 2012 at 5:38 PM, William J. Schmidt wschm...@linux.vnet.ibm.com wrote: On Mon, 2012-06-11 at 11:09 -0400, David Edelsohn wrote: On Mon, Jun 11, 2012 at 10:55 AM, Richard Guenther rguent...@suse.de wrote: Well, they are at least magic numbers and heuristics that apply

Re: [RFC] Target-specific limits on vector alignment

2012-06-11 Thread Richard Guenther
On Mon, Jun 11, 2012 at 5:25 PM, Richard Earnshaw rearn...@arm.com wrote: On 11/06/12 15:53, Richard Guenther wrote: On Mon, Jun 11, 2012 at 4:38 PM, Richard Earnshaw rearn...@arm.com wrote: On 11/06/12 15:17, Richard Guenther wrote: On Mon, Jun 11, 2012 at 3:16 PM, Richard Earnshaw

[PATCH][Cilkplus] Patch to make internal struct runtme independent

2012-06-11 Thread Iyer, Balaji V
Hello Everyone, This patch is for the Cilkplus branch affecting both C and C++ compilers. This patch will make the cilkrts_stack_frame struct independent of the runtime. Thanking You, Yours Sincerely, Balaji V. Iyer. Index: gcc/cilk.c

Re: [PATCH] Correct cost model for strided loads

2012-06-11 Thread Richard Guenther
On Mon, Jun 11, 2012 at 5:01 PM, William J. Schmidt wschm...@linux.vnet.ibm.com wrote: On Mon, 2012-06-11 at 16:10 +0200, Richard Guenther wrote: On Mon, 11 Jun 2012, William J. Schmidt wrote: On Mon, 2012-06-11 at 11:15 +0200, Richard Guenther wrote: On Sun, Jun 10, 2012 at 5:58 PM,

[PATCH][Cilkplus] Patch to make internal struct runtme independent

2012-06-11 Thread Iyer, Balaji V
Hello Everyone, This patch is for the Cilkplus branch affecting both C and C++ compilers. This patch will make the cilkrts_stack_frame structure independent of the runtime. Thanking You, Yours Sincerely, Balaji V. Iyer. Index: gcc/cilk.c

Make timevar phases mutually exclusive. (issue6302064)

2012-06-11 Thread Lawrence Crowl
The intent of the phases was to have a high-level but mutually exclusive accounting of compile time. We want to track compile time in a way that tells us which conceptual phases are taking the most time. That intent is not currently satisfied. This patch restores that intent. Add code to

[patch]: move stackalign tests that use dg-options

2012-06-11 Thread Janis Johnson
The tests in gcc.dg/torture/stackalign use two sets of torture options: the usual optimization sets used as default for torture tests, and up to four sets of options that are specific to stack alignment. The stack alignment options are passed via an option that is used by the dg-test option

Re: [PATCH] Fixincludes/VxWorks

2012-06-11 Thread rbmj
On 06/10/2012 02:38 PM, Bruce Korb wrote: On Sun, Jun 10, 2012 at 10:57 AM, Nathan Sidwellnat...@acm.org wrote: On 06/06/12 17:33, rbmj wrote: Hi everyone, This patch series is the result of this [1] thread about fixincludes on VxWorks. It resolves bugs 53457 and 53378, and a few other

Re: [Patch] Add AC_ARG_ENABLE for libstdc++-v3

2012-06-11 Thread rbmj
On 06/11/2012 08:02 AM, Paolo Bonzini wrote: Il 11/06/2012 13:59, rbmj ha scritto: On 05/22/2012 04:37 PM, rbmj wrote: This patch adds an AC_ARG_ENABLE option to build/not build libstdc++-v3. I wrote the patch in order to allow the user to override the automatic disable for libstdc++-v3 for

Re: [patch]: move stackalign tests that use dg-options

2012-06-11 Thread H.J. Lu
On Mon, Jun 11, 2012 at 12:00 PM, Janis Johnson janis_john...@mentor.com wrote: The tests in gcc.dg/torture/stackalign use two sets of torture options: the usual optimization sets used as default for torture tests, and up to four sets of options that are specific to stack alignment.  The stack

Re: [patch]: move stackalign tests that use dg-options

2012-06-11 Thread Janis Johnson
On 06/11/2012 12:15 PM, H.J. Lu wrote: On Mon, Jun 11, 2012 at 12:00 PM, Janis Johnson janis_john...@mentor.com wrote: The tests in gcc.dg/torture/stackalign use two sets of torture options: the usual optimization sets used as default for torture tests, and up to four sets of options that are

[committed] ChangeLog formatting fixes

2012-06-11 Thread Oleg Endo
I've just committed the attached patch as rev 188397 which fixes some formatting in the gcc/ChangeLog. Cheers, Oleg Index: gcc/ChangeLog === --- gcc/ChangeLog (revision 188396) +++ gcc/ChangeLog (working copy) @@ -452,13 +452,15 @@

Re: [patch]: move stackalign tests that use dg-options

2012-06-11 Thread Janis Johnson
On 06/11/2012 12:35 PM, H.J. Lu wrote: On Mon, Jun 11, 2012 at 12:21 PM, Janis Johnson janis_john...@mentor.com wrote: On 06/11/2012 12:15 PM, H.J. Lu wrote: On Mon, Jun 11, 2012 at 12:00 PM, Janis Johnson janis_john...@mentor.com wrote: The tests in gcc.dg/torture/stackalign use two sets of

Re: [patch]: move stackalign tests that use dg-options

2012-06-11 Thread H.J. Lu
On Mon, Jun 11, 2012 at 12:41 PM, Janis Johnson janis_john...@mentor.com wrote: On 06/11/2012 12:35 PM, H.J. Lu wrote: On Mon, Jun 11, 2012 at 12:21 PM, Janis Johnson janis_john...@mentor.com wrote: On 06/11/2012 12:15 PM, H.J. Lu wrote: On Mon, Jun 11, 2012 at 12:00 PM, Janis Johnson

Re: [patch]: move stackalign tests that use dg-options

2012-06-11 Thread Janis Johnson
On 06/11/2012 12:48 PM, H.J. Lu wrote: On Mon, Jun 11, 2012 at 12:41 PM, Janis Johnson janis_john...@mentor.com wrote: On 06/11/2012 12:35 PM, H.J. Lu wrote: On Mon, Jun 11, 2012 at 12:21 PM, Janis Johnson janis_john...@mentor.com wrote: On 06/11/2012 12:15 PM, H.J. Lu wrote: On Mon, Jun

Re: [patch]: move stackalign tests that use dg-options

2012-06-11 Thread H.J. Lu
On Mon, Jun 11, 2012 at 12:57 PM, Janis Johnson janis_john...@mentor.com wrote: On 06/11/2012 12:48 PM, H.J. Lu wrote: On Mon, Jun 11, 2012 at 12:41 PM, Janis Johnson janis_john...@mentor.com wrote: On 06/11/2012 12:35 PM, H.J. Lu wrote: On Mon, Jun 11, 2012 at 12:21 PM, Janis Johnson

Re: [PATCH] Hoist adjacent loads

2012-06-11 Thread William J. Schmidt
OK, once more with feeling... :) This patch differs from the previous one in two respects: It disables the optimization when either the then or else edge is well-predicted; and it now uses the existing l1-cache-line-size parameter instead of a new one (with updated commentary). Bootstraps and

Re: [PATCH] don't presume undelegitimized UNSPEC_TLS SYMBOL_REF is a decl

2012-06-11 Thread Richard Henderson
On 2012-06-06 14:04, Roland McGrath wrote: 2012-06-06 Roland McGrath mcgra...@google.com * dwarf2out.c (const_ok_for_output_1): Detect a TLS UNSPEC using SYMBOL_REF_TLS_MODEL rather than DECL_THREAD_LOCAL_P, in case it's not a VAR_DECL. Also don't limit it to UNSPECs

Re: [PATCH] don't presume undelegitimized UNSPEC_TLS SYMBOL_REF is a decl

2012-06-11 Thread Roland McGrath
On Mon, Jun 11, 2012 at 1:42 PM, Richard Henderson r...@redhat.com wrote: Applied. Thanks!

Re: [PATCH] libgcc: If __GLIBC__ is defined, don't refer to pthread_cancel.

2012-06-11 Thread Richard Henderson
On 2012-06-08 16:23, Roland McGrath wrote: 2012-06-08 Roland McGrath mcgra...@google.com * gthr-posix.h [neither FreeBSD nor Solaris] (__gthread_active_p): If __GLIBC__ is defined, refer to __pthread_key_create instead of pthread_cancel. Applied. r~

Re: [RFC PR48941 / 51980] Rewrite arm_neon.h to use __builtin_shuffle

2012-06-11 Thread Richard Henderson
On 2012-06-11 14:12, Ramana Radhakrishnan wrote: I could generate the masks with appropriate casts to unsigned variants instead.That doesn't seem to make a difference either. No, that shouldn't make any difference. r~

Re: [PATCH] libgcc: If __GLIBC__ is defined, don't refer to pthread_cancel.

2012-06-11 Thread Roland McGrath
On Mon, Jun 11, 2012 at 2:16 PM, Richard Henderson r...@redhat.com wrote: Applied. Thanks!

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

2012-06-11 Thread Richard Henderson
On 2012-05-15 17:24, Hans-Peter Nilsson wrote: gcc: * doc/tm.texi.in (Addressing Modes) TARGET_LEGITIMIZE_ADDRESS: Mention that this hook needs to be defined for native TLS. * doc/tm.texi: Regenerate. Ok. r~

Re: [PATCH] Fix ICE with asm goto and shrink wrapping (PR rtl-optimization/53589)

2012-06-11 Thread Richard Henderson
On 2012-06-06 23:53, Jakub Jelinek wrote: 2012-06-07 Jakub Jelinek ja...@redhat.com PR rtl-optimization/53589 * cfgrtl.c (force_nonfallthru_and_redirect): Do asm_goto_edge discovery even when e-dest != target. If any LABEL_REF points to e-dest label, redirect it

[SH] PR 50749 - Auto-inc-dec does not find subsequent contiguous mem accesses

2012-06-11 Thread Oleg Endo
Hello, I'd like to add the attached SH test cases to verify the auto-inc-dec pass operation on the SH target. Checked by invoking the individual test cases manually with make check-gcc on sh-sim. Some of the tests pass, some of them don't because of the auto-inc-dec issues mentioned in the PR.

Re: [RFC] Target-specific limits on vector alignment

2012-06-11 Thread Hans-Peter Nilsson
From: Richard Earnshaw rearn...@arm.com Date: Mon, 11 Jun 2012 15:16:50 +0200 The ARM ABI states that vectors larger than 64 bits in size still have 64-bit alignment; never-the-less, the HW supports alignment hints of up to 128-bits in some cases and will trap in a vector has an alignment

Re: constant that doesn't fit in 32bits in alpha.c

2012-06-11 Thread Richard Henderson
On 2012-06-10 02:18, Jay K wrote: gcc-4.7.0/gcc/config/alpha/alpha.c word1 = expand_and (DImode, word1, GEN_INT (0x0ffffff0), NULL); That big constant isn't portable since it doesn't fit in 32bits. 1) append LL or 2) break it up into an expression, like

Re: constant that doesn't fit in 32bits in alpha.c

2012-06-11 Thread Richard Henderson
Bah. Wrong patch. r~ * config/alpha/alpha.c (alpha_trampoline_init): Split large constants. diff --git a/gcc/config/alpha/alpha.c b/gcc/config/alpha/alpha.c index 6d15bf7..3dda9fb 100644 --- a/gcc/config/alpha/alpha.c +++ b/gcc/config/alpha/alpha.c @@ -5451,6 +5451,8 @@

Re: libgo patch committed: Update to Go 1.0.1 release

2012-06-11 Thread Mike Stump
On May 4, 2012, at 8:01 AM, Ian Lance Taylor wrote: This patch updates libgo to the Go 1.0.1 release. This is a relatively small collection of bug fixes, with no API changes. So, it is an important part of the test case for libgo/go/mime/multipart/testdata/nested-mime to have ^Ms in the file?

RE: constant that doesn't fit in 32bits in alpha.c

2012-06-11 Thread Jay K
Thank you. I like it. May I have another? book2:gcc jay$ grep -i epoch vms* vmsdbgout.c:/* Difference in seconds between the VMS Epoch and the Unix Epoch */ vmsdbgout.c:static const long long vms_epoch_offset = 3506716800ll; vmsdbgout.c:#define VMS_EPOCH_OFFSET 350671680 vmsdbgout.c: 

Re: constant that doesn't fit in 32bits in alpha.c

2012-06-11 Thread Mike Stump
On Jun 11, 2012, at 4:06 PM, Richard Henderson wrote: Bah. Wrong patch. r~ z.txt Hum, I'm trying to see how this patch works... I feel like there is something I'm missing, like a shift?

Re: [SH] PR 50749 - Auto-inc-dec does not find subsequent contiguous mem accesses

2012-06-11 Thread Kaz Kojima
Oleg Endo oleg.e...@t-online.de wrote: Some of the tests pass, some of them don't because of the auto-inc-dec issues mentioned in the PR. I thought that the tests which are known to fail are marked with XFAIL, though I'm not sure the usual way for the failing test cases. Regards, kaz

Re: constant that doesn't fit in 32bits in alpha.c

2012-06-11 Thread Richard Henderson
On 2012-06-11 16:23, Mike Stump wrote: On Jun 11, 2012, at 4:06 PM, Richard Henderson wrote: Bah. Wrong patch. r~ z.txt Hum, I'm trying to see how this patch works... I feel like there is something I'm missing, like a shift? Double-bah. That's what I get for changing the patch at

Re: constant that doesn't fit in 32bits in alpha.c

2012-06-11 Thread Richard Henderson
On 2012-06-11 16:23, Jay K wrote: Thank you. I like it. May I have another? book2:gcc jay$ grep -i epoch vms* vmsdbgout.c:/* Difference in seconds between the VMS Epoch and the Unix Epoch */ vmsdbgout.c:static const long long vms_epoch_offset = 3506716800ll; vmsdbgout.c:#define

Re: [SH] PR 50749 - Auto-inc-dec does not find subsequent contiguous mem accesses

2012-06-11 Thread Hans-Peter Nilsson
On Tue, 12 Jun 2012, Kaz Kojima wrote: Oleg Endo oleg.e...@t-online.de wrote: Some of the tests pass, some of them don't because of the auto-inc-dec issues mentioned in the PR. I thought that the tests which are known to fail are marked with XFAIL, Yes, with a clear reference to the PR

Re: MIPS testsuite patch for --with-synci configurations

2012-06-11 Thread David Daney
On 06/11/2012 12:30 PM, Steve Ellcey wrote: This patch is to fix MIPS failures that occcur when configuring GCC with the -with-synci option. In this case GCC will generate the synci instruction by default if on a MIPS architecture that supports it . When on an architecture that does not

Re: [SH] PR 50749 - Auto-inc-dec does not find subsequent contiguous mem accesses

2012-06-11 Thread Kaz Kojima
Hans-Peter Nilsson h...@bitrange.com wrote: On Tue, 12 Jun 2012, Kaz Kojima wrote: Oleg Endo oleg.e...@t-online.de wrote: Some of the tests pass, some of them don't because of the auto-inc-dec issues mentioned in the PR. I thought that the tests which are known to fail are marked

Re: [PATCH 3/3] rs6000: Rewrite sync patterns for atomic; expand early.

2012-06-11 Thread David Edelsohn
On Sat, Jun 9, 2012 at 10:40 AM, Richard Henderson r...@twiddle.net wrote: Nope.  I do see the obvious mistake in the atomic_load pattern though: The mode iterator should have been INT1 not INT. Did you want to commit the fix for the iterator? ... and for extra credit we ought to implement

[testsuite] gcc.dg/cpp: make message check lines unique in test summary

2012-06-11 Thread Janis Johnson
This test modifies dg-message test directives by adding comments that will be added to lines in the test summary to eliminate non-unique lines for checks of messages for the same line of source code in a test. Tested on i686-pc-linux-gnu and arm-none-eabi. OK for mainline? Janis 2012-06-11

[testsuite] c-c++-common: make message check lines unique in test summary

2012-06-11 Thread Janis Johnson
This test modifies dg-error and dg-warning test directives by adding comments that will be added to lines in the test summary to eliminate non-unique lines for checks of messages for the same line of source code in a test. Tested on i686-pc-linux-gnu and arm-none-eabi. OK for mainline? Janis

[testsuite] gcc.dg: make message check lines unique in test summary

2012-06-11 Thread Janis Johnson
This test modifies dg-error, dg-warning, dg-message, and dg-bogus test directives by adding comments that will be added to lines in the test summary to eliminate non-unique lines for checks of messages for the same line of source code in a test. Tested on i686-pc-linux-gnu and arm-none-eabi. OK

[testsuite] gcc.c-torture/compile: make msg check lines unique in test summary

2012-06-11 Thread Janis Johnson
This test modifies dg-message test directives by adding comments that will be added to lines in the test summary to eliminate non-unique lines for checks of messages for the same line of source code in a test. Tested on i686-pc-linux-gnu and arm-none-eabi. OK for mainline? Janis 2012-06-11

  1   2   >