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

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

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, and

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

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 Petterssonmikpe at it dot uu.se 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.

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 hjl.to...@gmail.com wrote: +     case '^': +       if (TARGET_64BIT Pmode == SImode) +         { +           fputs (addr32, file); +#ifndef HAVE_AS_IX86_REP_LOCK_PREFIX +           if (ASSEMBLER_DIALECT == ASM_ATT) +             fputs (addr32; ,

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

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 a...@gjlay.de wrote: Richard Guenther wrote: On Mon, Mar 5, 2012 at 6:15 PM, Georg-Johann Lay a...@gjlay.de wrote: Richard Guenther wrote: On Mon, Mar 5, 2012 at 4:56 PM, Georg-Johann Lay a...@gjlay.de wrote: Richard Guenther wrote: On Mon,

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 ken...@vlsi1.ultra.nyu.edu 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.

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 cloning is an

[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

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

2012-03-06 Thread Andreas Schwab
Ralf Corsepius ralf.corsep...@rtems.org 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.

[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 ging...@adacore.com * config/vms/vms.c (VMS_CRTL_LDBL): Rename from VMS_CRTL_PRNTF. *

[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

[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 rguent...@suse.de * fold-const.c (build_fold_addr_expr_with_type_loc): Fold MEM_REF with constant

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 a...@gjlay.de wrote: Richard Guenther wrote: On Mon, Mar 5, 2012 at 6:15 PM, Georg-Johann Lay a...@gjlay.de wrote: Richard Guenther wrote: On Mon, Mar 5, 2012 at 4:56 PM, Georg-Johann Lay a...@gjlay.de wrote: Richard

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 no

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 a...@gjlay.de wrote: Richard Guenther wrote: On Mon, Mar 5, 2012 at 6:54 PM, Georg-Johann Lay a...@gjlay.de wrote: Richard Guenther wrote: On Mon, Mar 5, 2012 at 6:15 PM, Georg-Johann Lay a...@gjlay.de wrote: Richard Guenther wrote: On Mon,

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 paolo.carl...@oracle.com PR c++/52422 * cp-tree.h (build_addr_func, decay_conversion,

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 kirill.yuk...@intel.com *

Re: [PATCH, i386] RTM support

2012-03-06 Thread Uros Bizjak
On Tue, Mar 6, 2012 at 12:59 PM, Kirill Yukhin kirill.yuk...@gmail.com 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

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 pins...@gmail.com 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

[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

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 jiangning@arm.com 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

Re: Remove obsolete Tru64 UNIX V5.1B support

2012-03-06 Thread Rainer Orth
Jonathan Wakely jwakely@gmail.com 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

Re: Remove obsolete Tru64 UNIX V5.1B support

2012-03-06 Thread Rainer Orth
Joseph S. Myers jos...@codesourcery.com 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

[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 bk...@gnu.org writes: On Mon, Mar 5, 2012 at 3:13 PM, Joseph S. Myers jos...@codesourcery.com 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,  

Re: Remove obsolete Tru64 UNIX V5.1B support

2012-03-06 Thread Rainer Orth
Arnaud Charlet char...@adacore.com 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 =

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 eager

[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

[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

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 tom_devr...@mentor.com wrote: On 13/02/12 12:54, Richard Guenther wrote: On Thu, Feb 2, 2012 at 11:44 AM, Tom de Vries tom_devr...@mentor.com wrote: Richard, this patch fixes PR52801. Consider test-case pr51879-12.c: ... __attribute__((pure))

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 hjl.to...@gmail.com wrote: On Sun, Mar 4, 2012 at 11:47 PM, Uros Bizjak ubiz...@gmail.com wrote: On Mon, Mar 5, 2012 at 4:53 AM, H.J. Lu hjl.to...@gmail.com wrote: and compiler does generate the same output. i386.c also has        xasm = jmp\t%A0;    

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 appropriate

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

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

Re: [Patch,AVR] Fix PR52505

2012-03-06 Thread Denis Chertykov
2012/3/6 Georg-Johann Lay a...@gjlay.de 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        *

Re: [Patch,AVR]: Fix PR52507

2012-03-06 Thread Denis Chertykov
2012/3/6 Georg-Johann Lay a...@gjlay.de: 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 a...@gjlay.de: 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.        

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;

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 ralf.corsep...@rtems.org 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?

[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

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 front

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. Myersjos...@codesourcery.com 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

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

2012-03-06 Thread Andreas Schwab
Richard Henderson r...@redhat.com writes: On 03/06/12 02:13, Andreas Schwab wrote: Ralf Corsepius ralf.corsep...@rtems.org writes: * Incompatiblible/insufficient binutils (I am using binutils-2.22)? * Some as/ld-flags getting lost/improperly propagated during assembly/linking? *

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 ---

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

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 r...@redhat.com writes: On 03/06/12 02:13, Andreas Schwab wrote: Ralf Corsepius ralf.corsep...@rtems.org writes: * Incompatiblible/insufficient binutils (I am using binutils-2.22)? * Some as/ld-flags getting lost/improperly

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 hjl.to...@gmail.com wrote: On Mon, Mar 5, 2012 at 9:11 AM, H.J. Lu hjl.to...@gmail.com wrote: On Sun, Mar 4, 2012 at 11:47 PM, Uros Bizjak ubiz...@gmail.com wrote: On Mon, Mar 5, 2012 at 4:53 AM, H.J. Lu hjl.to...@gmail.com wrote: and compiler does

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 hjl.to...@gmail.com wrote: On Tue, Mar 6, 2012 at 7:37 AM, H.J. Lu hjl.to...@gmail.com wrote: On Mon, Mar 5, 2012 at 9:11 AM, H.J. Lu hjl.to...@gmail.com wrote: On Sun, Mar 4, 2012 at 11:47 PM, Uros Bizjak ubiz...@gmail.com wrote: On Mon, Mar 5, 2012 at

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 ubiz...@gmail.com wrote: On Fri, Mar 2, 2012 at 10:14 PM, H.J. Lu hongjiu...@intel.com 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

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 czw)) + [(call (mem:QI (match_operand:C 0 call_insn_operand czw)) (match_operand 1 ))] - !SIBLING_CALL_P (insn) + !SIBLING_CALL_P (insn) +(GET_CODE (operands[0]) ==

Re: Remove obsolete OpenBSD/MIPS support

2012-03-06 Thread Richard Sandiford
Rainer Orth r...@cebitec.uni-bielefeld.de 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

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 al...@redhat.com 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

[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 bootstraptest 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

[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

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 al...@redhat.com 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

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 r...@redhat.com wrote: On 03/06/12 11:10, H.J. Lu wrote:  (define_insn *call -  [(call (mem:QI (match_operand:P 0 call_insn_operand czw)) +  [(call (mem:QI (match_operand:C 0 call_insn_operand czw))        (match_operand 1 ))] -  

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 ubiz...@gmail.com wrote: On Mon, Mar 5, 2012 at 11:42 PM, H.J. Lu hjl.to...@gmail.com wrote: +     case '^': +       if (TARGET_64BIT Pmode == SImode) +         { +           fputs (addr32, file); +#ifndef HAVE_AS_IX86_REP_LOCK_PREFIX +          

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 hongjiu...@intel.com * config/i386/i386.c (setup_incoming_varargs_64): Use word_mode

[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 pthau...@us.ibm.com * gcc.dg/torture/va-arg-25.c: Fix typo for powerpc64-*-* check. Index:

[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

[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:

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

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 hongjiu...@intel.com * config/i386/i386.c (function_value_64): Return pointers in word_mode instead of Pmode.

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.

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

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 Hendersonr...@redhat.com writes: On 03/06/12 02:13, Andreas Schwab wrote: Ralf Corsepiusralf.corsep...@rtems.org writes: * Incompatiblible/insufficient binutils (I am using binutils-2.22)? *

[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 hongjiu...@intel.com 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  hongjiu...@intel.com        * config/i386/i386.c (function_value_64): Return pointers in  

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