Re: Remove obsolete Tru64 UNIX V5.1B support

2012-03-06 Thread Tristan Gingold
On Mar 5, 2012, at 6:28 PM, Richard Henderson wrote: > On 03/05/2012 09:14 AM, Rainer Orth wrote: >> * In the alpha backend, there are a couple of cases that might be >> osf-specific, but I cannot tell for certain: >> >> macro osf5.halpha.h >> >> TARGE

Re: Remove obsolete Tru64 UNIX V5.1B support

2012-03-06 Thread Arnaud Charlet
The gnattools and gcc/ada parts are OK, except for the comment removal in s-tassta.adb: this comment is still useful, and needs to be revisited at some point ratheer than removed silently as you did, to understand why we can't use a when E: others => construct. So either remove the s-tassta.adb hu

Re: Remove obsolete Tru64 UNIX V5.1B support

2012-03-06 Thread Tristan Gingold
Rainer, On Mar 5, 2012, at 6:14 PM, Rainer Orth wrote: > The Tru64 UNIX V5.1 port has been obsoleted in GCC 4.7, and it's now > time to remove it from mainline. The following patch does just that, > and should be mostly uncontroversial, like removing target-specific > fixincludes hacks, files,

Re: [patch] Clean up f95-lang.c a bit

2012-03-06 Thread Tobias Burnus
On 03/06/2012 12:30 AM, Steven Bosscher wrote: This is a long-overdue cleanup of f95-lang.c. This file was once added as an almost one-to-one copy from one of the other languages and tweaked until it worked. But the comments in the file are misleading, out-dated, or wrong for other reasons, and t

Re: [PATCH][target/52481] m68k-*: internal compiler error: in extract_insn, at recog.c:2123

2012-03-06 Thread Ralf Corsepius
On 03/05/2012 07:44 PM, Richard Henderson wrote: --- Comment #1 from Mikael Pettersson 2012-03-04 21:01:28 UTC --- Created attachment 26827 --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=26827 reduced test case in C Depends on target CPU selection. -mcpu=680[012346]0 and -mcpu=cpu32 al

Re: [RFC PATCH]: Handle Pmode == SImode in stringop patterns

