[committed] PR 51931: force non-MIPS16ness for long-branch tests

2012-01-22 Thread Richard Sandiford
The MIPS16 port has never handled long branches properly; see PR 51931 for the details. It isn't easy to xfail MIPS16-specific problems at the dejagnu level because of -mflip-mips16, so the patch below forces a nomips16 attribute instead. Tested on mips64-linux-gnu and applied. Richard gcc/tes

Re: [RFC combine] PR48308 - Fix issue with missing(?) LOG_LINKS

2012-01-22 Thread Eric Botcazou
> The un-combining thing around line 2800 is somewhat of a kludge and I > wouldn't be surprised if there were other fallouts. But your change is > clearly correct and looks relatively safe, so OK for trunk and 4.6 branch > after full testing. I overlooked something though: it might be possible fo

Ping^4: Out-of-order update of new_spill_reg_store[]

2012-01-22 Thread Richard Sandiford
Ping for this reload patch: http://gcc.gnu.org/ml/gcc-patches/2011-12/msg00266.html or perhaps the original: http://gcc.gnu.org/ml/gcc-patches/2011-10/msg00663.html which fixes some wrong-code regressions on mips64-linux-gnu. Richard

Re: [PATCH] PR51280 LTO/trans-mem ICE with TM builtins

2012-01-22 Thread Dominique Dhumieres
> Dominique or Iain, may I ask you to test this patch on darwin? I have a > sporadic access to a darwin machine. The patch fixes the PR without regression on x86_64-apple-darwin10. Bootstraping on powerpc-apple-darwin9 will finish in a couple hours. Thanks for the debugging and the patch, Domini

Re: [PATCH] PR51280 LTO/trans-mem ICE with TM builtins

2012-01-22 Thread Iain Sandoe
Hi Patrick, thanks for doing this, On 21 Jan 2012, at 22:57, Patrick Marlier wrote: Dominique or Iain, may I ask you to test this patch on darwin? I have a sporadic access to a darwin machine. bootstrapped on i686-darwin9 - tests in progress... minor observations: PR lto/51916

Re: [PATCH] Prevent 'control reaches end of non-void function' warning for DO_WHILE

2012-01-22 Thread Tom de Vries
On 09/12/11 10:45, Richard Guenther wrote: > On Fri, Dec 9, 2011 at 9:38 AM, Tom de Vries wrote: >> Jakub, >> >> This patch fixes the problem reported in >> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25973#c4 . >> >> The test-case listed there is: >> ... >> struct Block >> { >> public: >>Blo

Re: [PATCH] RFC: Interix resurrection

2012-01-22 Thread Paolo Bonzini
On 01/01/2012 10:01 PM, Douglas Rupp wrote: -i[[34567]]86-*-interix3*) +i[[34567]]86-*-interix[[3-9]]*) # Interix 3.x gcc -fpic/-fPIC options generate broken code. # Instead, we relocate shared libraries at runtime. ;; Is this still true? Build parts are ok. Pa

Re: varpool PATCH for c++/51832 (multiply defined symbol with LTO plugin)

2012-01-22 Thread Richard Guenther
On Fri, Jan 20, 2012 at 8:59 PM, Jason Merrill wrote: > As a result of my recent mangling work, any code that uses variadic > templates will use the forward-compatibility aliases because the mangling of > argument packs has changed.  This exposed an issue that we weren't copying > DECL_EXTERNAL fr

Re: [ C frontend ] add support for builtins that take signed and unsigned md types

2012-01-22 Thread Richard Guenther
On Sat, Jan 21, 2012 at 12:21 AM, Mike Stump wrote: > unsigned builtins don't always work, as the type matching code can ignore the > signed/unsigned variants when searching for a type with a particular mode.   > This patch fixes this problem.  If a type with same unsignededness exists, we > pre

Re: [PATCH] RFC: Interix resurrection

2012-01-22 Thread Uros Bizjak
Hello! > Ping. > Patch retested with 1/18/2012 sources. > >> On 1/1/2012 1:01 PM, Douglas Rupp wrote: >> >>This is for 32bit, "C" language only. >>Tested by cross and then cross-native from x86_64 Linux > Build parts are ok. Looking at the patch, I didn't spotted anything obviously wrong

Fix PR rtl-optimization/51924

2012-01-22 Thread Eric Botcazou
This is another problem uncovered by the generalization of the ZEE pass: when the definition insn of an extension is itself an extension, modifying the insn for the first extension breaks the handling of the second extension. This was silent in ZEE because you never have 2 identical successive

Partial fix for LTO bootstrap with Ada

