[x32] PATCH: Check TARGET_LP64 for SIZE_TYPE and PTRDIFF_TYPE

2011-05-19 Thread H.J. Lu
Hi, I will check in this patch to check TARGET_LP64, instead of TARGET_64BIT, for SIZE_TYPE and PTRDIFF_TYPE. H.J. --- commit 4a3039882737da539adc84287d0e1ffa49a8ba62 Author: H.J. Lu Date: Thu May 19 20:22:15 2011 -0700 Check TARGET_LP64 for SIZE_TYPE and PTRDIFF_TYPE. diff --git a/gcc/C

Hello Would You Like To Earn

2011-05-19 Thread Carmille . Burns
Hello gcc-patches Would you like to earn an extra $200 everyday?, for just 45 minutes work? You could quit your job and make double the money at home working for yourself. visit->http:tinyurl.com/42e38u9 Regards, Carmille Burns Survey Human Resources Dept.

Re: Patch: New GTY ((atomic)) option

2011-05-19 Thread Laurynas Biveinis
2011/5/19 Nicola Pero : > Ok to go ? > +2011-05-20  Nicola Pero   > + > +       * gengtype.c (walk_type): Implemented "atomic" GTY option. > +       * doc/gty.texi (GTY Options): Document "atomic" GTY option. I assume it received the usual testing? If yes, then OK. Thanks! -- Laurynas

Re: [C++0x] contiguous bitfields race implementation

2011-05-19 Thread Aldy Hernandez
On 05/18/11 16:58, Jason Merrill wrote: It seems like you're calculating maxbits correctly now, but an access doesn't necessarily start from the beginning of the sequence of bit-fields, especially given store_split_bit_field. That is, This is what I was trying to explain to you on irc. And I o

Re: [PATCH, i386]: Fix __builtin_ia32_rdrand*_step expansion

2011-05-19 Thread H.J. Lu
On Thu, May 19, 2011 at 12:32 PM, Uros Bizjak wrote: > Hello! > > Attached patch fixes ICE with -m32 -march=i386 -mrdrand on freebsd > [1]. The problem was that __builtin_ia32_rdrand*_step expands to a > sequence that includes conditional move, so we should set > TARGET_CMOVE. Additionally, the pa

Re: [RFA] [PR44618] [PowerPC] Wrong code for -frename-registers

2011-05-19 Thread edmar
This time I put them in a tar file, to make sure the mail client don't mess with them. (ChangeLog is already inside the patch as well) After I complete the regression tests, I will post a new version against latest subversion revision. Will post it here and on bugzilla. Thanks, Edmar On 05/

Re: [RFA] [PR44618] [PowerPC] Wrong code for -frename-registers

