Re: [PATCH, ARM] Fix PR target/54892 - [4.7/4.8 Regression], ICE in extract_insn, at recog.c:2123

2012-10-18 Thread Ramana Radhakrishnan
On Fri, Oct 19, 2012 at 7:46 AM, Zhenqiang Chen wrote: > Hi, > > In function arm_expand_compare_and_swap, oldval is converted to SImode > when its "mode" is QImode/HImode. After "FALLTHRU" to "case SImode", > we should use "SImode", other than "mode" (which is QImode/HImode). > And INSN atomic_com

[PATCH, ARM] Fix PR target/54892 - [4.7/4.8 Regression], ICE in extract_insn, at recog.c:2123

2012-10-18 Thread Zhenqiang Chen
Hi, In function arm_expand_compare_and_swap, oldval is converted to SImode when its "mode" is QImode/HImode. After "FALLTHRU" to "case SImode", we should use "SImode", other than "mode" (which is QImode/HImode). And INSN atomic_compare_and_swap_1 expects the operand in SImode. No make check regre

RE: [PATCH RFA] Implement register pressure directed hoist pass

2012-10-18 Thread Bin Cheng
> -Original Message- > From: Jeff Law [mailto:l...@redhat.com] > Sent: Wednesday, October 17, 2012 1:02 AM > To: Bin Cheng > Cc: 'Steven Bosscher'; gcc-patches@gcc.gnu.org > Subject: Re: [PATCH RFA] Implement register pressure directed hoist pass > > On 10/16/2012 01:44 AM, Bin Cheng wro

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 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. There is no inher

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 writes: + /* index * scale + disp => new base + index * scale */ + enum reg_class cl = base_reg_class (mode, as, SCRATCH, SCRATCH); + + lra_assert (INDEX_REG_CLASS != NO_REGS); +

