Re: [PATCH] avoid undefined behavior in libiberty/cplus-dem.c

2013-01-07 Thread Jakub Jelinek
On Sun, Jan 06, 2013 at 11:25:44PM -0500, Nickolai Zeldovich wrote: @@ -494,20 +505,15 @@ while (ISDIGIT ((unsigned char)**type)) { - count *= 10; - - /* Check for overflow. - We assume that count is represented using two's-complement; - no power of two is

Re: [committed] 2011 and 2012 Copyright year updates

2013-01-07 Thread Richard Biener
On Sun, 6 Jan 2013, Jakub Jelinek wrote: On Sun, Jan 06, 2013 at 12:13:32PM +, Richard Sandiford wrote: I never remember to update the copyright years, so I thought I'd have a go. And you were right of course. It ended up being a huge time sink. Anyway, here's my attempt a script

[Patch, Fortran] PR55852 fix ubound /size ICE and cleanup class.c's finalizer

2013-01-07 Thread Tobias Burnus
Dear all, the following patch is based on Thomas' comment 5 in PR55852. He independently created a patch for the PR; his version is in comment 6: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55852#c6 The problem with the current trunk's version of gfc_build_intrinsic_call is that it assumes

Re: [PR libmudflap/53359] don't register symbols not emitted

2013-01-07 Thread Richard Biener
On Sun, Jan 6, 2013 at 8:47 PM, Alexandre Oliva aol...@redhat.com wrote: On Jan 2, 2013, Richard Biener richard.guent...@gmail.com wrote: On Sun, Dec 30, 2012 at 1:22 AM, Alexandre Oliva aol...@redhat.com wrote: On Dec 21, 2012, Richard Biener richard.guent...@gmail.com wrote: On Fri, Dec

[PATCH] Fix PR864, apply program_transform_name to ada tools

2013-01-07 Thread Richard Biener
This makes us unconditionally apply program_transform_name when installing $(ADA_TOOLS). It's a patch distros have been applying for ages (now I've updated it for 4.8 which meant re-writing it completely so I am eligible to submit it). Bootstrapped on x86_64-unknown-linux-gnu with

Re: [PATCH] Fix PR864, apply program_transform_name to ada tools

2013-01-07 Thread Arnaud Charlet
This makes us unconditionally apply program_transform_name when installing $(ADA_TOOLS). It's a patch distros have been applying for ages (now I've updated it for 4.8 which meant re-writing it completely so I am eligible to submit it). Bootstrapped on x86_64-unknown-linux-gnu with

[PATCH] Fix PR55876 - Make generation of paradoxical subreg in widen_operand more robust

2013-01-07 Thread Tom de Vries
[with CC to gcc-patches] Original Message Subject: [PATCH] Fix PR55876 - Make generation of paradoxical subreg in widen_operand more robust Date: Mon, 07 Jan 2013 11:13:49 +0100 From: Tom de Vries tom_devr...@mentor.com To: Richard Henderson r...@redhat.com CC: Steve Ellcey

RE: [PATCH][ARM][1/3] Add vectorization support for rounding functions

2013-01-07 Thread Kyrylo Tkachov
Ping. http://gcc.gnu.org/ml/gcc-patches/2012-12/msg0.html Thanks, Kyrill -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Kyrylo Tkachov Sent: 18 December 2012 13:33 To: gcc-patches@gcc.gnu.org Cc: Ramana

RE: [PATCH][ARM][2/3] Add vectorization support for rounding functions

2013-01-07 Thread Kyrylo Tkachov
Ping. http://gcc.gnu.org/ml/gcc-patches/2012-12/msg01113.html Thanks, Kyrill -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Kyrylo Tkachov Sent: 18 December 2012 13:34 To: gcc-patches@gcc.gnu.org Cc: Ramana

[AARCH64] Add support for vector and scalar floating-point immediate loads.

2013-01-07 Thread James Greenhalgh
Hi, Currently, the AArch64 port loads all floating-point values via a constant pool. For some cases, we can do a better job. This patch adds support for loading immediates which are either +0.0 or a value which can be represented in a quarter-precision format. That is to say, an 8-bit

Re: [Ada] Remove dead code in elaborate_entity

2013-01-07 Thread Eric Botcazou
Pasto: applied on mainline only. Similar patch that removes a useless test (always true) and adjust a comment. Tested on x86_64-suse-linux, applied on mainline. 2013-01-07 Eric Botcazou ebotca...@adacore.com * gcc-interface/decl.c (gnat_to_gnu_entity) E_Record_Type: Adjust

[AArch64] Make argument of ld1 intrinsics const.

2013-01-07 Thread James Greenhalgh
Hi, This patch fixes the definitions of the vld1* intrinsics to take a const-qualified argument. I've tested this on aarch64-none-elf with no regressions. Is this OK to commit? Thanks, James Greenhalgh --- 2013-01-07 James Greenhalgh james.greenha...@arm.com *

Re: [Patch, Fortran] PR55852 fix ubound /size ICE and cleanup class.c's finalizer

2013-01-07 Thread Paul Richard Thomas
Dear Tobias, It looks good to me. OK for trunk Thanks for the patch - we'll have regressions down below 20 by tonight. I am rather certain that I have the correct fix for PR55618. Let's see if we can get to single figures, once more, by the end of the month! Paul On 7 January 2013 10:13,

[PATCH, i386]: Remove definition of DEFAULT_PCC_STRUCT_RETURN from i386.c

2013-01-07 Thread Uros Bizjak
Hello! The same definition now lives in defaults.h, included through tm.h. 2013-01-07 Uros Bizjak ubiz...@gmail.com * config/i386/i386.c (DEFAULT_PCC_STRUCT_RETURN): Remove. Tested on x86_64-pc-linux-gnu, committed to mainline SVN. Uros. Index: config/i386/i386.c

[Patch,avr] PR55243: Mistimed use of STAMP in Ada build

2013-01-07 Thread Georg-Johann Lay
This patch is a rewrite of t-avr so that STAMP is not needed. Ada triggers a mistimed re-build of targets using STAMP which causes PR55243 because the respective places in gnattools ada/gcc-interface don't set STAMP. With this patch, files that were formerly auto-generated must be generated by

Re: [Patch,avr] PR55243: Mistimed use of STAMP in Ada build

2013-01-07 Thread Denis Chertykov
2013/1/7 Georg-Johann Lay a...@gjlay.de: This patch is a rewrite of t-avr so that STAMP is not needed. Ada triggers a mistimed re-build of targets using STAMP which causes PR55243 because the respective places in gnattools ada/gcc-interface don't set STAMP. With this patch, files that were

[Patch, AArch64-4.7] Implement vec_init.

2013-01-07 Thread Tejas Belagod
Hi, The attached patch implements vec_init for AArch64. This has been tested on aarch64-none-elf with no regressions. OK for aarch64-4.7-branch? Thanks, Tejas Belagod ARM. 2013-01-07 Tejas Belagod tejas.bela...@arm.com gcc/ * config/aarch64/aarch64-simd.md (vec_initmode): New.

[Patch, AArch64] Implement vec_init.

2013-01-07 Thread Tejas Belagod
Hi, The attached patch implements vec_init for AArch64. This has been tested on aarch64-none-elf with no regressions. OK for trunk? Thanks, Tejas Belagod ARM. 2013-01-07 Tejas Belagod tejas.bela...@arm.com gcc/ * config/aarch64/aarch64-simd.md (vec_initmode): New. *

[PATCH] Fix PR55888, fix PR55862 differently

2013-01-07 Thread Richard Biener
The following fixes a fallout of fixing PR55862 - the fix for it merely papered over the issue that we inconsistently prune expressions in clean (). Fixing that allows us to revert the original PR55862 fix and thus fix PR55888. Bootstrapped on x86_64-unknown-linux-gnu, testing in progress.

[PATCH][RFC] Fix PR55890

2013-01-07 Thread Richard Biener
Since we freely propagate function addresses we can end up with call statements with mismatched number and type of arguments than what the function decl referenced says. This is a problem for builtin decls and all code that tries to examine their arguments (which may not be there or be of bogus

Re: [PATCH][RFC] Fix PR55890

2013-01-07 Thread Jakub Jelinek
On Mon, Jan 07, 2013 at 02:55:48PM +0100, Richard Biener wrote: *** gcc/testsuite/gcc.dg/torture/pr55890-1.c (revision 0) --- gcc/testsuite/gcc.dg/torture/pr55890-1.c (working copy) *** *** 0 --- 1,6 + /* { dg-do compile } */ + + extern void *memmove(void *, void

Re: [PATCH][RFC] Fix PR55890

2013-01-07 Thread Richard Biener
On Mon, 7 Jan 2013, Jakub Jelinek wrote: On Mon, Jan 07, 2013 at 02:55:48PM +0100, Richard Biener wrote: *** gcc/testsuite/gcc.dg/torture/pr55890-1.c(revision 0) --- gcc/testsuite/gcc.dg/torture/pr55890-1.c(working copy) *** *** 0 --- 1,6 + /* {

Re: [PATCH][RFC] Fix PR55890

2013-01-07 Thread Jakub Jelinek
On Mon, Jan 07, 2013 at 03:06:50PM +0100, Richard Biener wrote: *** gcc/tree-ssa-strlen.c (revision 194970) --- gcc/tree-ssa-strlen.c (working copy) *** adjust_last_stmt (strinfo si, gimple stm *** 809,818 if (!is_gimple_call (last.stmt)) return; !

Re: [Patch, Fortran] Remove the Fortran-only flag -fno-whole-file

2013-01-07 Thread Tobias Burnus
Early * ping * the patch below, i.e. http://gcc.gnu.org/ml/fortran/2013-01/msg00033.html Other pending patches by me: http://gcc.gnu.org/ml/fortran/2013-01/msg00049.html http://gcc.gnu.org/ml/fortran/2013-01/msg00025.html Other pending patches by … Thomas:

Re: [PATCH][RFC] Fix PR55890

2013-01-07 Thread Richard Biener
On Mon, 7 Jan 2013, Jakub Jelinek wrote: On Mon, Jan 07, 2013 at 03:06:50PM +0100, Richard Biener wrote: *** gcc/tree-ssa-strlen.c (revision 194970) --- gcc/tree-ssa-strlen.c (working copy) *** adjust_last_stmt (strinfo si, gimple stm *** 809,818 if

Re: [PATCH] avoid undefined behavior in libiberty/cplus-dem.c

2013-01-07 Thread Nickolai Zeldovich
On Mon, 7 Jan 2013, Jakub Jelinek wrote: Won't the above preclude parsing 2147483640 up to 2147483647 ? Because then in the last iteration count 214748364 (INT_MAX - 9) / 10. You're right -- thanks for catching that! Below is a patch with a more precise check. Nickolai. ---

[Patch,avr] PR55897: Allocate __memx to .progmemx.data

2013-01-07 Thread Georg-Johann Lay
Currently, data in address space __memx is allocated to .progmem.data, i.e. the same section as for __flash resp. progmem. However, there os no need that __memx goes onto the same section and wastes precious memory in the first 64 Ki. The patch puts __memx data into .progmemx.data so that a

Re: [Patch, AArch64] Implement vec_init.

2013-01-07 Thread Marcus Shawcroft
On 07/01/13 13:11, Tejas Belagod wrote: Hi, The attached patch implements vec_init for AArch64. This has been tested on aarch64-none-elf with no regressions. OK for trunk? Thanks, Tejas Belagod ARM. 2013-01-07 Tejas Belagod tejas.bela...@arm.com gcc/ *

Re: [Patch, AArch64-4.7] Implement vec_init.

2013-01-07 Thread Marcus Shawcroft
On 07/01/13 13:11, Tejas Belagod wrote: Hi, The attached patch implements vec_init for AArch64. This has been tested on aarch64-none-elf with no regressions. OK for aarch64-4.7-branch? Thanks, Tejas Belagod ARM. 2013-01-07 Tejas Belagod tejas.bela...@arm.com gcc/ *

Re: [AArch64] Make argument of ld1 intrinsics const.

2013-01-07 Thread Marcus Shawcroft
On 07/01/13 10:54, James Greenhalgh wrote: Hi, This patch fixes the definitions of the vld1* intrinsics to take a const-qualified argument. I've tested this on aarch64-none-elf with no regressions. Is this OK to commit? Thanks, James Greenhalgh --- 2013-01-07 James Greenhalgh

Re: [AARCH64] Add support for vector and scalar floating-point immediate loads.

2013-01-07 Thread Marcus Shawcroft
On 07/01/13 10:36, James Greenhalgh wrote: bool aarch64_bitmask_imm (HOST_WIDE_INT val, enum machine_mode); -bool aarch64_const_double_zero_rtx_p (rtx); +bool aarch64_float_const_zero_rtx_p (rtx); bool aarch64_constant_address_p (rtx); Can we keep these in alphabetical order please?

[patch,doc,avr] ad PR54461: Document --with-avrlibc

2013-01-07 Thread Georg-Johann Lay
This addendum adds --with-avrlibc to the (web) documentation. It inserts the explanation after --with-newlib which seems to be a reasonable place. Ok to apply? Johann PR target/PR54461 * doc/install.texi (Cross-Compiler-Specific Options): Document --with-avrlibc.

RE: [patch,doc,avr] ad PR54461: Document --with-avrlibc

2013-01-07 Thread Weddington, Eric
-Original Message- From: Georg-Johann Lay Sent: Monday, January 07, 2013 8:02 AM To: gcc-patches@gcc.gnu.org Cc: Denis Chertykov; Weddington, Eric; Gerald Pfeifer Subject: [patch,doc,avr] ad PR54461: Document --with-avrlibc This addendum adds --with-avrlibc to the (web)

[PATCH] Bump LTO bytecode version

2013-01-07 Thread Richard Biener
This bumps the minor version, LTO bytecode from 4.7 is not compatible with one from 4.8. Committed as obvious. Thanks, Richard. 2013-01-07 Richard Biener rguent...@suse.de * lto-streamer.h (LTO_minor_version): Bump to 2. Index: gcc/lto-streamer.h

Re: [patch,doc,avr] ad PR54461: Document --with-avrlibc

2013-01-07 Thread Georg-Johann Lay
Weddington, Eric wrote: Georg-Johann Lay Sent Document --with-avrlibc This addendum adds --with-avrlibc to the (web) documentation. It inserts the explanation after --with-newlib which seems to be a reasonable place. Ok to apply? Change: +This option is only supported for

RE: [patch,doc,avr] ad PR54461: Document --with-avrlibc

2013-01-07 Thread Weddington, Eric
-Original Message- From: Georg-Johann Lay Sent: Monday, January 07, 2013 8:28 AM To: Weddington, Eric Cc: gcc-patches@gcc.gnu.org; Denis Chertykov; Gerald Pfeifer Subject: Re: [patch,doc,avr] ad PR54461: Document --with-avrlibc +This option is only supported for the AVR target

[patch] Let tree-ssa-sink also sink pure calls

2013-01-07 Thread Steven Bosscher
Hello, As-is, tree-ssa-sink.c can only sink GIMPLE_ASSIGN statements. This patch lets tree-ssa-sink.c sink pure calls also. This allows the pass to sink the call to use in the following test case (new test case, ssa-sink-10.c to be): -- 8 /* { dg-do compile

Re: [committed] 2011 and 2012 Copyright year updates

2013-01-07 Thread Joseph S. Myers
On Sun, 6 Jan 2013, Richard Sandiford wrote: Anyway, here's my attempt a script to convert to ranges and, if enabled, to include the current year. The script only updates FSF copyright notices and leaves others alone. I've tried my best to make sure that licences and imported FSF sources

Re: PING^2: [PATCH] Support -fuse-ld=bfd and -fuse-ld=gold

2013-01-07 Thread Joseph S. Myers
On Wed, 19 Dec 2012, H.J. Lu wrote: +#ifdef CROSS_DIRECTORY_STRUCTURE + = concat(target_machine, -, ld_suffixes[i], NULL); Missing space before '('. The patch is OK with that fixed. -- Joseph S. Myers jos...@codesourcery.com

Re: [patch] Let tree-ssa-sink also sink pure calls

2013-01-07 Thread Richard Biener
On Mon, 7 Jan 2013, Steven Bosscher wrote: Hello, As-is, tree-ssa-sink.c can only sink GIMPLE_ASSIGN statements. This patch lets tree-ssa-sink.c sink pure calls also. This allows the pass to sink the call to use in the following test case (new test case, ssa-sink-10.c to be):

[testsuite] Fix 32-bit gcc.dg/lto/pr55525 with Sun ld

2013-01-07 Thread Rainer Orth
gcc.dg/lto/pr55525 currently FAILs on Solaris with Sun ld for the 32-bit multilib: FAIL: gcc.dg/lto/pr55525 c_lto_pr55525_0.o-c_lto_pr55525_1.o link, -flto -w UNRESOLVED: gcc.dg/lto/pr55525 c_lto_pr55525_0.o-c_lto_pr55525_1.o execute -flto -w ld: fatal: symbol 's' has differing sizes:

RE: [RFC] New feature to reuse one multilib among different targets

2013-01-07 Thread Joseph S. Myers
On Fri, 7 Dec 2012, Terry Guo wrote: 2012-12-07 Terry Guo terry@arm.com * gcc/Makefile.in (s-mlib): New argument MULTILIB_REUSE. * gcc/doc/fragments.texi: Document MULTILIB_REUSE. * gcc/gcc.c (multilib_reuse): New internal spec. (set_multilib_dir):

Re: [patch,doc,avr] ad PR54461: Document --with-avrlibc

2013-01-07 Thread Georg-Johann Lay
Weddington, Eric wrote: Georg-Johann Lay Sent Document --with-avrlibc This option is only supported for the AVR target and only for non-RTEMS configurations, which currently use newlib. The option is supported since version I forgot to mention that it looks like there might be some work

Re: [PATCH] Enable non-complex math builtins from C99 for Bionic

2013-01-07 Thread Joseph S. Myers
On Fri, 21 Dec 2012, Alexander Ivchenko wrote: Hi, Thank you very much for your input! Please, take a look at the updated version: I fixed coding style, moved documentation for TARGET_LIBC_HAS_FUNCTION to target.def. Removed TARGET_C99_FUNCTIONS and TARGET_HAS_SINCOS and all their

Re: [Patch, Fortran] Remove the Fortran-only flag -fno-whole-file

2013-01-07 Thread Dominique Dhumieres
Sorry for the late and lengthy answer. To make the story short, I think the decision to remove -fno-whole-file should be based on the answer to the following questions: (1) Is there any hint of -fno-whole-file misbehavior (as suggested by the second point of the Reasoning)? As said in my

RE: [patch,doc,avr] ad PR54461: Document --with-avrlibc

2013-01-07 Thread Weddington, Eric
-Original Message- From: Georg-Johann Lay [mailto:a...@gjlay.de] Sent: Monday, January 07, 2013 9:17 AM To: Weddington, Eric Cc: gcc-patches@gcc.gnu.org; Denis Chertykov; Gerald Pfeifer Subject: Re: [patch,doc,avr] ad PR54461: Document --with-avrlibc I used a different

[v3, build] Restrict -Wa,-nH use to Solaris (PR libstdc++/55594)

2013-01-07 Thread Rainer Orth
As reported in the PR, the AIX assembler also accepts the -nH option, but with completely different semantics. To avoid this, the current patch restricts checking the option to Solaris targets. Bootstrapped without regressions on i386-pc-solaris2.10 with Sun as (-Wa,-nH is added),

Re: [PATCH, dataflow]: Fix PR55845, 454.calculix miscompares on x86 AVX due to movement of vzeroupper

2013-01-07 Thread Uros Bizjak
On Sun, Jan 6, 2013 at 5:22 PM, Jakub Jelinek ja...@redhat.com wrote: --- df-problems.c (revision 194945) +++ df-problems.c (working copy) @@ -3916,6 +3916,10 @@ can_move_insns_across (rtx from, rtx to, rtx acros break; if (NONDEBUG_INSN_P (insn)) { + /* Do

Re: C++ PATCH for c++/55753 (ICE with constexpr)

2013-01-07 Thread Jason Merrill
And another bug: we were building an AGGR_INIT_EXPR in non-dependent code, and then failing to handle it properly in fold_non_dependent_expr. Fixed by not creating AGGR_INIT_EXPR in templates. As a result, we end up with a CALL_EXPR to the address of a constructor, and we need to avoid an

Re: [asan] Fix asan -fsection-anchors handling

2013-01-07 Thread Dodji Seketeli
Jakub Jelinek ja...@redhat.com writes: 2012-12-21 Jakub Jelinek ja...@redhat.com * varasm.c (output_constant_def_contents): For asan_protect_global protected strings, adjust DECL_ALIGN if needed, before testing for anchored symbols. (place_block_symbol): Adjust

Re: Support for MIPS r5900

2013-01-07 Thread Jeff Law
On 01/06/2013 03:56 PM, Jürgen Urban wrote: Hello, I created a patch from scratch to support MIPS r5900 used in the Playstation 2, but I have some problems with it. The attached patch only works with the latest binutils from CVS. The binutils forces the compiler to use r5900 compatible

Re: [Patch, Fortran] PR55763 - improve init-data checks for pointers

2013-01-07 Thread Mikael Morin
Hello, Le 04/01/2013 13:51, Tobias Burnus a écrit : diff --git a/gcc/fortran/expr.c b/gcc/fortran/expr.c index 2610784..146154e 100644 --- a/gcc/fortran/expr.c +++ b/gcc/fortran/expr.c @@ -3291,22 +3291,21 @@ gfc_check_assign (gfc_expr *lvalue, gfc_expr *rvalue, int conform) gfc_try

[PATCH] Ensure gimplify_one_sizepos doesn't change something with INTEGER_TYPE into something with e.g. ENUMERAL_TYPE (PR middle-end/55851, take 2)

2013-01-07 Thread Jakub Jelinek
On Fri, Jan 04, 2013 at 02:21:43PM +0100, Richard Biener wrote: The other way would be Index: gcc/fold-const.c === --- gcc/fold-const.c(revision 194900) +++ gcc/fold-const.c(working copy) @@ -900,9 +900,9 @@

Re: [PATCH] Fix PR55876 - Make generation of paradoxical subreg in widen_operand more robust

2013-01-07 Thread Richard Henderson
On 01/07/2013 02:16 AM, Tom de Vries wrote: - /* If MODE is no wider than a single word, we return a paradoxical - SUBREG. */ + /* If MODE is no wider than a single word, we return a + lowpart or paradoxical SUBREG. */ if (GET_MODE_SIZE (mode) = UNITS_PER_WORD) -return

Re: Use libstdc++-raw-cxx.m4 in libjava

2013-01-07 Thread H.J. Lu
On Fri, Jan 4, 2013 at 6:56 AM, H.J. Lu hjl.to...@gmail.com wrote: On Fri, Jan 4, 2013 at 2:06 AM, Andreas Schwab sch...@linux-m68k.org wrote: H.J. Lu hjl.to...@gmail.com writes: On Thu, Jan 3, 2013 at 10:09 AM, Andreas Schwab sch...@linux-m68k.org wrote: H.J. Lu hjl.to...@gmail.com writes:

Re: [Patch,avr] PR55897: Allocate __memx to .progmemx.data

2013-01-07 Thread Denis Chertykov
2013/1/7 Georg-Johann Lay a...@gjlay.de Currently, data in address space __memx is allocated to .progmem.data, i.e. the same section as for __flash resp. progmem. However, there os no need that __memx goes onto the same section and wastes precious memory in the first 64 Ki. The patch

Re: [Patch, Fortran] Remove the Fortran-only flag -fno-whole-file

2013-01-07 Thread Tobias Burnus
Dominique Dhumieres wrote: (1) Is there any hint of -fno-whole-file misbehavior (as suggested by the second point of the Reasoning)? The middle end is simply not prepared for having multiple declaration for a single file (translation unit), which leads to wrong optimizations and thus to

FW: AVX generic mode tuning discussion.

2013-01-07 Thread Jagasia, Harsha
We would like to propose changing AVX generic mode tuning to generate 128-bit AVX instead of 256-bit AVX. You indicate a 3% reduction on bulldozer with avx256. How does avx128 compare to -mno-avx -msse4.2? Will the next AMD generation have a useable avx256? I'm not keen on the idea of

[patch, mips, stabs] Do any MIPS targets use stabs?

2013-01-07 Thread Steve Ellcey
While testing all the variations of my mips-mti-elf target I found that a number of debug tests like gcc.dg/debug/trivial.c fail when compiled using a stabs debug flag (-gstabs3 for example) and -mips16. While running the GNU simulator I get: mips-core: 1 byte read to unmapped address 0xe820

[patch, mips] Change -mad to -mmad to match documentation

2013-01-07 Thread Steve Ellcey
The MIPS documentation describes a -mmad (-mno-mad) flag, but it is incorrectly implemented as -mad (-mno-ad) instead. This patch changes the implementation to match the documentation. This means that users who are currently using the -mad or -mno-ad flags would have to change. If we want we

Re: Control dependence vs. builtin_unreachable

2013-01-07 Thread Jeff Law
On 01/05/2013 01:10 PM, Steven Bosscher wrote: Presumably BB7 was created in response to the builtin_unreachable? Yes. The block only contains the BB_UNREACHABLE call. It is cleaned up at the end of the GIMPLE passes pipeline, in the fold-all-builtins pass (most __builtin_unreachable calls

Re: [committed] 2011 and 2012 Copyright year updates

2013-01-07 Thread Richard Sandiford
Joseph S. Myers jos...@codesourcery.com writes: On Sun, 6 Jan 2013, Richard Sandiford wrote: Anyway, here's my attempt a script to convert to ranges and, if enabled, to include the current year. The script only updates FSF copyright notices and leaves others alone. I've tried my best to

Re: RFA: RL78: Always select register bank 0 at the start of an ISR

2013-01-07 Thread DJ Delorie
I think that's right, since the ISR return restores the flag register, which has the bank select bits in it.

[Patch,avr] Add predicate symbols for the linker script to bump sections containing addr-space data

2013-01-07 Thread Georg-Johann Lay
This is a tentative patch that adds symbols that can be used as predicates in the linker script. Background is binutils PR14406: Data in address space __flash1 must be located in such a way that the high byte (hh8) of the address is 0x1. This is needed because avr-gcc sets RAMPZ to 0x1 before

Re: [committed] 2011 and 2012 Copyright year updates

2013-01-07 Thread Bruce Korb
On Mon, Jan 7, 2013 at 11:49 AM, Richard Sandiford rdsandif...@googlemail.com wrote: I think a patch for each directory will need posting separately for review of such things as whether any imported / generated files are mistakenly changed. So fixincludes/ separate from gcc/, and every

Re: [patch, mips, stabs] Do any MIPS targets use stabs?

2013-01-07 Thread Richard Sandiford
Steve Ellcey sell...@mips.com writes: While testing all the variations of my mips-mti-elf target I found that a number of debug tests like gcc.dg/debug/trivial.c fail when compiled using a stabs debug flag (-gstabs3 for example) and -mips16. While running the GNU simulator I get:

Re: [committed] 2011 and 2012 Copyright year updates

2013-01-07 Thread Joseph S. Myers
On Mon, 7 Jan 2013, Richard Sandiford wrote: Joseph S. Myers jos...@codesourcery.com writes: On Sun, 6 Jan 2013, Richard Sandiford wrote: Anyway, here's my attempt a script to convert to ranges and, if enabled, to include the current year. The script only updates FSF copyright notices

Re: atomic update of profile counters (issue7000044)

2013-01-07 Thread Richard Henderson
On 01/03/2013 04:42 PM, Rong Xu wrote: It links libatomic when -fprofile-gen-atomic is specified for FDO instrumentation build. Here I assume libatomic is always installed. Andrew: do you think if this is reasonable? It also disables the functionality if target does not support weak (ie.

Re: [patch, mips, stabs] Do any MIPS targets use stabs?

2013-01-07 Thread Richard Sandiford
Richard Sandiford rdsandif...@googlemail.com writes: Steve Ellcey sell...@mips.com writes: While testing all the variations of my mips-mti-elf target I found that a number of debug tests like gcc.dg/debug/trivial.c fail when compiled using a stabs debug flag (-gstabs3 for example) and

Re: [patch, mips, stabs] Do any MIPS targets use stabs?

2013-01-07 Thread Steve Ellcey
On Mon, 2013-01-07 at 20:30 +, Richard Sandiford wrote: Steve Ellcey sell...@mips.com writes: While testing all the variations of my mips-mti-elf target I found that a number of debug tests like gcc.dg/debug/trivial.c fail when compiled using a stabs debug flag (-gstabs3 for example)

Re: Control dependence vs. builtin_unreachable

2013-01-07 Thread Steven Bosscher
On Mon, Jan 7, 2013 at 8:45 PM, Jeff Law l...@redhat.com wrote: Before diving into the patch I think we should figure out why we see such different effects of eliminating these paths from the CFG. Your assertion is eliminating them will result in more false positives and less optimization

Re: Support for MIPS r5900

2013-01-07 Thread Richard Sandiford
Jeff Law l...@redhat.com writes: On 01/06/2013 03:56 PM, Jürgen Urban wrote: Hello, I created a patch from scratch to support MIPS r5900 used in the Playstation 2, but I have some problems with it. The attached patch only works with the latest binutils from CVS. The binutils forces the

Re: RFA: RL78: Correct values of the MDBL and MDBH registers

2013-01-07 Thread DJ Delorie
I did notice that the hardware didn't work the same way as the documentation... this would explain it. Yes, please apply it :-)

Re: [patch, mips, stabs] Do any MIPS targets use stabs?

2013-01-07 Thread Steve Ellcey
On Mon, 2013-01-07 at 20:38 +, Richard Sandiford wrote: Ah, sorry, this reminded me of a patch I'd written a year ago and never got around to sanity-checking. Does it fix the problem for you? Richard It fixes my small test case, I will try it on the full testsuite overnight. Steve

Re: atomic update of profile counters (issue7000044)

2013-01-07 Thread Rong Xu
Function __gcov_indirect_call_profiler_atomic (which contains call to the atomic function) is always emitted in libgcov. Since we only link libatomic when -fprofile-gen-atomic is specified, we have to make the atomic function weak -- otherwise, there is a unsat for regular FDO gen build (of

Re: [v3, build] Restrict -Wa,-nH use to Solaris (PR libstdc++/55594)

2013-01-07 Thread Jonathan Wakely
On 7 January 2013 16:39, Rainer Orth wrote: As reported in the PR, the AIX assembler also accepts the -nH option, but with completely different semantics. To avoid this, the current patch restricts checking the option to Solaris targets. Bootstrapped without regressions on

[PATCH] Minor VRP simplify_float_conversion_using_ranges tweaks (PR tree-optimization/54120)

2013-01-07 Thread Jakub Jelinek
Hi! As mentioned in the PR, the actual important bug has been fixed in range_fits_type_p by Richard Sandiford recently, so here are just two minor nits. An extension of a signed src_type to unsigned wider type is fine only if the vr doesn't have negative min or max, so always assuming we can

[PATCH] Another non-prototype builtin issue (PR middle-end/55890)

2013-01-07 Thread Jakub Jelinek
Hi! evaluate_stmt can be crashed similarly to what you've fixed today. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2013-01-07 Jakub Jelinek ja...@redhat.com PR middle-end/55890 * tree-ssa-ccp.c (evaluate_stmt): Use gimple_call_builtin_p. *

Re: [patch] Performance patch for MIPS conditional move in expr.c

2013-01-07 Thread Steve Ellcey
On Thu, 2012-11-15 at 13:39 -0800, Andrew Pinski wrote: I was posting it for Steve's benefit really. I was in the process of updating the patch to the trunk and trying it out there before doing a formal submission :). As I found out the testcase needs to be changed to work with the new

Re: Support for MIPS r5900

2013-01-07 Thread Richard Sandiford
Jürgen Urban juergenur...@gmx.de writes: ll, sc, dmult, ddiv, cvt.w.s, 64 bit FPU instructions. ll and sc is disabled with -mno-llsc and works. cvt.w.s is replaced by trunc.w.s. This seems to work. Probably showing my ignorance, but I couldn't see this in the patch. I disabled 64 bit FPU

Re: [PATCH] Adding target rdos to GCC

2013-01-07 Thread Leif Ekblad
Hello Uros, I tried to use your suggestion and use DEFAULT_SECTION_THRESHOLD in i386.opt, however this doesn't work for me. It seems the default is always 65536 regardless of what I define in rdos.h. Therefore, I kept the design as it was before. Changing name to TARGET_RDOS seems reasonable,

Re: [Patch, wwwdocs] Update Fortran part of the GCC 4.8 release notes

2013-01-07 Thread Gerald Pfeifer
On Thu, 20 Dec 2012, Tobias Burnus wrote: The following patch updates the Fortran part of the GCC 4.8 release notes at http://gcc.gnu.org/gcc-4.8/changes.html#fortran Nice! I had a look and made a couple of editorial changes which should not change any of the statements. If you'd like to see

[wwwdocs] Objective-C/Darwin related updates for the GCC 4.6 release notes

2013-01-07 Thread Gerald Pfeifer
Going through some older mails (as some may have noticed), I found this patch which Iain was had done, and which I had reviewed, but which somehow did not make it in. Applied now, with some fuzzing manually adjusted. Gerald --- changes.html.orig 2012-10-03 07:02:42.198917592 -1000 +++

Re: [Patch, fortran] PR55618 - [4.6/4.7/4.8 Regression] Failures with ISO_Varying_String test suite

2013-01-07 Thread Steve Kargl
On Mon, Jan 07, 2013 at 11:57:27PM +0100, Paul Richard Thomas wrote: Dear All, This is a rather 'obvious' fix that returns gfortran to full compliance in the IVS testsuite. The testcase checks that various other combinations of arguments to elemental procedures work correctly. I gave up

Re: [PATCH, dataflow]: Fix PR55845, 454.calculix miscompares on x86 AVX due to movement of vzeroupper

2013-01-07 Thread Jakub Jelinek
On Mon, Jan 07, 2013 at 05:52:23PM +0100, Uros Bizjak wrote: TBH, I'm not that familiar with the RTL infrastructure enough to answer these questions. While I can spend some time on this problem, and probably waste quite some reviewer's time, the problem is not that trivial as I hoped to be, so

Re: [wwwdocs] Document libstdc++ changes and minimum MinGW-w64 version requirement.

2013-01-07 Thread Jonathan Wakely
On 8 January 2013 00:17, Jonathan Wakely wrote: On 3 January 2013 05:14, Gerald Pfeifer wrote: On Sat, 15 Dec 2012, Jonathan Wakely wrote: Committed to wwwdocs. Thanks, Jonathan. Where it says, Added code--disable-libstdcxx-verbose/code configure option, would it make sense to add a

Re: Support for MIPS r5900

2013-01-07 Thread Maciej W. Rozycki
On Mon, 7 Jan 2013, Richard Sandiford wrote: ll, sc, dmult, ddiv, cvt.w.s, 64 bit FPU instructions. ll and sc is disabled with -mno-llsc and works. cvt.w.s is replaced by trunc.w.s. This seems to work. Probably showing my ignorance, but I couldn't see this in the patch. This has raised

Re: Support for MIPS r5900

2013-01-07 Thread Jeff Law
On 01/07/2013 02:52 PM, Richard Sandiford wrote: I disabled 64 bit FPU instructions by -msoft-float. This works, but using -msingle-float fails. This would be the better configuration. There are still 64 bit FPU instructions used (e.g. dmfc1 $2,$f0 when using long double multiplication). So

[AARCH64] Remove %r from asm_printf

2013-01-07 Thread Hurugalawadi, Naveen
Hi, ASM_FPRINTF_EXTENSIONS was copied from the arm back-end and is not needed. Please find attached the patch that replaces %r in asm_fprintf with reg_names. The definition of ASM_FPRINTF_EXTENSIONS is removed as it is no longer needed. Please review the patch and let me know if there should be

[Patch, AARCH64] Define REVERSIBLE_CC_MODE

2013-01-07 Thread Hurugalawadi, Naveen
Hi, AARCH64 back-end defines REVERSE_CONDITION but not REVERSIBLE_CC_MODE. REVERSE_CONDITION is used only in case REVERSIBLE_CC_MODE (mode) is nonzero. Please find attached the patch that defines REVERSIBLE_CC_MODE for aarch64 back-end. Please review the patch and let me know if there should be

Re: [Patch, fortran] PR55618 - [4.6/4.7/4.8 Regression] Failures with ISO_Varying_String test suite

2013-01-07 Thread Steve Kargl
On Tue, Jan 08, 2013 at 07:20:19AM +0100, Paul Richard Thomas wrote: Dear Steve, Thanks - committed as revision 195004. I should also have asked if it's OK for 4.6 and 4.7? Yes, it is okay with the usual 2 to 3 days of cooking on trunk before committing. Of course, as you noted the

Re: [PATCH, dataflow]: Fix PR55845, 454.calculix miscompares on x86 AVX due to movement of vzeroupper

2013-01-07 Thread Uros Bizjak
On Tue, Jan 8, 2013 at 12:26 AM, Jakub Jelinek ja...@redhat.com wrote: On Mon, Jan 07, 2013 at 05:52:23PM +0100, Uros Bizjak wrote: TBH, I'm not that familiar with the RTL infrastructure enough to answer these questions. While I can spend some time on this problem, and probably waste quite

Re: Support for MIPS r5900

2013-01-07 Thread Richard Sandiford
Richard Sandiford rdsandif...@googlemail.com writes: Jürgen Urban juergenur...@gmx.de writes: ll, sc, dmult, ddiv, cvt.w.s, 64 bit FPU instructions. ll and sc is disabled with -mno-llsc and works. cvt.w.s is replaced by trunc.w.s. This seems to work. Probably showing my ignorance, but I

Re: [asan] Use -fno-shrink-wrap for null-deref-1.c test

2013-01-07 Thread Dodji Seketeli
Jakub Jelinek ja...@redhat.com writes: 2012-12-19 Jakub Jelinek ja...@redhat.com * c-c++-common/asan/null-deref-1.c: Add -fno-shrink-wrap to dg-options. This looks OK to me, thanks. -- Dodji