Re: [C++] Handle ?: for vectors

2012-10-18 Thread Marc Glisse
On Wed, 17 Oct 2012, Jason Merrill wrote: On 10/17/2012 10:30 PM, Marc Glisse wrote: For each component of a vector type, result[i] = if MSB of c[i] is set ? b[i] : a[i]. Curious. Do you know why they produce and expect -1 for true? It certainly seems to be a deliberate design choice, so

Re: [C++] Handle ?: for vectors

2012-10-18 Thread Jason Merrill
On 10/17/2012 11:15 PM, Marc Glisse wrote: In the middle-end, VEC_COND_EXPR is documented to take a vector of 0 and -1, and at expansion time, it gives vcond(x0,y,z) to the target if its first argument is not a comparison. Maybe we should leave the first argument alone and let the middle end

[PATCH] Fix dumps for IPA passes

2012-10-18 Thread Sharad Singhai
Hi, This patch fixes a problem with the new dump infrastructure as discussed in http://gcc.gnu.org/ml/gcc/2012-10/msg00227.html. It removes a check for current_function_decl so that dumps will work for IPA passes. In addition, this patch also adds a new inline function to check if any dump files

Re: [C++ Patch] PR 54501

2012-10-18 Thread Paolo Carlini
Hi, On 10/18/2012 03:17 AM, Jason Merrill wrote: Hmm, I thought I fixed a very similar bug recently. I'm concerned that this change will cause problems with brace-elision situations. But then again, can we have a zero-length array followed by anything else? If I understand correctly your

[v3] more constexpr hacking

2012-10-18 Thread Benjamin De Kosnik
as per LWG comments on N330[2-5]. Couple of notes: new fail has been temporarily worked around pending the resolution of http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54968 For tuple, the big fun remaining is piecewise_construct_t, which should allow tie and forward_as_tuple to be

[PATCH] Intrinsics for fxsave[,64], xsave[,64], xsaveopt[,64]

2012-10-18 Thread Alexander Ivchenko
Hi, this patch adds new intrinsics for fxsave, fxsave64, xsave, xsave64, xsaveopt and xsaveopt64 instructions Bootstrapped on x86-64 Is it ok for trunk? Changelog entry: 2012-10-16 Alexander Ivchenko alexander.ivche...@intel.com * gcc/common/config/i386/i386-common.c

[lra] Fix mipsisa32-elf build

2012-10-18 Thread Richard Sandiford
Hi Vlad, newlib failed to build for mipsisa32-elf because of a case in which a call-crossing pseudo P had been allocated a call-clobbered register and in which LRA hadn't inserted the save and restore. We then tripped the lra-assigns.c sanity check for this situation (thanks for adding that

Re: wide int patch #6: Replacement of hwi extraction from int-csts.

2012-10-18 Thread Richard Biener
On Wed, Oct 17, 2012 at 11:47 PM, Kenneth Zadeck zad...@naturalbridge.com wrote: Richi, I apologize for the size of this patch, but it only does one very small thing, it is just that it does it all over the middle end. This patch introduces a new api for extracting a signed or unsigned hwi

[Patch ARM] Add initial armv8 support.

2012-10-18 Thread Ramana Radhakrishnan
Hi, This adds command line options and basic pre-defined macro support for ARMv8 in the AArch32 port. Tested on trunk with arm-linux-gnueabi cross, no regressions. Applied. regards, Ramana 2012-10-17 Matthew Gretton-Dann matthew.gretton-d...@arm.com * config/arm/lib1funcs.S

[PATCH][LTO] Adjust INTEGER_CST streaming

2012-10-18 Thread Richard Biener
This makes us stream TREE_OVERFLOW INTEGER_CSTs the normal way (and thus possibly allow pickle refs to them). Shared INTEGER_CSTs which are special use LTO_integer_cst now instead of the weird special-casing of the LTO tag for INTEGER_CST. LTO bootstrapped and tested on

[Patch ARM] Add support for fma intrinsics for Neon

2012-10-18 Thread Ramana Radhakrishnan
Hi, This patch originally by Matt, adds support for the fma intrinsics in arm_neon.h at the correct architecture levels. Tested on arm-linux-gnueabi with no regressions. Applied, cheers Ramana 2012-10-18 Matthew Gretton-Dann matthew.gretton-d...@arm.com Ramana

[Patch,avr] Document __AVR_ARCH__