[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 * lra-int.h (lra_special_reload_pseudos): Remove. (lra_get_hard_regno_and_offset): Dit

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 wrote: > +few more folks > > libasan does not (and should not) depend on libstdc++-v3. > In llvm build system it is

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 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 (plus:DI (re

[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

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 wrote: > 2012-10-18 Michael Meissner > > * config/rs6000/rs6000-cpus.def (POWERPC_MASKS): Don't assume > OPTION_MASK_STRICT_ALIGN is defined. > (MASK_STRICT_ALIGN): If OPTION_MASK_STRICT_ALIGN is not defined, > d

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

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

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 > > > > > > * config/rs6000/rs6000.opt (rs6000_isa_flags): New flag word to > > > replace target_flags that gives us 63

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 x8

Re: [C++ Patch] PR 54501

2012-10-18 Thread Jason Merrill
OK. Jason

[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 * config/i386/i386.md (rdpmc): Remove expander. (rdtsc): Ditto. (rdtscp): Ditto. (rdpmc): Rename from *rdpmc. (rdpmc_rex64): Rena

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

Re: Tidy extract_bit_field_1 & co.

2012-10-18 Thread Richard Sandiford
Richard Sandiford 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 could be done as a simple

Re: Tidy store_bit_field_1 & co.

2012-10-18 Thread Richard Sandiford
Hi Eric, Thanks for the review. Eric Botcazou 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. + >> a fu

[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. * testsui

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

2012-10-18 Thread Richard Sandiford
Joern Rennecke writes: > Quoting Richard Sandiford : >> 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)

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 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; Greta Yorsh;

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

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

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

2012-10-18 Thread Joern Rennecke
Quoting Richard Sandiford : 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 won't set something_changed if

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 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 rs600 > > > #ifnd

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 * include/debug/formatter.h (_Debug_msg_id): Add __msg_valid_load_factor. * include/debug/macros.

Re: [asan] migrate runtime from llvm

2012-10-18 Thread Wei Mi
On Thu, Oct 18, 2012 at 11:16 AM, Jakub Jelinek 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 >> %{fgnu-tm:%:include(libitm.spec)%(link_itm)}\

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,

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

2012-10-18 Thread Richard Sandiford
Joern Rennecke 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 can't record lengths of d

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 wrote: > On Thu, Oct 18, 2012 at 11:22 AM, Jakub Jelinek wrote: >> On Thu, Oct 18, 2012 at 11:15:33AM -0700, Xinliang David

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 testcase

Re: [asan] migrate runtime from llvm

2012-10-18 Thread Xinliang David Li
On Thu, Oct 18, 2012 at 11:22 AM, Jakub Jelinek 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: [PATCH] Rs6000 infrastructure cleanup (switches), revised patch #2f

2012-10-18 Thread Dominique Dhumieres
> 2012-10-17 Michael Meissner > > * 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): Save area for > ... Boot

[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 * c-ada-spec.c (LOCATION_COL): Delete. (compare_location): New function.

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 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 assume that binutil

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

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 "\ > %{fprofile-arcs|f

Re: [asan] migrate runtime from llvm

2012-10-18 Thread Xinliang David Li
On Tue, Oct 16, 2012 at 12:27 AM, Jakub Jelinek 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 >> %{fgnu-tm:%:include(libitm.spec)%(link_itm)}\

[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 gcc/ * config/sh/sh.c: Fix comment to silence warning. diff --git a/gcc/config/sh/sh.c

[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; jani...@cod

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.

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

[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 PR c++/29633 * g++.dg/template/pr29633.C: New. Index: g++.dg/template/pr29633.C === --- g++.dg/te

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 wrote: > In GCC 4.8, module variables/procedures

[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 Ramana Radhakrishnan Sameera Deshpande * config/arm/cortex-a15-neon.md: New f

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

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 wrote: > On Thu, Oct 18, 2012 at 11:16 AM, Alexander Ivchenko > 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? >>

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

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 EN

Re: [asan] Fix some asan ICEs

2012-10-18 Thread Xinliang David Li
On Thu, Oct 18, 2012 at 5:58 AM, Jakub Jelinek 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, which is t

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

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 * except.c (sjlj_emit_function_enter): Remove unused variable. -- Eric BotcazouIndex: except.c ===

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

2012-10-18 Thread Eric Botcazou
> 2012-10-17 Michael Meissner > > * 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): Save area for > rs6

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

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 wrote: > In GCC 4.8, module variables/procedures

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 ext

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 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 mainline? > > > Segher >

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 wrote: On 10/18/2012 06:22 AM, Richard Biener wrote: On We

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 Greta Yorsh * config/arm/arm.c (thumb2_emit_strd_push): New function. (arm_expand_prologue): Use the n

[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 * gcc/config/rs6000/altivec.md (a

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 Greta Yorsh * config/arm/arm-protos.h (offset_ok_for_ldrd_strd): New declaration. (operands_ok_ldrd_strd)

[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

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

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

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 wrote: > > On 10/18/2012 06:22 AM, Richard Biener wrote: >> >> On Wed, Oct 17, 2012 at 11:47 PM, Kenneth Zadeck >> wrote: >>> >>> Richi, >>> >>> I apologize for the size of this patch, but it only does one very small >>> thing, it is just that it do

Re: building gcc with powerpc gold

2012-10-18 Thread David Edelsohn
On Wed, Oct 17, 2012 at 11:06 PM, Alan Modra 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 option. Both mac

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

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

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 Date: Wed, 17 Oct 2012 18:40:14 +0100 Subject: [PATCH 2/3] neon fma intrinsics. --- gcc/config/arm

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

[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___ is not defined for cores.

[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 Ramana Radhakrishnan * config/arm/arm

[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 x86_64-unknown-linux-gnu,

[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 * config/arm/lib1funcs.S (__ARM_ARCH__): Define for ARMv8

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 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 from > an integer cs

[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 btw).

[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 * gcc/common/config/i386/i386-common.c (OPTION_MASK_ISA_FXSAVE_SET): New.

[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 , the big fun remaining is piecewise_construct_t, which should allow tie and forward_as_tuple to be constexpr. I'm

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 hes

[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