2012-01-22 Thread Eric Botcazou
The LTO bootstrap of the Ada compiler is currently plagued by at least three different problems. The attached patch is for the easy one: it prevents fake variables whose type contains a placeholder from being created, which doesn't work in LTO mode because you cannot instantiate the PLACEHOLDER

Fix flag_trapping_math in java frontend

2012-01-22 Thread Andreas Schwab
The java frontend wants that floating point operations are assumed to never trap, thus clears flag_trapping_math in java_init_options_struct. But this is no longer effective after revision 165823, because set_fast_math_flags is now always called during option processing, overriding anything what th

[Patch, Fortran] PR41600 - fix ICE with type conversion in default initializer

2012-01-22 Thread Tobias Burnus
Dear all, the middle end does not like to fold_convert a real number to an integer, but gfortran does not type-convert the expressions of initialization expressions. This patch fixes the issue - and thus part of of the issue of the PR. Build and regtested on x86-64-linux. OK for the trunk?

Re: [Patch, Fortran] PR41600 - fix ICE with type conversion in default initializer

2012-01-22 Thread Tobias Burnus
Tobias Burnus wrote: the middle end does not like to fold_convert a real number to an integer, but gfortran does not type-convert the expressions of initialization expressions. This patch fixes the issue And Tobias Schlüter wrote in the PR: (In reply to comment #3) + if (comp->ts.typ

Re: [ C frontend ] add support for builtins that take signed and unsigned md types

2012-01-22 Thread Mike Stump
On Jan 22, 2012, at 4:53 AM, Richard Guenther wrote: > Why add a new loop? It seems to be bogus to not check signedness in > the existing loop (mind that for fixed-point types you need to check > saturating/nonsaturating flag. We can remove the other loop, if people don't want any of the old sem

[patch PR 51900]: Fix PE's delegitimize_address functions

2012-01-22 Thread Kai Tietz
Hello, this patch fixes reported issue in PR about common-symbols and fix behavior about -fcommon/-fno-common. Additionally it adds proper support for weakref, local-variant of weaks, and tries to handle resolution-file information for PE-COFF. I did regression tests of all standard-languages fo

Re: [patch PR 51900]: Fix PE's delegitimize_address functions

2012-01-22 Thread Jakub Jelinek
On Sun, Jan 22, 2012 at 08:05:12PM +0100, Kai Tietz wrote: > this patch fixes reported issue in PR about common-symbols and fix > behavior about -fcommon/-fno-common. Additionally it adds proper > support for weakref, local-variant of weaks, and tries to handle > resolution-file information for PE

Re: [patch PR 51900]: Fix PE's delegitimize_address functions

2012-01-22 Thread Kai Tietz
2012/1/22 Jakub Jelinek : > On Sun, Jan 22, 2012 at 08:05:12PM +0100, Kai Tietz wrote: >> this patch fixes reported issue in PR about common-symbols and fix >> behavior about -fcommon/-fno-common.  Additionally it adds proper >> support for weakref, local-variant of weaks, and tries to handle >> re

Re: [patch PR 51900]: Fix PE's delegitimize_address functions

2012-01-22 Thread Kai Tietz
2012/1/22 Kai Tietz : > 2012/1/22 Jakub Jelinek : >> On Sun, Jan 22, 2012 at 08:05:12PM +0100, Kai Tietz wrote: >>> this patch fixes reported issue in PR about common-symbols and fix >>> behavior about -fcommon/-fno-common.  Additionally it adds proper >>> support for weakref, local-variant of weak

[Patch, fortran] PR51870 - [OOP] ICE with ALLOCATE and SOURCE-expr function returning BT_CLASS

2012-01-22 Thread Paul Richard Thomas
Dear All, The attached is quite straightforward - for non-variable class STATUS expressions, the class object is extracted, together with the element size for the dynamic type. These are then used for the allocation and the copy of the source data into the allocated object. Note that I have begg

Re: [PATCH] RFC: Interix resurrection

2012-01-22 Thread Douglas Rupp
On 1/22/2012 4:41 AM, Paolo Bonzini wrote: On 01/01/2012 10:01 PM, Douglas Rupp wrote: -i[[34567]]86-*-interix3*) +i[[34567]]86-*-interix[[3-9]]*) # Interix 3.x gcc -fpic/-fPIC options generate broken code. # Instead, we relocate shared libraries at runtime. ;; Is thi

Re: Commit: RX: Add return pattern

2012-01-22 Thread Richard Henderson
On 01/13/2012 04:31 AM, nick clifton wrote: > Hi Richard, > >> Not an ideal solution, since the availability of this pattern implies >> it's extremely cheap, and we'll replace jumps to the epilogue with >> this pattern. >> Or to define an availability predicate similar to i386, testing if >> the e