2011-05-19 Thread David Edelsohn
010-06-21 Edmar Wienskoski ed...@freescale.com * rs6000.md (save_gpregs_): Replaced pattern with a set of similar patterns, where the MATCH_OPERAND for the function argument is replaced with individual references to hardware registers. * rs6000.md (save_fp

[v3] Add cv tuple_size and tuple_element

2011-05-19 Thread Paolo Carlini
Hi, tested x86_64-linux, committed. Paolo. /// 2011-05-19 Paolo Carlini * include/std/tuple (tuple_element<__i, const _Tp>, tuple_element<__i, volatile _Tp>, tuple_element<__i, const volatile _Tp>, tuple_size, tuple_size, tuple_size): Add.

Re: [patch tree-ssa-reassoc.c]: Better reassoication for comparision and boolean-logic

2011-05-19 Thread Kai Tietz
2011/5/19 Eric Botcazou : >> Bootstrapped and tested for x86_64-pc-linux-gnu for all standard >> languages plus ADA and Obj-C++. > > Ada, not ADA, this is the first name of the Countess of Lovelace: >  http://en.wikipedia.org/wiki/Ada_Lovelace Well, I glad to hear that isn't drived by Russian word

Re: [patch tree-ssa-reassoc.c]: Better reassoication for comparision and boolean-logic

2011-05-19 Thread Eric Botcazou
> Bootstrapped and tested for x86_64-pc-linux-gnu for all standard > languages plus ADA and Obj-C++. Ada, not ADA, this is the first name of the Countess of Lovelace: http://en.wikipedia.org/wiki/Ada_Lovelace -- Eric Botcazou

Re: Patch: New GTY ((atomic)) option

2011-05-19 Thread Basile Starynkevitch
On Thu, 19 May 2011 21:45:49 +0200 Basile Starynkevitch wrote: > However, did you check that the atomic qualifier is correctly written & > re-read in the state (I believe you did, otherwise it probably won't > work). This is needed for plugins using it, or using atomic qualified > fields of existi

Re: Patch: New GTY ((atomic)) option

2011-05-19 Thread Basile Starynkevitch
On Thu, 19 May 2011 20:10:35 +0200 (CEST) "Nicola Pero" wrote: > >>        * gengtype.c (walk_type): Implemented "atomic" GTY option. > >>        * doc/gty.texi (GTY Options): Document "atomic" GTY option. > > > > The patch is OK, with difference between "skip" and "atomic" options > > documented

[PATCH, i386]: Fix __builtin_ia32_rdrand*_step expansion

2011-05-19 Thread Uros Bizjak
Hello! Attached patch fixes ICE with -m32 -march=i386 -mrdrand on freebsd [1]. The problem was that __builtin_ia32_rdrand*_step expands to a sequence that includes conditional move, so we should set TARGET_CMOVE. Additionally, the patch fixes expansion with void return: void foo (unsigned short *

Re: [google] Re: Build more of libstdc++ exception throwing code with frame pointers (issue4539068)

2011-05-19 Thread Ollie Wild
Approved for google/integration (since Diego is on vacation). Paul, please also merge this change only to google/main and google/gcc-4_6. Thanks, Ollie On Thu, May 19, 2011 at 1:24 PM, Paul Pluzhnikov wrote: > > This patch is for google/integration branch. > > Sorry about not setting the marker

Re: [Patch, Fortran] Implement/fix cobounds for scalar coarrays on tree level

2011-05-19 Thread Thomas Koenig
Hi Tobias, *ping* http://gcc.gnu.org/ml/fortran/2011-05/msg00123.html OK. Thanks for the patch! Thomas

Re: [build] Define HAVE_GAS_HIDDEN on Darwin

2011-05-19 Thread Jason Merrill
On 05/19/2011 01:33 PM, Rainer Orth wrote: The cleanest way to account for this seems to allow overriding USE_LINKONCE_INDIRECT in target headers (darwin.h in this case). If Jason or Richard consider this appropriate, I'll modify the patch accordingly and apply after retesting. Not really. Th

Re: New options to disable/enable any pass for any functions (issue4550056)

2011-05-19 Thread Xinliang David Li
On Thu, May 19, 2011 at 11:17 AM, Andi Kleen wrote: > On Thu, May 19, 2011 at 11:10:24AM -0700, Xinliang David Li wrote: >> On Thu, May 19, 2011 at 11:04 AM, Andi Kleen wrote: >> > davi...@google.com (David Li) writes: >> > >> >> -fdisable-tree-ccp1    <--- disable ccp1 for all functions >> >> -f

Re: [google] Increase inlining limits with FDO/LIPO

2011-05-19 Thread Xinliang David Li
I have done some SPEC testing evaluating the performance impact of your patch. They look very positive. LIPO got helped even more than FDO (I only did SPEC2k LIPO testing). Thanks, David 1. SPEC06 (C/C++) with FDO before after Improvement -

[google] Re: Build more of libstdc++ exception throwing code with frame pointers (issue4539068)

2011-05-19 Thread Paul Pluzhnikov
This patch is for google/integration branch. Sorry about not setting the markers correctly. Tested by doing a bootstrap build and verifying that __gnu_cxx::__verbose_terminate_handler is built with frame pointers. On Thu, May 19, 2011 at 10:46 AM, Paul Pluzhnikov wrote: > 2011-05-19  Paul Pluzh

Re: New options to disable/enable any pass for any functions (issue4550056)

2011-05-19 Thread Andi Kleen
On Thu, May 19, 2011 at 11:10:24AM -0700, Xinliang David Li wrote: > On Thu, May 19, 2011 at 11:04 AM, Andi Kleen wrote: > > davi...@google.com (David Li) writes: > > > >> -fdisable-tree-ccp1    <--- disable ccp1 for all functions > >> -fenable-tree-cunroll=1   <--- enable complete unroll for the

Re: RFA: MN10300: Add TLS support

2011-05-19 Thread Richard Henderson
> + fmt = GET_RTX_FORMAT (GET_CODE (x)); > + for (i = GET_RTX_LENGTH (GET_CODE (x)) - 1; i >= 0; i--) > +{ > + if (fmt[i] == 'E') > + { > + int j; > + > + for (j = XVECLEN (x, i) - 1; j >= 0; j--) > + if (! is_legitimate_tls_operand (XVECEXP (x, i, j))) > +

Re: Patch: New GTY ((atomic)) option

2011-05-19 Thread Nicola Pero
>>        * gengtype.c (walk_type): Implemented "atomic" GTY option. >>        * doc/gty.texi (GTY Options): Document "atomic" GTY option. > > The patch is OK, with difference between "skip" and "atomic" options > documented. (Can be done as a follow-up patch). Thanks for the quick review. Here's

Re: New options to disable/enable any pass for any functions (issue4550056)

2011-05-19 Thread Xinliang David Li
On Thu, May 19, 2011 at 11:04 AM, Andi Kleen wrote: > davi...@google.com (David Li) writes: > >> -fdisable-tree-ccp1    <--- disable ccp1 for all functions >> -fenable-tree-cunroll=1   <--- enable complete unroll for the function >>                            whose cgraphnode uid is 1 >> -fdisable

Re: New options to disable/enable any pass for any functions (issue4550056)

2011-05-19 Thread Andi Kleen
davi...@google.com (David Li) writes: > -fdisable-tree-ccp1<--- disable ccp1 for all functions > -fenable-tree-cunroll=1 <--- enable complete unroll for the function >whose cgraphnode uid is 1 > -fdisable-rtl-gcse2=1:100,300,400:1000 <-- disable gcse2 for >

Re: [patch tree-ssa-reassoc.c]: Better reassoication for comparision and boolean-logic

2011-05-19 Thread Kai Tietz
To illustrate in which scenario code in tree-ssa-forwprop doesn't help is binop-tor4.c w/o this patch we get foo (int a, int b, int c) { int e; int d; int D.2701; _Bool D.2700; _Bool D.2699; _Bool D.2698; _Bool D.2697; _Bool D.2696; int D.2695; : D.2695_3 = b_2(D) | a_1(D);

Build more of libstdc++ exception throwing code with frame pointers (issue4539068)

2011-05-19 Thread Paul Pluzhnikov
2011-05-19 Paul Pluzhnikov * libstdc++-v3/libsupc++/Makefile.am: Add -fno-omit-frame-pointer to vterminate. * libstdc++-v3/libsupc++/Makefile.in: Regenerate. Index: libstdc++-v3/libsupc++/Makefile.in === -

Re: [build] Define HAVE_GAS_HIDDEN on Darwin

2011-05-19 Thread Rainer Orth
Mike Stump writes: > On May 4, 2011, at 5:08 AM, Rainer Orth wrote: >> The following patch is a prerequisite for making >> >> [lto, testsuite] Don't use visibility on targets that don't support it >> (PR lto/47334) >>http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00295.html > >> -# d

[v3] More noexcept work on , etc

2011-05-19 Thread Paolo Carlini
Hi, also exploiting the new std::is_nothrow_move_assignable, plus various fixes (added 2 testcases). Tested x86_64-linux, committed to mainline. Paolo. /// 2011-05-19 Paolo Carlini * include/std/tuple (tuple<>::operator=(tuple&&)): Specify as noexcept.

Re: [PATCH] fix vfmsubaddpd/vfmaddsubpd generation

2011-05-19 Thread Quentin Neill
On Wed, May 18, 2011 at 10:32 AM, Uros Bizjak wrote: > Hello! > >> This patch fixes an obvious problem: the fma4_fmsubadd/fma4_fmaddsub >> instruction templates don't generate vfmsubaddpd/vfmaddsubpd because >> they don't use >> >> This passes bootstrap on x86_64 on trunk.  Okay to commit? > > Se

Re: Make collect2 use common diagnostics code

2011-05-19 Thread Gabriel Dos Reis
On Thu, May 19, 2011 at 11:11 AM, Joseph S. Myers wrote: > This patch makes collect2 use the common diagnostics infrastructure, > instead of its own diagnostic functions.  (This is in preparation for > having a hooks structure shared between the driver and cc1.  Recall > that as described in

Make collect2 use common diagnostics code

2011-05-19 Thread Joseph S. Myers
This patch makes collect2 use the common diagnostics infrastructure, instead of its own diagnostic functions. (This is in preparation for having a hooks structure shared between the driver and cc1. Recall that as described in the idea is that coll

[RFA] [PR44618] [PowerPC] Wrong code for -frename-registers

2011-05-19 Thread edmar
The original patch posted in bugzilla was fully tested for 4.4/4.5/4.6 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44618 I am reposting it here as a reference for review. More recently, I re-tested it for 4.6, and it still pass with no regressions on powerpc (32 bits) If the general approach o

Re: Faster type merging pair cache

2011-05-19 Thread Richard Guenther
On Thu, 19 May 2011, Jan Hubicka wrote: > Hi, > type pair cache is compilation time hog just because of constant factors of > our > hashtable implementation as well as memory hog. This patch turns it into > simple > constantly sized hash with conflicts not handled (i.e. new pair kills the old >

Faster type merging pair cache

2011-05-19 Thread Jan Hubicka
Hi, type pair cache is compilation time hog just because of constant factors of our hashtable implementation as well as memory hog. This patch turns it into simple constantly sized hash with conflicts not handled (i.e. new pair kills the old pair). This seems to remove pair cache overhead out of

Fix typo in i386.md (tls_initial_exec_64_sun)

2011-05-19 Thread Rainer Orth
A recent patch broke bootstrap on Solaris/x86. Fixed as follows, bootstrapped on i386-pc-solaris2.11, installed on mainline. Rainer 2011-05-19 Rainer Orth * config/i386/i386.md (tls_initial_exec_64_sun): Add semicolon. Index: gcc/config/i386/i386.md

[build] Support COMDAT group with Sun as (PR target/40483), final version

2011-05-19 Thread Rainer Orth
This patch had long been approved in the original thread [build, c++, lto] Support COMDAT group with Sun as (PR target/40483) http://gcc.gnu.org/ml/gcc-patches/2010-05/msg01365.html http://gcc.gnu.org/ml/gcc-patches/2010-06/msg00600.html but installation had been stalled o

[PATCH] Add powi-to-multiply expansion to cse_sincos pass

2011-05-19 Thread William J. Schmidt
As Richard Guenther suggested, I'm submitting a separate patch to add powi expansion to multiplies during the cse_sincos pass. This was originally submitted as part of a larger patch to fix PR46728. Richard noted that tree_expand_builtin_powi, powi_as_mults, and powi_as_mults_1 more properly belo

Re: [patch tree-ssa-reassoc.c]: Better reassoication for comparision and boolean-logic

2011-05-19 Thread Kai Tietz
2011/5/19 Richard Guenther : > On Thu, May 19, 2011 at 3:36 PM, Kai Tietz wrote: >> 2011/5/19 Richard Guenther : >>> On Thu, May 19, 2011 at 3:30 PM, Kai Tietz wrote: 2011/5/19 Richard Guenther : > On Thu, May 19, 2011 at 3:08 PM, Kai Tietz > wrote: >> 2011/5/19 Richard Guenthe

Re: [PR testsuite/47013] Fix SMS testsuite faliures (re-submission)

2011-05-19 Thread Rainer Orth
Revital, you should seek review from a subject matter expert (Ayal, I suppose). Just a few nits on the testsuite side: > testsuite/Changelog > > PR rtl-optimization/47013 > * gcc.dg/sms-2.c: Change scan-tree-dump-times and the code itself > to preserve the function. >

Re: [patch tree-ssa-reassoc.c]: Better reassoication for comparision and boolean-logic

2011-05-19 Thread Richard Guenther
On Thu, May 19, 2011 at 3:36 PM, Kai Tietz wrote: > 2011/5/19 Richard Guenther : >> On Thu, May 19, 2011 at 3:30 PM, Kai Tietz wrote: >>> 2011/5/19 Richard Guenther : On Thu, May 19, 2011 at 3:08 PM, Kai Tietz wrote: > 2011/5/19 Richard Guenther : >> On Thu, May 19, 2011 at 2:59 PM,

Remove obfuscation from the scheduler

2011-05-19 Thread Bernd Schmidt
In the scheduler initialization code, we want to set up the h_i_d array. So we call haifa_init_h_i_d, which has a table of function pointers which it passes to sched_scan, which copies the function pointer table pointer to a global variable and then calls functions like init_insn, which checks the

Re: [patch tree-ssa-reassoc.c]: Better reassoication for comparision and boolean-logic

2011-05-19 Thread Kai Tietz
2011/5/19 Richard Guenther : > On Thu, May 19, 2011 at 3:30 PM, Kai Tietz wrote: >> 2011/5/19 Richard Guenther : >>> On Thu, May 19, 2011 at 3:08 PM, Kai Tietz wrote: 2011/5/19 Richard Guenther : > On Thu, May 19, 2011 at 2:59 PM, Kai Tietz > wrote: >> 2011/5/19 Richard Guenthe

Re: [patch tree-ssa-reassoc.c]: Better reassoication for comparision and boolean-logic

2011-05-19 Thread Richard Guenther
On Thu, May 19, 2011 at 3:30 PM, Kai Tietz wrote: > 2011/5/19 Richard Guenther : >> On Thu, May 19, 2011 at 3:08 PM, Kai Tietz wrote: >>> 2011/5/19 Richard Guenther : On Thu, May 19, 2011 at 2:59 PM, Kai Tietz wrote: > 2011/5/19 Richard Guenther : >> On Thu, May 19, 2011 at 2:48 PM,

Re: [patch tree-ssa-reassoc.c]: Better reassoication for comparision and boolean-logic

2011-05-19 Thread Kai Tietz
2011/5/19 Richard Guenther : > On Thu, May 19, 2011 at 3:08 PM, Kai Tietz wrote: >> 2011/5/19 Richard Guenther : >>> On Thu, May 19, 2011 at 2:59 PM, Kai Tietz wrote: 2011/5/19 Richard Guenther : > On Thu, May 19, 2011 at 2:48 PM, Kai Tietz > wrote: >> Hello, >> >> This

[PATCH][?/n] Cleanup LTO type merging

2011-05-19 Thread Richard Guenther
This moves all pointer-to, main-variant and canonical-type setup to a central place which happens after we fixed up and registered types for a set of new tree nodes. In particular delaying canonical type computation after setting up the pointer-to chains should enable a fix for PR48849 and its ev

Re: [patch, ARM] Fix PR42017, LR not used in leaf functions

2011-05-19 Thread Eric Botcazou
> > It seems a lot of other ports suffer from the same problem though. > > I wonder which targets really do want to make a register live throughout > > the function? If none do, perhaps we should say that this macro is > > only meaningful once the epilogue has been generated. > > To answer my own

[PATCH] Fix OpenMP return checking with lambda functions (PR c++/49043)

2011-05-19 Thread Jakub Jelinek
Hi! return is just fine in lambda functions and in methods in classes declared inside of OpenMP regions. Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, committed to trunk and 4.6 branch. 2011-05-19 Jakub Jelinek PR c++/49043 * decl.c (check_omp_return):

Re: [patch tree-ssa-reassoc.c]: Better reassoication for comparision and boolean-logic

2011-05-19 Thread Richard Guenther
On Thu, May 19, 2011 at 3:08 PM, Kai Tietz wrote: > 2011/5/19 Richard Guenther : >> On Thu, May 19, 2011 at 2:59 PM, Kai Tietz wrote: >>> 2011/5/19 Richard Guenther : On Thu, May 19, 2011 at 2:48 PM, Kai Tietz wrote: > Hello, > > This patch improves reassociation folding for com

Re: [patch tree-ssa-reassoc.c]: Better reassoication for comparision and boolean-logic

2011-05-19 Thread Kai Tietz
2011/5/19 Richard Guenther : > On Thu, May 19, 2011 at 2:59 PM, Kai Tietz wrote: >> 2011/5/19 Richard Guenther : >>> On Thu, May 19, 2011 at 2:48 PM, Kai Tietz wrote: Hello, This patch improves reassociation folding for comparision. It expands expressions within binary-AND/OR

[PATCH] Fix PR49063

2011-05-19 Thread Richard Guenther
I shouldn't have moved the test w/o re-testing. The following fixes it. Tested on x86_64-unknown-linux-gnu, applied. Richard. 2011-05-19 Richard Guenther PR testsuite/49063 * gcc.dg/builtin-object-size-11.c: Use -std=gnu89. Index: gcc/testsuite/gcc.dg/builtin-object-size-1

Re: [patch tree-ssa-reassoc.c]: Better reassoication for comparision and boolean-logic

2011-05-19 Thread Richard Guenther
On Thu, May 19, 2011 at 2:59 PM, Kai Tietz wrote: > 2011/5/19 Richard Guenther : >> On Thu, May 19, 2011 at 2:48 PM, Kai Tietz wrote: >>> Hello, >>> >>> This patch improves reassociation folding for comparision. It expands >>> expressions within binary-AND/OR expression like (X | Y) == 0 to (X ==

Re: [patch tree-ssa-reassoc.c]: Better reassoication for comparision and boolean-logic

2011-05-19 Thread Richard Guenther
On Thu, May 19, 2011 at 2:56 PM, Richard Guenther wrote: > On Thu, May 19, 2011 at 2:48 PM, Kai Tietz wrote: >> Hello, >> >> This patch improves reassociation folding for comparision. It expands >> expressions within binary-AND/OR expression like (X | Y) == 0 to (X == >> 0 && Y == 0) >> and (X |

Re: [patch tree-ssa-reassoc.c]: Better reassoication for comparision and boolean-logic

2011-05-19 Thread Kai Tietz
2011/5/19 Richard Guenther : > On Thu, May 19, 2011 at 2:48 PM, Kai Tietz wrote: >> Hello, >> >> This patch improves reassociation folding for comparision. It expands >> expressions within binary-AND/OR expression like (X | Y) == 0 to (X == >> 0 && Y == 0) >> and (X | Y) != 0 to (X != 0 || Y != 0)

Re: [patch tree-ssa-reassoc.c]: Better reassoication for comparision and boolean-logic

2011-05-19 Thread Richard Guenther
On Thu, May 19, 2011 at 2:48 PM, Kai Tietz wrote: > Hello, > > This patch improves reassociation folding for comparision. It expands > expressions within binary-AND/OR expression like (X | Y) == 0 to (X == > 0 && Y == 0) > and (X | Y) != 0 to (X != 0 || Y != 0).  This is necessary to allow > bette

Re: [patch tree-ssa-reassoc.c]: Better reassoication for comparision and boolean-logic

2011-05-19 Thread Kai Tietz
2011/5/19 Jakub Jelinek : > On Thu, May 19, 2011 at 02:48:08PM +0200, Kai Tietz wrote: >> ChangeLog >> >>       * gcc.dg/binop-tand1.c >>       * gcc.dg/binop-tand2.c >>       * gcc.dg/binop-tand3.c >>       * gcc.dg/binop-tand4.c >>       * gcc.dg/binop-tor1.c >>       * gcc.dg/binop-tor2.c >>    

Re: [PATCH] Handle flexible arrays in builtin-object-size

2011-05-19 Thread H.J. Lu
On Thu, May 19, 2011 at 3:44 AM, Richard Guenther wrote: > > This patch makes us deal with flexible array members and global > initializers correctly, thus for > > struct s { >    int i; >    char c[]; > } s = { 1, "01234" }; > > return 6 for the size of s.c instead of 0 (which is wrong, it > shou

Re: [patch tree-ssa-reassoc.c]: Better reassoication for comparision and boolean-logic

2011-05-19 Thread Jakub Jelinek
On Thu, May 19, 2011 at 02:48:08PM +0200, Kai Tietz wrote: > ChangeLog > > * gcc.dg/binop-tand1.c > * gcc.dg/binop-tand2.c > * gcc.dg/binop-tand3.c > * gcc.dg/binop-tand4.c > * gcc.dg/binop-tor1.c > * gcc.dg/binop-tor2.c > * gcc.dg/binop-tor3.c > * g

[patch tree-ssa-reassoc.c]: Better reassoication for comparision and boolean-logic

2011-05-19 Thread Kai Tietz
Hello, This patch improves reassociation folding for comparision. It expands expressions within binary-AND/OR expression like (X | Y) == 0 to (X == 0 && Y == 0) and (X | Y) != 0 to (X != 0 || Y != 0). This is necessary to allow better reassociation on weak pre-folded logical expressions. This un

Re: [PATCH] Fix PR target/48807

2011-05-19 Thread Henderson, Stuart
Ping http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00503.html Hi, The attached patch is a fix for PR/48807, which is a segfault when compiling the bfin compiler. The problem appears to be that we're not checking the return value of cgraph_local_info for null before attempting to dereference it.

Re: [patch][combine] Fix 16-bit -> 64-bit multiply and accumulate

2011-05-19 Thread Bernd Schmidt
On 05/18/2011 11:55 AM, Andrew Stubbs wrote: > > case SIGN_EXTEND: > + op = make_compound_operation (op, SET); > + I would have expected that bit to go into the MULT case (doing it for both operands), so that the effect is limited to just making widening multiplies more obvious. Also,

Re: [patch gimplifier]: Change TRUTH_(AND|OR|XOR) expressions to binary form

2011-05-19 Thread Kai Tietz
2011/5/19 Richard Guenther : > On Thu, May 19, 2011 at 12:45 PM, Eric Botcazou wrote: >>> Eric, how will this interact with Ada boolean types with 8 bit precision >>> and those "invalid" values - will invalid values ever enter into >>> expressions or are they checked before? >> >> IIUC the Ada sem

Re: [SPARC] Hookize PRINT_OPERAND, PRINT_OPERAND_ADDRESS and PRINT_OPERAND_PUNCT_VALID_P

2011-05-19 Thread Rainer Orth
Hi Anatoly, >> 2011-05-05 Rainer Orth > >> * config/sparc/sol2.h (ASM_OUTPUT_CALL): Use >> targetm.asm_out.print_operand. >> * config/sol2.c: Include target.h. > > This patch adds a missing dependency to config/t-sol2. > > Patch is untested. > > OK to install? I'

Re: [patch gimplifier]: Change TRUTH_(AND|OR|XOR) expressions to binary form

2011-05-19 Thread Richard Guenther
On Thu, May 19, 2011 at 12:45 PM, Eric Botcazou wrote: >> Eric, how will this interact with Ada boolean types with 8 bit precision >> and those "invalid" values - will invalid values ever enter into >> expressions or are they checked before? > > IIUC the Ada semantics is that they must be checked

Re: [patch gimplifier]: Change TRUTH_(AND|OR|XOR) expressions to binary form

2011-05-19 Thread Eric Botcazou
> Eric, how will this interact with Ada boolean types with 8 bit precision > and those "invalid" values - will invalid values ever enter into > expressions or are they checked before? IIUC the Ada semantics is that they must be checked prior to being used. Once they are used, they are "recycled"

[PATCH] Handle flexible arrays in builtin-object-size

2011-05-19 Thread Richard Guenther
This patch makes us deal with flexible array members and global initializers correctly, thus for struct s { int i; char c[]; } s = { 1, "01234" }; return 6 for the size of s.c instead of 0 (which is wrong, it should have been reported as -1 instead - I didn't try to fix that particular i

Re: [Patch, Fortran] Implement/fix cobounds for scalar coarrays on tree level

2011-05-19 Thread Tobias Burnus
*ping* http://gcc.gnu.org/ml/fortran/2011-05/msg00123.html On 05/16/2011 12:00 AM, Tobias Burnus wrote: The patch is a follow-up to the patch at http://gcc.gnu.org/ml/fortran/2011-05/msg00067.html. With that patch, all* my coarray example compile (and run with -fcoarray=single or -fcoarray=li

[PATCH][?/n] LTO type merging cleanup

2011-05-19 Thread Richard Guenther
This fixes a bug with the previous recursion inhibit of gimple_register_type. It also re-introduces the patch that makes us compare names not of the main-variant but of the type itself. And finally it makes us produce more stable hashes by hashing the types in the tree SCC as it was read in orig

Re: [Patch, fortran] Update documentation and error messages for -ffpe-trap

2011-05-19 Thread Tobias Burnus
On 05/18/2011 10:12 PM, Janne Blomqvist wrote: Regtested on x86_64-unknown-linux-gnu, Ok for trunk? OK. Thanks for the patch! Tobias frontend ChangeLog: 2011-05-18 Janne Blomqvist * gfortran.texi (set_fpe): Update documentation. * invoke.texi (-ffpe-trap): Likewise.

Re: [patch gimplifier]: Change TRUTH_(AND|OR|XOR) expressions to binary form

2011-05-19 Thread Richard Guenther
On Wed, May 18, 2011 at 8:30 PM, Kai Tietz wrote: > 2011/5/18 Kai Tietz : >> Hello >> >> As follow-up for logical to binary transition >> >> 2011-05-18  Kai Tietz   >> >>        * tree-cfg.c (verify_gimple_assign_binary): Barf on >>        TRUTH_AND_EXPR, TRUTH_OR_EXPR, and TRUTH_XOR_EXPR. >>    

[PATCH PR45098, 1/10] Proc object-size fix

2011-05-19 Thread Tom de Vries
On 05/17/2011 09:14 AM, Tom de Vries wrote: > On 05/17/2011 09:10 AM, Tom de Vries wrote: >> Hi Zdenek, >> >> I have a patch set for for PR45098. >> >> 01_object-size-target.patch >> 02_pr45098-rtx-cost-set.patch >> 03_pr45098-computation-cost.patch >> 04_pr45098-iv-init-cost.patch >> 05_pr45098-bo

Unreviewed build patches

2011-05-19 Thread Rainer Orth
The following two build-related patches have remained unreviewed for 2 or more weeks: [build, lto] Check plugin linker for level of plugin support http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00226.html Embedded in a different thread, but reminded 4 times now and all open question

[testsuite] Don't XFAIL gcc.c-torture/execute/960321-1.c on 64-bit Solaris 10+/x86

2011-05-19 Thread Rainer Orth
With the latest versions of Solaris 10 (patch 119961-10) and 11 (snv_164), Sun bug 7012583 fbe/as emits invalid R_AMD64_32 relocation has finally been fixed and gcc.c-torture/execute/960321-1.c doesn't fail any longer on 64-bit Solaris/x86. The corresponding .x file can thus be removed, which th

New Chinese (simplified) PO file for 'gcc' (version 4.6.0)

2011-05-19 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'gcc' has been submitted by the Chinese (simplified) team of translators. The file is available at: http://translationproject.org/latest/gcc/zh_CN.po (This file, 'gcc-4.6.0.

Re: [PATCH, SMS] Fix calculation of issue_rate

2011-05-19 Thread Revital Eres
Hello, > > * modulo-sched.c (sms_schedule): Fix stage_count calculation. > Sorry, just noticed that the changelog entry is wrong, here a correction: * modulo-sched.c (sms_schedule): Fix issue_rate calculation. Thanks, Revital