2012-10-18 Thread Georg-Johann Lay
This documents __AVR_ARCH__ built-in macro that was missing up to now. Pointer to --help=target is removed because the documentation of -mmcu= is complete. Ok to apply? Johann * doc/invoke.texi (AVR Options): Document __AVR_ARCH__. Note __AVR_device__ is not defined for cores.

[PATCH] Improve expansion into DEBUG_IMPLICIT_PTR (PR debug/54970)

2012-10-18 Thread Jakub Jelinek
Hi! This patch improves debug info quality in two small changes. One is that for DEBUG stmts like DEBUG p = MEM[a, 8] for a non-addressable we were giving up instead of creating DEBUG_IMPLICIT_PTR (plus 8) for it. The reason is if (GET_CODE (op0) == DEBUG_IMPLICIT_PTR ||

Re: [Patch ARM] Add support for fma intrinsics for Neon

2012-10-18 Thread Ramana Radhakrishnan
On 10/18/12 13:05, Ramana Radhakrishnan wrote: Applied, And this time with the patch. RamanaFrom ae8086e2d1ecae8fee711942d4b530947001e8ef Mon Sep 17 00:00:00 2001 From: Ramana Radhakrishnan ramana.radhakrish...@linaro.org Date: Wed, 17 Oct 2012 18:40:14 +0100 Subject: [PATCH 2/3] neon fma

Re: [PATCH] Improve expansion into DEBUG_IMPLICIT_PTR (PR debug/54970)

2012-10-18 Thread Richard Biener
On Thu, Oct 18, 2012 at 2:17 PM, Jakub Jelinek ja...@redhat.com wrote: Hi! This patch improves debug info quality in two small changes. One is that for DEBUG stmts like DEBUG p = MEM[a, 8] for a non-addressable we were giving up instead of creating DEBUG_IMPLICIT_PTR (plus 8) for it. The

Re: wide int patch #6: Replacement of hwi extraction from int-csts.

2012-10-18 Thread Kenneth Zadeck
On 10/18/2012 06:22 AM, Richard Biener wrote: On Wed, Oct 17, 2012 at 11:47 PM, Kenneth Zadeck zad...@naturalbridge.com wrote: Richi, I apologize for the size of this patch, but it only does one very small thing, it is just that it does it all over the middle end. This patch introduces a new

Re: building gcc with powerpc gold

2012-10-18 Thread David Edelsohn
On Wed, Oct 17, 2012 at 11:06 PM, Alan Modra amo...@gmail.com wrote: These two tests currently fail if using gold, in the first instance because powerpc64 gold doesn't support mixing old dot-sym objects with new objects, and in the second instance because gold doesn't have a --no-toc-sort

Re: wide int patch #6: Replacement of hwi extraction from int-csts.

2012-10-18 Thread Richard Biener
On Thu, Oct 18, 2012 at 2:52 PM, Kenneth Zadeck zad...@naturalbridge.com wrote: On 10/18/2012 06:22 AM, Richard Biener wrote: On Wed, Oct 17, 2012 at 11:47 PM, Kenneth Zadeck zad...@naturalbridge.com wrote: Richi, I apologize for the size of this patch, but it only does one very small

[asan] Fix some asan ICEs

2012-10-18 Thread Jakub Jelinek
Hi! Dodji reported to me an ICE about NOTE_INSN_BASIC_BLOCK in a middle of a bb. The following patch (cfgexpand.c hunk) fixes that. I then run asan cc1/cc1plus with -O2 -fasan on a portion of cc1files and got other ICEs, which is the reason for the two asan.c changes - one was that base wasn't

[lra] Fix rtl checking failure

2012-10-18 Thread Richard Sandiford
This is the first of two patches needed to get a clean bootstrap on x86_64-linux-gnu with RTL checking enabled. It simply stops us from pushing notes, barriers, etc. onto the stack, which then trigger a checking failure when we try to access their patterns later. Tested on x86_64-linux-gnu. OK

[lra] Fix rtl checking failure, part 2

2012-10-18 Thread Richard Sandiford
Another simple patch for an rtl checking problem. We were trying to apply REGNO to something that was no longer a register (because of equiv substitution). Tested on x86_64-linux-gnu. OK for lra branch? Richard gcc/ * lra-constraints.c (check_and_process_move): Only apply REGNO

Re: [PATCH, ARM][1/4] New RTL patterns for LDRD/STRD in Thumb mode

2012-10-18 Thread Richard Earnshaw
On 10/10/12 16:03, Greta Yorsh wrote: This patch adds define_insn patterns for LDRD and STRD in Thumb mode. ChangeLog gcc/ 2012-09-13 Sameera Deshpande sameera.deshpa...@arm.com Greta Yorsh greta.yo...@arm.com * config/arm/arm-protos.h (offset_ok_for_ldrd_strd): New

[PATCH] rs6000: Get rid of MD warnings

2012-10-18 Thread Segher Boessenkool
This fixes all warnings from the machine description (mostly these were missing modes). Bootstrapped and tested on powerpc64-linux; no regressions. The testsuite covers all changed patterns. Okay for mainline? Segher 2012-10-18 Segher Boessenkool seg...@kernel.crashing.org *

Re: [PATCH, ARM][2/4] Prologue using STRD in Thumb mode

2012-10-18 Thread Richard Earnshaw
On 10/10/12 16:03, Greta Yorsh wrote: Generate prologue using STRD when prefer_ldrd_strd is set in tune_params. ChangeLog gcc/ 2012-09-13 Sameera Deshpande sameera.deshpa...@arm.com Greta Yorsh greta.yo...@arm.com * config/arm/arm.c (thumb2_emit_strd_push): New

Re: wide int patch #6: Replacement of hwi extraction from int-csts.

2012-10-18 Thread Kenneth Zadeck
you know richi, i did not know who i was actually talking to. i said who is this richard beiner person and then i saw the email address. On 10/18/2012 08:58 AM, Richard Biener wrote: On Thu, Oct 18, 2012 at 2:52 PM, Kenneth Zadeck zad...@naturalbridge.com wrote: On 10/18/2012 06:22 AM,

Re: [PATCH] rs6000: Get rid of MD warnings

2012-10-18 Thread David Edelsohn
On Thu, Oct 18, 2012 at 9:40 AM, Segher Boessenkool seg...@kernel.crashing.org wrote: This fixes all warnings from the machine description (mostly these were missing modes). Bootstrapped and tested on powerpc64-linux; no regressions. The testsuite covers all changed patterns. Okay for

Re: [PATCH] Reduce conservativeness in REE using machine model (issue6631066)

2012-10-18 Thread Teresa Johnson
The attached patch implements avoids conservative behavior in REE by allowing removal of redundant extends when the def feeds another extend with a different mode. This works because in merge_def_and_ext only calls combine_set_extension if the candidate for removal has a wider mode than the def

Re: [Patch, Fortran] PR54884 - Fix TREE_PUBLIC()=0 regression for module procedures

2012-10-18 Thread Paul Richard Thomas
Hi Tobias, The patch to fortran/cpp.c looks like a very worthy bit of housekeeping but doesn't seem to me to have much to do with PR54844 :-) The rest is fine and is OK for trunk. Thanks for the fix. Paul On 17 October 2012 12:02, Tobias Burnus bur...@net-b.de wrote: In GCC 4.8, module

Re: [PATCH] gcc-{ar,nm,ranlib}: Find binutils binaries relative to self

2012-10-18 Thread Meador Inge
Ping ^ 2. On 10/09/2012 09:44 PM, Meador Inge wrote: Ping. On 10/04/2012 03:45 PM, Meador Inge wrote: Hi All, Currently the gcc-{ar,nm,ranlib} utilities assume that binutils is in path when invoking the wrapped binutils program. This goes against the accepted practice in GCC to find

Re: [PATCH] Rs6000 infrastructure cleanup (switches), revised patch #2f

2012-10-18 Thread Eric Botcazou
2012-10-17 Michael Meissner meiss...@linux.vnet.ibm.com * config/rs6000/rs6000.opt (rs6000_isa_flags): New flag word to replace target_flags that gives us 63 possible switches. (x_rs6000_isa_flags): Save area for rs6000_isa_flags. (x_rs6000_isa_flags_explicit):

Restore bootstrap on IA-64/Linux

2012-10-18 Thread Eric Botcazou
An unused variable left in except.c after Easwaran's recent patch. Tested on ia64-suse-linux, applied on the mainline as obvious. 2012-10-18 Eric Botcazou ebotca...@adacore.com * except.c (sjlj_emit_function_enter): Remove unused variable. -- Eric BotcazouIndex: except.c

Re: [PATCH] Intrinsics for fxsave[,64], xsave[,64], xsaveopt[,64]

2012-10-18 Thread Uros Bizjak
On Thu, Oct 18, 2012 at 11:16 AM, Alexander Ivchenko aivch...@gmail.com wrote: Hi, this patch adds new intrinsics for fxsave, fxsave64, xsave, xsave64, xsaveopt and xsaveopt64 instructions Bootstrapped on x86-64 Is it ok for trunk? Changelog entry: 2012-10-16 Alexander Ivchenko

Re: [asan] Fix some asan ICEs

2012-10-18 Thread Xinliang David Li
On Thu, Oct 18, 2012 at 5:58 AM, Jakub Jelinek ja...@redhat.com wrote: Hi! Dodji reported to me an ICE about NOTE_INSN_BASIC_BLOCK in a middle of a bb. The following patch (cfgexpand.c hunk) fixes that. I then run asan cc1/cc1plus with -O2 -fasan on a portion of cc1files and got other ICEs,

DATA STUDIO - ESTÚDIO DE GRAVAÇÃO DE BANDAS, JINGLES E SPOTS PUBLICITÁRIOS.

2012-10-18 Thread Data Studio
OLÁ, VENHO ATRAVÉS DESTE E-MAIL APRESENTAR O DATA STUDIO: PARA CONSULTAS DE ORÇAMENTOS DE JINGLES, SPOTS, VINHETAS, LOCUÇÕES E TAMBÉM CONVERSÃO DE VINIL E CASSETE PARA CD E VHS PARA DVD, BASTA ENTRAR EM CONTATO POR ESTE E-MAIL OU PELO SITE. QUEM TIVER INTERESSE TENHO UM PORTIFÓLIO DE AUDIO NO

Re: [C++ Patch] PR 54501

2012-10-18 Thread Jason Merrill
On 10/18/2012 01:15 AM, Paolo Carlini wrote: If I understand correctly your hesitations, I don't think there are exceptions to the general rule that if the size of the array is zero there can be no initializers. I'm thinking of a testcase like this, which is currently accepted: struct A {

Re: [PATCH] Intrinsics for fxsave[,64], xsave[,64], xsaveopt[,64]

2012-10-18 Thread Uros Bizjak
On Thu, Oct 18, 2012 at 5:45 PM, Uros Bizjak ubiz...@gmail.com wrote: On Thu, Oct 18, 2012 at 11:16 AM, Alexander Ivchenko aivch...@gmail.com wrote: Hi, this patch adds new intrinsics for fxsave, fxsave64, xsave, xsave64, xsaveopt and xsaveopt64 instructions Bootstrapped on x86-64 Is it

[PATCH] Don't fold PTR - (UN)SIGNED comparison (PR middle-end/54945)

2012-10-18 Thread Marek Polacek
This patch changes folding in fold_sign_changed_comparison in a way that when we have pointer/non-pointer comparison, we give up folding here. The issue is e.g. when we have (intptr_t) MEM[(void *)x + 4B] == (intptr_t) y and forwprop wants to fold this one via fold_binary_loc--we then end up

Re: RFC: Merge the GUPC branch into the GCC 4.8 trunk (patch 01 of 16)

2012-10-18 Thread Gary Funck
Joseph, In this rather long reply, I have attempted to collect all your recent feedback, and to provide a response where possible. JSM: On Mon, 15 Oct 2012, Gary Funck wrote: JSM: GF: Various UPC language related checks and operations JSM: GF: are called in the C front-end and middle-end. JSM:

[Patch ARM] Add Cortex A15 neon scheduler

2012-10-18 Thread Ramana Radhakrishnan
Hi, This adjusts the existing A15 scheduler description to support Neon. Tested on arm-linux-gnueabi cross. Applied to trunk. regards, Ramana 2012-10-18 Matthew Gretton-Dann matthew.gretton-d...@arm.com Ramana Radhakrishnan ramana.radhakrish...@arm.com Sameera

Re: [Patch, Fortran] PR54884 - Fix TREE_PUBLIC()=0 regression for module procedures

2012-10-18 Thread Paul Richard Thomas
Hi Tobias, The patch to fortran/cpp.c looks like a very worthy bit of housekeeping but doesn't seem to me to have much to do with PR54844 :-) The rest is fine and is OK for trunk. Thanks for the fix. Paul On 17 October 2012 12:02, Tobias Burnus bur...@net-b.de wrote: In GCC 4.8, module

[C++ testcase] PR 29633

2012-10-18 Thread Paolo Carlini
Hi, testcase added, PR closed as fixed in mainline. Thanks, Paolo. / 2012-10-18 Paolo Carlini paolo.carl...@oracle.com PR c++/29633 * g++.dg/template/pr29633.C: New. Index: g++.dg/template/pr29633.C

Re: [C++ Patch] PR 54501

2012-10-18 Thread Paolo Carlini
On 10/18/2012 06:02 PM, Jason Merrill wrote: On 10/18/2012 01:15 AM, Paolo Carlini wrote: If I understand correctly your hesitations, I don't think there are exceptions to the general rule that if the size of the array is zero there can be no initializers. I'm thinking of a testcase like

Re: wide int patch #7: Replacement of INT_CST_LT and INT_CST_LT_UNSIGNED

2012-10-18 Thread Kenneth Zadeck
This patch replaces all instances of INT_CST_LT and INT_CST_LT_UNSIGNED with the new functions tree_int_cst_lts_p and tree_int_cst_ltu_p. With the new implementation of int_cst these functions will be too big to do inline. This is a small patch that has no prerequisites. Tested on x86-64.

[PING][PATCH, Testsuite] Add new effective target arm_prefer_ldrd_strd

2012-10-18 Thread Greta Yorsh
Ping! Thanks. -Original Message- From: Greta Yorsh [mailto:greta.yo...@arm.com] Sent: 10 October 2012 15:28 To: GCC Patches Cc: Ramana Radhakrishnan; Richard Earnshaw; ni...@redhat.com; p...@codesourcery.com; Greta Yorsh; mikest...@comcast.net; r...@cebitec.uni-bielefeld.de;

[PATCH, COMMITTED] Fix comment in gcc/config/sh/sh.c

2012-10-18 Thread Jan-Benedict Glaw
Hi! This patch silences the following warning: ../../../../gcc/gcc/config/sh/sh.c:9455:1: warning: /* within comment [-Wcomment] Comitted as obvious. MfG, JBG 2012-10-18 Jan-Benedict Glaw jbg...@lug-owl.de gcc/ * config/sh/sh.c: Fix comment to silence warning. diff --git

Re: [asan] migrate runtime from llvm

2012-10-18 Thread Xinliang David Li
On Tue, Oct 16, 2012 at 12:27 AM, Jakub Jelinek ja...@redhat.com wrote: On Mon, Oct 15, 2012 at 11:39:52PM -0700, Wei Mi wrote: --- gcc/gcc.c (revision 192487) +++ gcc/gcc.c (working copy) @@ -679,6 +679,7 @@ proper position among the other output f

Re: [asan] migrate runtime from llvm

2012-10-18 Thread Jakub Jelinek
On Thu, Oct 18, 2012 at 09:46:36AM -0700, Wei Mi wrote: --- gcc/gcc.c (revision 192567) +++ gcc/gcc.c (working copy) @@ -679,6 +679,7 @@ proper position among the other output f %{fgnu-tm:%:include(libitm.spec)%(link_itm)}\ %(mflib) STACK_SPLIT_SPEC \

Re: [asan] migrate runtime from llvm

2012-10-18 Thread Jakub Jelinek
On Thu, Oct 18, 2012 at 11:15:33AM -0700, Xinliang David Li wrote: --- Makefile.def (revision 192487) +++ Makefile.def (working copy) @@ -119,6 +119,7 @@ target_modules = { module= libstdc++-v3; lib_path=src/.libs; raw_cxx=true; };

[patch] PR 54930: add -Wreturn-local-addr

2012-10-18 Thread Jonathan Wakely
This adds a warning switch for the existing returning address of local variable warnings in the C and C++ FEs which are enabled by default but have no switch controlling them. Adding a switch allows it to be turned into an error with -Werror=return-local-addr. I've added two tests checking the

Re: [PATCH] gcc-{ar,nm,ranlib}: Find binutils binaries relative to self

2012-10-18 Thread Bernhard Reutner-Fischer
On 18 October 2012 17:30:20 Meador Inge mead...@codesourcery.com wrote: Ping ^ 2 Been a while but wasn't --with-build-sysroot for exactly this? On 10/09/2012 09:44 PM, Meador Inge wrote: Ping. On 102012 03:45 PM, Meador Inge wrote: Hi All, Currently the gcc-{ar,nm,ranlib} utilities

[c-family] Small fix for -fdump-ada-spec

2012-10-18 Thread Eric Botcazou
The recent introduction of virtual locations revealed a weak spot in the implementation of -fdump-ada-spec. Tested on x86_64-suse-linux, applied on the mainline as obvious. 2012-10-18 Eric Botcazou ebotca...@adacore.com * c-ada-spec.c (LOCATION_COL): Delete.

Re: [PATCH] Rs6000 infrastructure cleanup (switches), revised patch #2f

2012-10-18 Thread Dominique Dhumieres
2012-10-17 Michael Meissner meiss...@linux.vnet.ibm.com * config/rs6000/rs6000.opt (rs6000_isa_flags): New flag word to replace target_flags that gives us 63 possible switches. (x_rs6000_isa_flags): Save area for rs6000_isa_flags. (x_rs6000_isa_flags_explicit):

Re: [asan] migrate runtime from llvm

2012-10-18 Thread Xinliang David Li
On Thu, Oct 18, 2012 at 11:22 AM, Jakub Jelinek ja...@redhat.com wrote: On Thu, Oct 18, 2012 at 11:15:33AM -0700, Xinliang David Li wrote: --- Makefile.def (revision 192487) +++ Makefile.def (working copy) @@ -119,6 +119,7 @@ target_modules = { module= libstdc++-v3;

Re: [C++ Patch] PR 54501

2012-10-18 Thread Paolo Carlini
Hi again, On 10/18/2012 06:02 PM, Jason Merrill wrote: On 10/18/2012 01:15 AM, Paolo Carlini wrote: If I understand correctly your hesitations, I don't think there are exceptions to the general rule that if the size of the array is zero there can be no initializers. I'm thinking of a

Re: [asan] migrate runtime from llvm

2012-10-18 Thread Xinliang David Li
+ kcc the author of the libasan library. Koystya, does libasan build (need to ) depend on C++ library ? thanks, David On Thu, Oct 18, 2012 at 12:26 PM, Xinliang David Li davi...@google.com wrote: On Thu, Oct 18, 2012 at 11:22 AM, Jakub Jelinek ja...@redhat.com wrote: On Thu, Oct 18, 2012 at

Re: Ping: RFA: add lock_length attribute to break branch-shortening cycles

2012-10-18 Thread Richard Sandiford
Joern Rennecke joern.renne...@embecosm.com writes: @@ -1360,12 +1369,20 @@ shorten_branches (rtx first ATTRIBUTE_UN else inner_length = insn_current_length (inner_insn); - if (inner_length != insn_lengths[inner_uid]) + /* We

Re: RFC: Merge the GUPC branch into the GCC 4.8 trunk (patch 01 of 16)

2012-10-18 Thread Joseph S. Myers
On Thu, 18 Oct 2012, Gary Funck wrote: GUPC (then called GCC/UPC) dates back to the GCC 2.7 and GCC 2.95 days. The GCC 2.7 based implementation was a prototype, and the GCC 2.95 version was a first attempt to implement a UPC compiler that fits within the GCC build framework. At the time, we

Re: [asan] migrate runtime from llvm

2012-10-18 Thread Wei Mi
On Thu, Oct 18, 2012 at 11:16 AM, Jakub Jelinek ja...@redhat.com wrote: On Thu, Oct 18, 2012 at 09:46:36AM -0700, Wei Mi wrote: --- gcc/gcc.c (revision 192567) +++ gcc/gcc.c (working copy) @@ -679,6 +679,7 @@ proper position among the other output f

Re: Yet another debug check for unordered containers

2012-10-18 Thread François Dumont
On 10/18/2012 11:12 AM, Paolo Carlini wrote: On 10/17/2012 10:05 PM, François Dumont wrote: Ok to commit ? Ok, thanks. Paolo. Attached patch applied. 2012-10-18 François Dumont fdum...@gcc.gnu.org * include/debug/formatter.h (_Debug_msg_id): Add __msg_valid_load_factor. *

Re: [PATCH] Rs6000 infrastructure cleanup (switches), revised patch #2f

2012-10-18 Thread Mike Stump
On Oct 18, 2012, at 12:08 PM, Dominique Dhumieres domi...@lps.ens.fr wrote: --- gcc/config/rs6000/rs6000.c2012-10-18 00:34:51.0 +0200 +++ ../work/gcc/config/rs6000/rs6000.c2012-10-18 08:55:28.0 +0200 @@ -1123,6 +1123,7 @@ static const struct attribute_spec

Re: Ping: RFA: add lock_length attribute to break branch-shortening cycles

2012-10-18 Thread Joern Rennecke
Quoting Richard Sandiford rdsandif...@googlemail.com: The fact that we even have shared unique ids is pretty bad -- and surely a contradiction in terms -- but I think both ways of handling them rely on the length being the same for all copies. If we don't record a length (your version), we

[lra] patch to fix a SPEC2000 failure.

2012-10-18 Thread Vladimir Makarov
The following patch solves a wrong code generation for SPEC2000 galgel on x86. The reason of the failure was in wrong assignment after live range splitting which resulted in dead code elimination. Wrong assignment was a result of that LRA trusts allocation of IRA based on LIVE info

Re: [lra] Fix mipsisa32-elf build

2012-10-18 Thread Vladimir Makarov
On 10/18/2012 05:28 AM, Richard Sandiford wrote: Hi Vlad, newlib failed to build for mipsisa32-elf because of a case in which a call-crossing pseudo P had been allocated a call-clobbered register and in which LRA hadn't inserted the save and restore. We then tripped the lra-assigns.c sanity

Re: [lra] Fix rtl checking failure, part 2

2012-10-18 Thread Vladimir Makarov
On 10/18/2012 09:33 AM, Richard Sandiford wrote: Another simple patch for an rtl checking problem. We were trying to apply REGNO to something that was no longer a register (because of equiv substitution). Tested on x86_64-linux-gnu. OK for lra branch? Yes. Thanks, Richard.

Re: [PING][PATCH, Testsuite] Add new effective target arm_prefer_ldrd_strd

2012-10-18 Thread Mike Stump
On Oct 18, 2012, at 10:39 AM, Greta Yorsh greta.yo...@arm.com wrote: Ping! Thanks. -Original Message- From: Greta Yorsh [mailto:greta.yo...@arm.com] Sent: 10 October 2012 15:28 To: GCC Patches Cc: Ramana Radhakrishnan; Richard Earnshaw; ni...@redhat.com; p...@codesourcery.com;

Re: Ping: RFA: add lock_length attribute to break branch-shortening cycles

2012-10-18 Thread Richard Sandiford
Joern Rennecke joern.renne...@embecosm.com writes: Quoting Richard Sandiford rdsandif...@googlemail.com: The fact that we even have shared unique ids is pretty bad -- and surely a contradiction in terms -- but I think both ways of handling them rely on the length being the same for all copies.

[v3] add C++11 allocator support to forward_list

2012-10-18 Thread Jonathan Wakely
I should get some of the allocator work out of my git tree while we're in stage 1, this is for forward_list. * include/bits/forward_list.h: Add C++11 allocator support. * include/bits/forward_list.tcc: Likewise. * doc/xml/manual/status_cxx2011.xml: Update. *

Re: Tidy store_bit_field_1 co.

2012-10-18 Thread Richard Sandiford
Hi Eric, Thanks for the review. Eric Botcazou ebotca...@adacore.com writes: -476,34 +468,36 @@ store_bit_field_1 (rtx str_rtx, unsigne } /* If the target is a register, overwriting the entire object, or storing - a full-word or multi-word field can be done with just a SUBREG. +

Re: Tidy extract_bit_field_1 co.

2012-10-18 Thread Richard Sandiford
Richard Sandiford rdsandif...@googlemail.com writes: Partnering the store_bit_field_1 patch that I just posted, this patch tidies up the extract_bit_field code in the same way. There is one deliberate behavioural change here. The old code had a single check for cases where the extraction

Re: [PATCH] Rs6000 infrastructure cleanup (switches), revised patch #2f

2012-10-18 Thread Michael Meissner
On Thu, Oct 18, 2012 at 09:08:34PM +0200, Dominique Dhumieres wrote: 2012-10-17 Michael Meissner meiss...@linux.vnet.ibm.com * config/rs6000/rs6000.opt (rs6000_isa_flags): New flag word to replace target_flags that gives us 63 possible switches. (x_rs6000_isa_flags): Save

[PATCH, i386]: Cleanup rdpmc, rdtsc and rdtscp handling

2012-10-18 Thread Uros Bizjak
Hello! Attached patch moves a pack of duplicated code from i386.md to i386.c. No functional change. 2012-10-18 Uros Bizjak ubiz...@gmail.com * config/i386/i386.md (rdpmc): Remove expander. (rdtsc): Ditto. (rdtscp): Ditto. (rdpmc): Rename from *rdpmc.

Re: [C++ Patch] PR 54501

2012-10-18 Thread Jason Merrill
OK. Jason

Minimize downward code motion during reassociation

2012-10-18 Thread Easwaran Raman
Hi, During expression reassociation, statements are conservatively moved downwards to ensure that dependences are correctly satisfied after reassocation. This could lead to lengthening of live ranges. This patch moves statements only to the extent necessary. Bootstraps and no test regression on

Re: [PATCH] Rs6000 infrastructure cleanup (switches), revised patch #2f

2012-10-18 Thread Michael Meissner
On Thu, Oct 18, 2012 at 05:35:39PM -0400, Michael Meissner wrote: On Thu, Oct 18, 2012 at 09:08:34PM +0200, Dominique Dhumieres wrote: 2012-10-17 Michael Meissner meiss...@linux.vnet.ibm.com * config/rs6000/rs6000.opt (rs6000_isa_flags): New flag word to replace target_flags

Re: [path] PR 54900: store data race in if-conversion pass

2012-10-18 Thread Aldy Hernandez
On 10/16/12 20:33, Ian Lance Taylor wrote: On Tue, Oct 16, 2012 at 4:54 PM, Aldy Hernandez al...@redhat.com wrote: On 10/16/12 13:28, Jakub Jelinek wrote: location of considered if-conversion on mem if (some_condition) call (); mem = something; Are we not currently being

RE: [Patch,avr] Document __AVR_ARCH__

2012-10-18 Thread Weddington, Eric
-Original Message- From: Georg-Johann Lay Sent: Thursday, October 18, 2012 5:08 AM To: gcc-patches@gcc.gnu.org Cc: Denis Chertykov; Weddington, Eric Subject: [Patch,avr] Document __AVR_ARCH__ This documents __AVR_ARCH__ built-in macro that was missing up to now. Pointer to

Re: [PATCH] Rs6000 infrastructure cleanup (switches), revised patch #2f

2012-10-18 Thread David Edelsohn
On Thu, Oct 18, 2012 at 7:06 PM, Michael Meissner meiss...@linux.vnet.ibm.com wrote: 2012-10-18 Michael Meissner meiss...@linux.vnet.ibm.com * config/rs6000/rs6000-cpus.def (POWERPC_MASKS): Don't assume OPTION_MASK_STRICT_ALIGN is defined. (MASK_STRICT_ALIGN): If

[asan] Change shadow type from unsigned to signed

2012-10-18 Thread Xinliang David Li
The following is small bug fix caught during testing. The shadow value for redzone can be negative, so the shadow type must be signed for the check to succeed. Ok for the branch? thanks, David Index: asan.c === --- asan.c

Re: Fix twolf -funroll-loops -O3 miscompilation (a semi-latent web.c bug)

2012-10-18 Thread Bin.Cheng
On Wed, Oct 17, 2012 at 6:54 AM, Steven Bosscher stevenb@gmail.com wrote: On Tue, Oct 16, 2012 at 1:38 PM, Paolo Bonzini wrote: Bottom line: This is a bug in df_kill_notes. Yep, and it could cause wrong code generation, couldn't it? Because the new (reg:DI 72) is obviously not equal to

Re: [asan] migrate runtime from llvm

2012-10-18 Thread Xinliang David Li
thanks. If libasan is statically linked in (which is the case for clang), removing the dependency on libstdc++ is a must. David On Thu, Oct 18, 2012 at 9:05 PM, Kostya Serebryany k...@google.com wrote: +few more folks libasan does not (and should not) depend on libstdc++-v3. In llvm build

[lra] one more patch from RIchard's reviews

2012-10-18 Thread Vladimir Makarov
The following patch implements some Richard Sandiford's review of LRA. The patch was successfully bootstrapped and tested on x86-64/x86. Committed as rev. 192601. 2012-10-18 Vladimir Makarov vmaka...@redhat.com * lra-int.h (lra_special_reload_pseudos): Remove.

Re: RFC: LRA for x86/x86-64 [7/9] -- continuation

2012-10-18 Thread Vladimir Makarov
On 12-10-17 7:24 AM, Richard Sandiford wrote: Thanks for all the updates. Vladimir Makarov vmaka...@redhat.com writes: + /* index * scale + disp = new base + index * scale */ + enum reg_class cl = base_reg_class (mode, as, SCRATCH, SCRATCH); + + lra_assert

Re: RFC: LRA for x86/x86-64 [7/9] -- continuation

2012-10-18 Thread Vladimir Makarov
On 12-10-15 8:06 AM, Richard Sandiford wrote: Vladimir Makarov vmaka...@redhat.com writes: if that's accurate. I dropped the term reload pseudo because of the general comment in my earlier reply about the use of reload pseudo when the code seems to include inheritance and split pseudos too.