Re: [Patch, fortran] PR51870 - [OOP] ICE with ALLOCATE and SOURCE-expr function returning BT_CLASS

2012-01-22 Thread Tobias Burnus
Dear Paul, Paul Richard Thomas wrote: The attached is quite straightforward - for non-variable class STATUS expressions, I believe that you mean source-expr (i.e. SOURCE= and MOLD=) and not STATUS. Note that I have begged off including variables in this process, given the stage that we are

Re: Add -lssp_nonshared to LINK_SSP_SPEC

2012-01-22 Thread Gerald Pfeifer
On Sat, 21 Jan 2012, Tijl Coosemans wrote: > I've been using this patch now. It's similar to the above url, but > conditional on TARGET_LIBC_PROVIDES_SSP to support older FreeBSD > versions. > > Gerald volunteered to commit. Gerald, just trunk for now or older > branches too? If Richi agries, I'd

Re: [PATCH, trans-mem]: Fix PR51830, FAIL: libitm.c/mem(cpy|set)-1.c execution test on x86_32

2012-01-22 Thread Richard Henderson
On 01/19/2012 06:26 AM, Uros Bizjak wrote: > 2012-01-18 Uros Bizjak > > PR libitm/51830 > * builtin-types.def (BT_FN_UINT_UINT_VAR): New. > * gtm-builtins.def (BUILT_IN_TM_START): Declare as BT_FN_UINT_UINT_VAR. > > libitm/ChangeLog: > > 2012-01-18 Uros Bizjak > >

Re: [PATCH, trans-mem]: Fix PR51830, FAIL: libitm.c/mem(cpy|set)-1.c execution test on x86_32

2012-01-22 Thread Richard Henderson
On 01/20/2012 12:51 AM, Uros Bizjak wrote: > OTOH, in GTM_beginTransaction we can > still access other variable arguments through the pointer to CFA. Well, no, not really. If we really want GTM_beginTransaction to have access to the variadic portions, we'll need to have the sjlj stub pass in a v

Re: [libitm, build] Support sun symbol versioning

2012-01-22 Thread Richard Henderson
On 01/17/2012 03:41 AM, Rainer Orth wrote: > 2012-01-14 Rainer Orth > > * libitm.h (_ITM_getThreadnum): Remove. > * libitm.map (_ITM_getThreadnum): Remove from LIBITM_1.0. Ok. r~

Re: [libitm, build] Clear hardware capabilities on libitm.so with Sun ld

2012-01-22 Thread Richard Henderson
On 01/17/2012 03:36 AM, Rainer Orth wrote: > 2011-11-18 Rainer Orth > > * clearcap.map: New file. > * acinclude.m4 (LIBITM_CHECK_LINKER_HWCAP): New test. > * configure.ac: Call it. > Clear HWCAP_LDFLAGS if defaulting to -mavx. > * Makefile.am (AM_LDFLAGS): Add $(HW

Re: [libitm] Link eh-1.exe with -shared-libgcc on Solaris (PR libitm/51822)

2012-01-22 Thread Richard Henderson
On 01/17/2012 04:07 AM, Rainer Orth wrote: > * The 32-bit failures on Solaris 8 to 10 have a different root cause: > _Unwind_Find_FDE returns NULL for an address in _ZGTtL2f1v (f1()). It > turns out that there are two copies of the unwinder in eh-1.exe: one > from libgcc_s.so.1, and another

Re: [wwwdocs] List new Fortran options under Fortran section

2012-01-22 Thread Gerald Pfeifer
On Wed, 18 Jan 2012, Tobias Burnus wrote: > Looks OK to me. Though, I had made the N and M italic at: > +-freal-N-real-M have been added. These options promote > and > +N andM can be 4 and 8 and, if available, Something like -freal-N-real-M have been added. These options promote and N

[PATCH] Fix PR 29333: Jump threading getting in the way of PHI-OPT

2012-01-22 Thread Andrew Pinski
Hi, The problem here is not really jump threading getting in the way of PHI-OPT rather there is no cleanup of the IR after jump thread but before phi-opt. This patch adds a pass_phi_only_cprop right after the first vrp and pass_merge_phi right before the first and last phiopts. Since VRP does ju

Re: [PATCH] RFC: Interix resurrection

2012-01-22 Thread Paolo Bonzini
On 01/22/2012 10:16 PM, Douglas Rupp wrote: May I regenerate libiberty/configure and check it in, or does that require a libiberty maintainer? Yes, regenerate everything that is affected. Paolo