2012-03-06 Thread Uros Bizjak
On Mon, Mar 5, 2012 at 11:42 PM, H.J. Lu wrote: +     case '^': +       if (TARGET_64BIT && Pmode == SImode) +         { +           fputs ("addr32", file); +#ifndef HAVE_AS_IX86_REP_LOCK_PREFIX +           if (ASSEMBLER_DIALECT == ASM_ATT) +             fputs ("

Re: [patch] PR 51417

2012-03-06 Thread Richard Guenther
On Mon, 5 Mar 2012, Ralf Corsépius wrote: > Hi, > > The patch below addresses an issue with gcc-4.7.0 the issue I had reported in > http://gcc.gnu.org/ml/gcc/2012-03/msg00035.html > > and somebody else had bz'ed as > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51417 > > Tested by cross-building

[PATCH] Fix PR52097

2012-03-06 Thread Richard Guenther
This is another fix I postponed for 4.8 (or 4.7.1 if it doesn't have any fallout). We fail to honor the (unwritten) rule that type variants ought to share their TYPE_FIELDs. Which breaks users that compare FIELD_DECLs by pointers. This should also save some memory at WPA/LTRANS stage. LTO boot

Re: [Patch,AVR,trunk,4.7] PR52461: Fix RAMPZ clobbering and RAMP* in epilogue

2012-03-06 Thread Richard Guenther
On Mon, Mar 5, 2012 at 6:54 PM, Georg-Johann Lay wrote: > Richard Guenther wrote: >> On Mon, Mar 5, 2012 at 6:15 PM, Georg-Johann Lay wrote: >>> Richard Guenther wrote: On Mon, Mar 5, 2012 at 4:56 PM, Georg-Johann Lay wrote: > Richard Guenther wrote: >> On Mon, Mar 5, 2012 at 4:25 P

Re: Delete dead (?) code in expand_expr_real_2

2012-03-06 Thread Richard Guenther
On Tue, Mar 6, 2012 at 12:51 AM, Richard Kenner wrote: >> I found a weird piece of code that was added by kenner in a really early >> revision. It checks for VAR_DECLs with frame or stack pointers as >> DECL_RTL, and the comment in front of it mentions strength reduction. >> Presumably this was fo

Re: [PATCH, 4.4, PR 52430] IPA-CP has to clone or leave alone externally_visible nodes

2012-03-06 Thread Martin Jambor
Hi, On Fri, Mar 02, 2012 at 06:54:13PM +1300, Maxim Kuvyrkov wrote: > On 2/03/2012, at 1:53 AM, Martin Jambor wrote: > > > Hi, > > > > the following patch backports 4.5 behavior to 4.4 IPA-CP > > initialization when it decides which nodes need to be duplicated (we > > often say cloned, but cloni

[VMS]: override version in t-vmsnative

2012-03-06 Thread Tristan Gingold
Hi, as a hack, we need to redefined the makefile variable 'version' so that the version subdir is 4_X_Y, because VMS doesn't like dots in directory names. But we need to do that only if we build for VMS, not for cross compilers. Also, remove the stamp declarations that prevented some run of fi

Re: [PATCH][target/52481] m68k-*: internal compiler error: in extract_insn, at recog.c:2123

2012-03-06 Thread Andreas Schwab
Ralf Corsepius writes: > * Incompatiblible/insufficient binutils (I am using binutils-2.22)? > * Some as/ld-flags getting lost/improperly propagated during > assembly/linking? > * Bad/incompatible inline-asm hard-coded somewhere? TAS needs at least ISA_B, ie. at least CFV4. Andreas. -- Andrea

[VMS] rename PRNTF to LDBL in crtl map

2012-03-06 Thread Tristan Gingold
Hi, the name PRNTF was not well chosen, and I prefer to rename it to LDBL to make it clear that it refers to long double. Applied on trunk. Tristan. 2012-03-06 Tristan Gingold * config/vms/vms.c (VMS_CRTL_LDBL): Rename from VMS_CRTL_PRNTF. * config/vms/vms-crtlmap.map: Rena

[VMS] minor updates to vms-c.c

2012-03-06 Thread Tristan Gingold
Hi, this patch: * handles 'octaword' alignment in pragma __nomember_alignment. * handles 'relaxed_refdef' in pragma __extern_model. * allow preprocessor expansion in pragma nomember_alignment. * improve the documentation of external model. Committed on trunk. Tristan. 2012-03-06 Tristan Gingo

[PATCH] Fix PR52493

2012-03-06 Thread Richard Guenther
This fixes PR52493 by robustifying the ptr_derefs_may_alias_p code. It should handle being passed &MEM[0 + 0] (which is a missed folding, to be addressed by a separate patch). The simplification code always should recurse, we failed to do so for the &MEM_REF case. Bootstrap & regtest pending on

[PATCH] Properly fold &MEM[0 + 0]

2012-03-06 Thread Richard Guenther
This adds folding of &MEM[CST + CST] to build_fold_addr_expr_with_type_loc. Bootstrap and regtest pending on x86_64-unknown-linux-gnu. Richard. 2012-03-06 Richard Guenther * fold-const.c (build_fold_addr_expr_with_type_loc): Fold MEM_REF with constant pointer operand. Inde

Re: [Patch,AVR,trunk,4.7] PR52461: Fix RAMPZ clobbering and RAMP* in epilogue

2012-03-06 Thread Georg-Johann Lay
Richard Guenther wrote: > On Mon, Mar 5, 2012 at 6:54 PM, Georg-Johann Lay wrote: >> Richard Guenther wrote: >>> On Mon, Mar 5, 2012 at 6:15 PM, Georg-Johann Lay wrote: Richard Guenther wrote: > On Mon, Mar 5, 2012 at 4:56 PM, Georg-Johann Lay wrote: >> Richard Guenther wrote: >

Re: Delete dead (?) code in expand_expr_real_2

2012-03-06 Thread Richard Kenner
> I don't see how a VAR_DECL can ever get a DECL_RTL equal to one of > the mentioned regs. Doesn't that happen when you have a local variable that's a variable-sized object? What would have changed that would cause it to no longer happen? This is tree-level stuff, not RTL. > The patch is ok if a

Re: Delete dead (?) code in expand_expr_real_2

2012-03-06 Thread Jakub Jelinek
On Tue, Mar 06, 2012 at 06:17:52AM -0500, Richard Kenner wrote: > > I don't see how a VAR_DECL can ever get a DECL_RTL equal to one of > > the mentioned regs. > > Doesn't that happen when you have a local variable that's a > variable-sized object? What would have changed that would cause it to >

Re: [Patch,AVR,trunk,4.7] PR52461: Fix RAMPZ clobbering and RAMP* in epilogue

2012-03-06 Thread Richard Guenther
On Tue, Mar 6, 2012 at 12:13 PM, Georg-Johann Lay wrote: > Richard Guenther wrote: >> On Mon, Mar 5, 2012 at 6:54 PM, Georg-Johann Lay wrote: >>> Richard Guenther wrote: On Mon, Mar 5, 2012 at 6:15 PM, Georg-Johann Lay wrote: > Richard Guenther wrote: >> On Mon, Mar 5, 2012 at 4:56

Re: Delete dead (?) code in expand_expr_real_2

2012-03-06 Thread Richard Kenner
> VLA VAR_DECLs have just DECL_HAS_VALUE_EXPR_P set and DECL_VALUE_EXPR being > INDIRECT_REF (or MEM_REF now?) dereferencing some DECL_ARTIFICIAL VAR_DECL > that is initialized from alloca builtin.So the VLA VAR_DECLs don't have > any DECL_RTL at all (kept for debug info purposes only), and the

[Patch PING] Re: [4.8, C++ Patch] PR 52422

2012-03-06 Thread Paolo Carlini
Hi just rediffed and retested on x86_64-linux the new version of the patch. Is it Ok for mainline? Thanks, Paolo. /// /cp 2012-03-06 Paolo Carlini PR c++/52422 * cp-tree.h (build_addr_func, decay_conversion, build_m_component_ref): Add tsubst_fl

Re: [PATCH, i386] RTM support

2012-03-06 Thread Kirill Yukhin
> I'd suggest you generate local label in the expander and pass it to > insn RTX. This way, we can also reuse insn pattern later with eventual > different code label. Thanks! Done. New patch attached. Updated changelogs: ChangeLog: 2012-02-16 Kirill Yukhin * common/config/i386/i386-co

Re: [PATCH, i386] RTM support

2012-03-06 Thread Uros Bizjak
On Tue, Mar 6, 2012 at 12:59 PM, Kirill Yukhin wrote: >> I'd suggest you generate local label in the expander and pass it to >> insn RTX. This way, we can also reuse insn pattern later with eventual >> different code label. > > Thanks! Done. > > New patch attached. > Updated changelogs: > > Change

Re: [PATCH] Partially fix 51988: value_replacement in PHIOPT should handle even the cases where there are other PHIs even with non equal value

2012-03-06 Thread Richard Guenther
On Wed, Jan 25, 2012 at 6:36 AM, Andrew Pinski wrote: > Hi, >  value_replacement in PHIOPT currently works only when there is one > PHI (which is non virtual). > http://gcc.gnu.org/ml/gcc-patches/2012-01/msg01195.html improves the > situation but we can improve it even more as replacing a PHI argu

[Patch,AVR]: Fix PR52506

2012-03-06 Thread Georg-Johann Lay
The problem is that RAMPD/X/Y/Z are restored in the wrong order in ISR epilogue. Ok for trunk? Johann PR target/52506 * gcc/config/avr/avr.c (expand_epilogue): Fix order of restoration to: RAMPZ, RAMPY, RAMPX, RAMPD. (expand_prologue): Only clear RAMPZ if it has e

Re: [PATCH, i386] RTM support

2012-03-06 Thread Kirill Yukhin
Updated patch attached. > > Technically OK, but let's wait for rth's comments about -mrtm option. Thanks! Let's wait then. > >       break; > +    case INT_FTYPE_VOID: > > Please add vertical space. > Added. > +(define_expand "xbegin" > +  [(set (match_operand:SI 0 "register_operand" "=a") > +  

[RFC]: Add support for pragma pointer_size

2012-03-06 Thread Tristan Gingold
Hi, this patch has some impacts to the C front-end, and I'd like to know wether my approach is correct before finalizing it. So comments are welcome. It implements the VMS 'pragma pointer_size 32/64/short/long', which change the default size of a pointer. This feature allows to use a 32 bits

Re: [PATCH] Improve SCEV for array element

2012-03-06 Thread Richard Guenther
On Fri, Jan 20, 2012 at 10:06 AM, Jiangning Liu wrote: >> It's definitely not ok at this stage but at most for next stage1. > > OK. I may wait until next stage1. > >> This is a very narrow pattern-match.  It doesn't allow for &a[i].x for >> example, >> even if a[i] is a one-element structure.  I t

Re: Remove obsolete Tru64 UNIX V5.1B support

2012-03-06 Thread Rainer Orth
Jonathan Wakely writes: > On 5 March 2012 17:01, Rainer Orth wrote: >> >> * The libstdc++ testsuite is messy since every thing pthread test >>  includes the complete list of targets where it should be run, and the >>  options required.  I've long meant to clean this up, but this will >>  have to

Re: Remove obsolete Tru64 UNIX V5.1B support

2012-03-06 Thread Rainer Orth
"Joseph S. Myers" writes: >> There's one particular issue: the change to java/io/File.java required >> my to regenerate the .class file in classpath. I've used Sun javac >> -target 1.5 for that and hope I got it right. > > I'd have expected regeneration to use GCJ built to use ECJ, though

[Patch,AVR]: Fix PR52507

2012-03-06 Thread Georg-Johann Lay
This fixes a wrong loop label in the movmemhi worker for the __memx address space case. Ok for trunk? Johann PR target/52507 * config/avr/lib1funcs.S (__movmemx_hi): Fix loop label in RAM-part. Index: libgcc/config/avr/lib1funcs.S =

Re: Remove obsolete Tru64 UNIX V5.1B support

2012-03-06 Thread Rainer Orth
Bruce Korb writes: > On Mon, Mar 5, 2012 at 3:13 PM, Joseph S. Myers > wrote: >> On Mon, 5 Mar 2012, Rainer Orth wrote: >> >>> * There are some fixincludes hacks that from their names seem to be >>>   osf-specific, but are not restricted to alpha*-dec-osf*.  Bruce, >>>   what's the best way to

Re: Remove obsolete Tru64 UNIX V5.1B support

2012-03-06 Thread Rainer Orth
Arnaud Charlet writes: > The gnattools and gcc/ada parts are OK, except for the comment removal in > s-tassta.adb: this comment is still useful, and needs to be revisited at > some point ratheer than removed silently as you did, to understand > why we can't use a when E: others => construct. > >

Re: Remove obsolete Tru64 UNIX V5.1B support

2012-03-06 Thread Rainer Orth
Tristan, >> * As I've mentioned, I've ripped out the #pragma extern_prefix support: >> while VMS has something similar, it doesn't use the common code. > > in fact VMS use some of the already existing #pragma extern_prefix support. > You're removing too much code here ! oops, seems I was too e

[Patch,AVR] Fix PR52505

2012-03-06 Thread Georg-Johann Lay
This is fix for __memx address space reads that might read from RAM or flash. For flash reads, never read from RAM so that I/O latches don't get read out unintentionally. Ok for trunk? Johann libgcc/ PR target/52505 * config/avr/lib1funcs.S (__xload_1): Don't read unintentionally

[PATCH][1/n] No longer sign-extend sizetype constants, remove TYPE_IS_SIZETYPE

2012-03-06 Thread Richard Guenther
This splits out some small chunks from the patch killing TYPE_IS_SIZETYPE. It introduces a helper function that performs object-size checks and uses it in the few places that would need adjustments when host_integerp is no longer appropriate for verifying that the size fits in half of the addres

Re: [PATCH] Fix for PR52081 - Missed tail merging with pure calls

2012-03-06 Thread Richard Guenther
On Mon, Feb 13, 2012 at 1:36 PM, Tom de Vries wrote: > On 13/02/12 12:54, Richard Guenther wrote: >> On Thu, Feb 2, 2012 at 11:44 AM, Tom de Vries wrote: >>> Richard, >>> >>> this patch fixes PR52801. >>> >>> Consider test-case pr51879-12.c: >>> ... >>> __attribute__((pure)) int bar (int); >>> __

4.4 branch frozen

2012-03-06 Thread Jakub Jelinek
Hi! The 4.4 branch is now frozen, all commits require RM approval. There will be the 4.4.7 release next week released from it and after that the branch will be closed. Jakub

Re: PATCH [1/n] addr32: Properly use Pmode and word_mode

2012-03-06 Thread H.J. Lu
On Mon, Mar 5, 2012 at 9:11 AM, H.J. Lu wrote: > On Sun, Mar 4, 2012 at 11:47 PM, Uros Bizjak wrote: >> On Mon, Mar 5, 2012 at 4:53 AM, H.J. Lu wrote: >> >>> and compiler does generate the same output. i386.c also has >>> >>>        xasm = "jmp\t%A0"; >>>    xasm = "call\t%A0"; >>> >>> for calls

Re: [PATCH][1/n] No longer sign-extend sizetype constants, remove TYPE_IS_SIZETYPE

2012-03-06 Thread Richard Guenther
On Tue, 6 Mar 2012, Richard Guenther wrote: > > This splits out some small chunks from the patch killing > TYPE_IS_SIZETYPE. It introduces a helper function that > performs object-size checks and uses it in the few places > that would need adjustments when host_integerp is no longer > appropria

Re: [patch] fix memory corruption bug in tm_region_init

2012-03-06 Thread Aldy Hernandez
On 03/05/12 12:47, Richard Henderson wrote: On 03/05/2012 10:37 AM, Aldy Hernandez wrote: I thought there'd be a lot less overhead by callocing the value myself. Is the overhead negligible? Yes, it's negligible. I can certainly make it a VEC in a follow up patch if you want, though I'll c

Re: [patch] fix memory corruption bug in tm_region_init

2012-03-06 Thread Richard Henderson
On 03/06/12 07:55, Aldy Hernandez wrote: > + bb_regions = VEC_alloc (tm_region_p, heap, last_basic_block); > + VEC_reserve (tm_region_p, heap, bb_regions, last_basic_block); > + for (i = 0; i < last_basic_block; ++i) > +VEC_quick_insert (tm_region_p, bb_regions, i, NULL); The reserve is red

Re: [Patch,AVR] Fix PR52505

2012-03-06 Thread Denis Chertykov
2012/3/6 Georg-Johann Lay > > This is fix for __memx address space reads that might read from RAM or flash. > For flash reads, never read from RAM so that I/O latches don't get read out > unintentionally. > > Ok for trunk? > > Johann > > libgcc/ >        PR target/52505 >        * config/avr/lib1f

Re: [Patch,AVR]: Fix PR52507

2012-03-06 Thread Denis Chertykov
2012/3/6 Georg-Johann Lay : > This fixes a wrong loop label in the movmemhi worker for the __memx address > space case. > > Ok for trunk? > > Johann > >        PR target/52507 >        * config/avr/lib1funcs.S (__movmemx_hi): Fix loop label in RAM-part. Approved. Denis.

Re: [Patch,AVR]: Fix PR52506

2012-03-06 Thread Denis Chertykov
2012/3/6 Georg-Johann Lay : > The problem is that RAMPD/X/Y/Z are restored in the wrong order in ISR > epilogue. > > Ok for trunk? > > Johann > >        PR target/52506 >        * gcc/config/avr/avr.c (expand_epilogue): Fix order of restoration >        to: RAMPZ, RAMPY, RAMPX, RAMPD. >        (ex

Re: [patch] fix memory corruption bug in tm_region_init

2012-03-06 Thread Jakub Jelinek
On Tue, Mar 06, 2012 at 08:04:12AM -0800, Richard Henderson wrote: > On 03/06/12 07:55, Aldy Hernandez wrote: > > + bb_regions = VEC_alloc (tm_region_p, heap, last_basic_block); > > + VEC_reserve (tm_region_p, heap, bb_regions, last_basic_block); > > + for (i = 0; i < last_basic_block; ++i) > >

Re: [patch] fix memory corruption bug in tm_region_init

2012-03-06 Thread Aldy Hernandez
On 03/06/12 10:20, Jakub Jelinek wrote: On Tue, Mar 06, 2012 at 08:04:12AM -0800, Richard Henderson wrote: On 03/06/12 07:55, Aldy Hernandez wrote: + bb_regions = VEC_alloc (tm_region_p, heap, last_basic_block); + VEC_reserve (tm_region_p, heap, bb_regions, last_basic_block); + for (i = 0; i

Re: [PATCH][target/52481] m68k-*: internal compiler error: in extract_insn, at recog.c:2123

2012-03-06 Thread Richard Henderson
On 03/06/12 02:13, Andreas Schwab wrote: > Ralf Corsepius writes: > >> * Incompatiblible/insufficient binutils (I am using binutils-2.22)? >> * Some as/ld-flags getting lost/improperly propagated during >> assembly/linking? >> * Bad/incompatible inline-asm hard-coded somewhere? > > TAS needs at

[Fortran-dev] Merged from trunk; [patch, committed] ISO_Fortran_binding.h

2012-03-06 Thread Tobias Burnus
Dear Paul, dear all, I have merged (Rev. 184999) the trunk (Rev. 184980) to the fortran-dev branch, which now followed GCC 4.8. Additionally, I have committed to the fortran-dev branch (Rev. 185004) the first rough and incomplete version of ISO_Fortran_binding.h, which tries to follow TS 291

Re: [PATCH][1/n] No longer sign-extend sizetype constants, remove TYPE_IS_SIZETYPE

2012-03-06 Thread Joseph S. Myers
On Tue, 6 Mar 2012, Richard Guenther wrote: > * c-decl.c (grokdeclarator): Properly check for sizes that > cover more than half of the address-space. The C front-end change is OK. -- Joseph S. Myers jos...@codesourcery.com

Re: [RFC]: Add support for pragma pointer_size

2012-03-06 Thread Joseph S. Myers
On Tue, 6 Mar 2012, Tristan Gingold wrote: > The patch is simple: the C front-end will now calls c_build_pointer_type > (instead of build_pointer_type), which in turn calls > build_pointer_type_for_mode using the right mode. There seem to be quite a lot of build_pointer_type calls in the C fron

Re: Remove obsolete Tru64 UNIX V5.1B support

2012-03-06 Thread David Daney
On 03/06/2012 05:14 AM, Rainer Orth wrote: "Joseph S. Myers" writes: There's one particular issue: the change to java/io/File.java required my to regenerate the .class file in classpath. I've used Sun javac -target 1.5 for that and hope I got it right. I'd have expected regeneratio

Re: [PATCH][target/52481] m68k-*: internal compiler error: in extract_insn, at recog.c:2123

2012-03-06 Thread Andreas Schwab
Richard Henderson writes: > On 03/06/12 02:13, Andreas Schwab wrote: >> Ralf Corsepius writes: >> >>> * Incompatiblible/insufficient binutils (I am using binutils-2.22)? >>> * Some as/ld-flags getting lost/improperly propagated during >>> assembly/linking? >>> * Bad/incompatible inline-asm hard

libgo patch committed: Update to weekly.2012-03-04 release

2012-03-06 Thread Ian Lance Taylor
I have committed a patch to update libgo to the weekly.2012-03-04 release. As usual, this e-mail message only includes the diffs to files that are specific to gccgo. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r 99cd08e36ace libgo/MERGE --- a

Re: [PATCH][1/n] No longer sign-extend sizetype constants, remove TYPE_IS_SIZETYPE

2012-03-06 Thread Jason Merrill
OK. Jason

Re: [PR51752] publication safety violations in loop invariant motion pass

2012-03-06 Thread Aldy Hernandez
On 02/29/12 03:22, Richard Guenther wrote: So fixing up individual passes is easier - I can only think of PRE being problematic right now, I am not aware that any other pass moves loads or stores. So I'd simply pre-compute the stmt bit in PRE and adjust the if (gimple_has_volatile_o

Re: PR middle-end/52373: two pc_rtxs, etc.

2012-03-06 Thread Richard Henderson
On 02/27/12 11:58, Richard Sandiford wrote: > gcc/ > * rtl.h (pc_rtx, ret_rtx, simple_return_rtx, cc0_rtx): Redefine as > variables. > (GR_PC, GR_CC0, GR_RETURN, GR_SIMPLE_RETURN): Delete. > * emit-rtl.c (pc_rtx, ret_rtx, simple_return_rtx, cc0_rtx): New > variables. >

Re: [PATCH][target/52481] m68k-*: internal compiler error: in extract_insn, at recog.c:2123

2012-03-06 Thread Richard Henderson
On 03/06/12 09:55, Andreas Schwab wrote: > Richard Henderson writes: > >> On 03/06/12 02:13, Andreas Schwab wrote: >>> Ralf Corsepius writes: >>> * Incompatiblible/insufficient binutils (I am using binutils-2.22)? * Some as/ld-flags getting lost/improperly propagated during assemb

PATCH: Properly check mode for x86 call/jmp address

2012-03-06 Thread H.J. Lu
On Tue, Mar 6, 2012 at 7:37 AM, H.J. Lu wrote: > On Mon, Mar 5, 2012 at 9:11 AM, H.J. Lu wrote: >> On Sun, Mar 4, 2012 at 11:47 PM, Uros Bizjak wrote: >>> On Mon, Mar 5, 2012 at 4:53 AM, H.J. Lu wrote: >>> and compiler does generate the same output. i386.c also has        xasm =

Re: PATCH: Properly check mode for x86 call/jmp address

2012-03-06 Thread Uros Bizjak
On Tue, Mar 6, 2012 at 8:10 PM, H.J. Lu wrote: > On Tue, Mar 6, 2012 at 7:37 AM, H.J. Lu wrote: >> On Mon, Mar 5, 2012 at 9:11 AM, H.J. Lu wrote: >>> On Sun, Mar 4, 2012 at 11:47 PM, Uros Bizjak wrote: On Mon, Mar 5, 2012 at 4:53 AM, H.J. Lu wrote: > and compiler does generate th

Re: [PATCH 10/10] addr32: Add *zero_extendsidi2_x32.

2012-03-06 Thread H.J. Lu
On Mon, Mar 5, 2012 at 1:38 AM, Uros Bizjak wrote: > On Fri, Mar 2, 2012 at 10:14 PM, H.J. Lu wrote: > >> This is the last patch for Pmode == SImod in x32. In x32, the return value >> of the symbol address must be zero-extended to DImode, This patch adds >> *zero_extendsidi2_x32 to load the addre

Re: PATCH: Properly check mode for x86 call/jmp address

2012-03-06 Thread Richard Henderson
On 03/06/12 11:10, H.J. Lu wrote: > (define_insn "*call" > - [(call (mem:QI (match_operand:P 0 "call_insn_operand" "zw")) > + [(call (mem:QI (match_operand:C 0 "call_insn_operand" "zw")) >(match_operand 1 "" ""))] > - "!SIBLING_CALL_P (insn)" > + "!SIBLING_CALL_P (insn) > + && (GET_C

Re: Remove obsolete OpenBSD/MIPS support

2012-03-06 Thread Richard Sandiford
Rainer Orth writes: > I'm currently working on removing the obsolete Tru64 UNIX and IRIX > ports. When IRIX is gone, the obsoleted OpenBSD/MIPS is the only > remaining port that uses MIPS_DEBUGGING_INFO (which I plan to remove as > a followup once IRIX is gone). > > The following patch has been i

Re: [PR51752] publication safety violations in loop invariant motion pass

2012-03-06 Thread Richard Guenther
On Tue, Mar 6, 2012 at 6:55 PM, Aldy Hernandez wrote: > On 02/29/12 03:22, Richard Guenther wrote: > >> So fixing up individual passes is easier - I can only think of PRE being >> problematic right now, I am not aware that any other pass moves loads >> or stores.  So I'd simply pre-compute the stm

[patch] Cleanup fortran/convert.c

2012-03-06 Thread Steven Bosscher
Hi, This cleans up some remnants of the ancestors of fortran's convert.c, which was copied from GNAT IIRC. I would bootstrap&test this, but trunk appears to be broken for x86_64-linux right now (ICE in patch_jump_insn). But I can post this for review, at least. OK for trunk, after bootstrap+test

[PATCH] Fix PR18589

2012-03-06 Thread William J. Schmidt
Hi, This is a re-post of the patch I posted for comments in January to address http://gcc.gnu.org/bugzilla/show_bug.cgi?id=18589. The patch modifies reassociation to expose repeated factors from __builtin_pow* calls, optimally reassociate repeated factors, and possibly reconstitute __builtin_powi

Re: [PR51752] publication safety violations in loop invariant motion pass

2012-03-06 Thread Torvald Riegel
On Tue, 2012-03-06 at 21:18 +0100, Richard Guenther wrote: > On Tue, Mar 6, 2012 at 6:55 PM, Aldy Hernandez wrote: > > On 02/29/12 03:22, Richard Guenther wrote: > > > >> So fixing up individual passes is easier - I can only think of PRE being > >> problematic right now, I am not aware that any ot

Re: PATCH: Properly check mode for x86 call/jmp address

2012-03-06 Thread H.J. Lu
On Tue, Mar 6, 2012 at 11:47 AM, Richard Henderson wrote: > On 03/06/12 11:10, H.J. Lu wrote: >>  (define_insn "*call" >> -  [(call (mem:QI (match_operand:P 0 "call_insn_operand" "zw")) >> +  [(call (mem:QI (match_operand:C 0 "call_insn_operand" "zw")) >>        (match_operand 1 "" ""))] >> -  "!S

Re: [RFC PATCH]: Handle Pmode == SImode in stringop patterns

2012-03-06 Thread H.J. Lu
On Tue, Mar 6, 2012 at 1:37 AM, Uros Bizjak wrote: > On Mon, Mar 5, 2012 at 11:42 PM, H.J. Lu wrote: > +     case '^': > +       if (TARGET_64BIT && Pmode == SImode) > +         { > +           fputs ("addr32", file); > +#ifndef HAVE_AS_IX86_REP_LOCK_PREFIX > +           i

PATCH: Use word_mode to push/pop register for x86

2012-03-06 Thread H.J. Lu
Hi, push/pop in x86 only works on word_mode registers. This patch properly handles push/pop on registers in x86. Tested on Linux/x86-64. OK for trunk? Thanks. H.J. --- 2012-03-06 H.J. Lu * config/i386/i386.c (setup_incoming_varargs_64): Use word_mode with integer paramete

[PATCH, testsuite committed] Fix va-arg-25.c for non-vmx hardware

2012-03-06 Thread Pat Haugen
Committed the following as obvious to prevent the test from running on non-vmx powerpc64 hardware. Tested on a Power5 system to verify. -Pat testsuite/ChangeLog: 2012-03-06 Pat Haugen * gcc.dg/torture/va-arg-25.c: Fix typo for powerpc64-*-* check. Index: gcc/testsuite/gcc.dg/tortu

[IA-64] Work around thinko in 'x' constraint implementation

2012-03-06 Thread Eric Botcazou
We have a regression on one of the testcases of our internal testsuite on IA-64 with a 4.7-based compiler, which is of the form: test_vec_madd.adb: In function 'Test_Vec_Madd': test_vec_madd.adb:160:5: error: could not split insn (insn 887 4859 889 16 (set (reg:TI 158 f30 [orig:417 m ] [417])

[IA-64] Implement static stack checking

2012-03-06 Thread Eric Botcazou
This at last implements static stack checking for the IA-64, i.e. stack checking of the static part of the frame, and makes it possible to pass the entire ACATS testsuite. The peculiarity is the second stack in memory, namely the Backing Store of the Register Stack Engine, that needs to be deal

[patch] Fix non-standard Ada bootstrap failure on IA-64

2012-03-06 Thread Eric Botcazou
If you try to bootstrap the GNAT 4.7.0 compiler on IA-64/Linux with non-default options (-gnatpg replaced with -gnatpgn), you get another comparison failure caused by debug insns, stemming from the machine-specific reorg pass (aka insn bundling on IA-64). With -g , when cselib is called on: (i

Re: [PATCH][1/n] No longer sign-extend sizetype constants, remove TYPE_IS_SIZETYPE

2012-03-06 Thread Eric Botcazou
> Well. I suppose fixing that negative DECL_FIELD_OFFSET thing should > be #1 priority. OK, let me try over the next few days. -- Eric Botcazou

Re: PR middle-end/52373: two pc_rtxs, etc.

2012-03-06 Thread Hans-Peter Nilsson
> From: Richard Sandiford > Date: Mon, 27 Feb 2012 20:58:42 +0100 > This patch fixed PR 52373, ITYM 52372. > Tested on mips-sde-elf with rtl checking enabled. There were no > differences. Also tested with a mips-sgi-irix6.5 cc1 cross to verify > that it fixes the bug. OK to install? > > Ric

PATCH: Pass and return pointers in word_mode

2012-03-06 Thread H.J. Lu
Pmode may not be word_mode for TARGET_64BIT. This patch changes to word_mode instead of Pmode. OK for trunk? Thanks. H.J. 2012-03-02 H.J. Lu * config/i386/i386.c (function_value_64): Return pointers in word_mode instead of Pmode. (ix86_promote_function_mode): Likewi

Support for Runtime CPU type detection via builtins (issue5754058)

2012-03-06 Thread Sriraman Tallam
Patch for CPU detection at run-time. === Patch for CPU detection at run-time, to be used in dispatching of multi-versioned functions. Please see this discussion: http://gcc.gnu.org/ml/gcc-patches/2011-08/msg01355.html when this patch for reviewed the last time. F

libgo patch committed: Implement runtime.Caller and friends

2012-03-06 Thread Ian Lance Taylor
The standard Go library provides a couple of functions that can be used to get filename and line number information of the running program. An example is runtime.Caller, which can be used to return the filename and line number of the caller of the current function. In the gccgo world, this kind o

Re: [PATCH][target/52481] m68k-*: internal compiler error: in extract_insn, at recog.c:2123

2012-03-06 Thread Ralf Corsepius
On 03/06/2012 07:22 PM, Richard Henderson wrote: On 03/06/12 09:55, Andreas Schwab wrote: Richard Henderson writes: On 03/06/12 02:13, Andreas Schwab wrote: Ralf Corsepius writes: * Incompatiblible/insufficient binutils (I am using binutils-2.22)? * Some as/ld-flags getting lost/improperl

[Ping][PATCH, libstdc++-v3] Enable to cross-test libstdc++ on simulator

2012-03-06 Thread Terry Guo
Hello, Can anybody please review and approve the following simple patch? Thanks very much. http://gcc.gnu.org/ml/libstdc++/2011-08/msg00063.html BR, Terry

Re: PATCH: Pass and return pointers in word_mode

2012-03-06 Thread Uros Bizjak
On Tue, Mar 6, 2012 at 11:45 PM, H.J. Lu wrote: > Pmode may not be word_mode for TARGET_64BIT.  This patch changes to > word_mode instead of Pmode.  OK for trunk? > 2012-03-02  H.J. Lu   > >        * config/i386/i386.c (function_value_64): Return pointers in >        word_mode instead of Pmode. >

Re: [patch] PR 51417

2012-03-06 Thread Ralf Corsepius
On 03/06/2012 10:43 AM, Richard Guenther wrote: On Mon, 5 Mar 2012, Ralf Corsépius wrote: Hi, The patch below addresses an issue with gcc-4.7.0 the issue I had reported in http://gcc.gnu.org/ml/gcc/2012-03/msg00035.html and somebody else had bz'ed as http://gcc.gnu.org/bugzilla/show_bug.cgi?i