Re: [google] Add testcase for relative paths for FDO (issue4354042)

On Fri, Apr 1, 2011 at 19:37, Martin Thuresson wrote: > 2011-04-01  Martin Thuresson   > >        PR gcov-profile/47793 >        * gcc/testsuite/gcc.dg/pr47793.c: New test OK. Diego.

Re: [x32] PR target/47744: [x32] ICE: in reload_cse_simplify_operands, at postreload.c:403

On Fri, Apr 1, 2011 at 10:30 PM, H.J. Lu wrote: > I checked in this patch. > > H.J. > --- > commit 230fad69a62607b1845bc04f4b33bdad398cc4e4 > Author: H.J. Lu > Date:   Thu Mar 17 18:26:35 2011 -0700 > >    Add ZERO_EXTEND PLUS base support to ix86_simplify_base_disp. > > diff --git a/gcc/ChangeLo

[PATCH] start a gitignore

I committed this to the sourceware repo a while ago before I was made aware of the top level needing to be kept in sync with gcc. So this syncs that commit from sourceware CVS to gcc SVN. --- .gitignore | 40 ChangeLog |4 2 files changed, 44

Re: PATCH: copy_addr_to_reg: Convert to Pmode if needed

On Mon, Mar 21, 2011 at 9:05 AM, Richard Sandiford wrote: > "H.J. Lu" writes: >> diff --git a/gcc/explow.c b/gcc/explow.c >> index 460af1f..9b3c082 100644 >> --- a/gcc/explow.c >> +++ b/gcc/explow.c >> @@ -611,6 +611,8 @@ copy_to_reg (rtx x) >>  rtx >>  copy_addr_to_reg (rtx x) >>  { >> +  if (GE

[x32] PR target/47744: [x32] ICE: in reload_cse_simplify_operands, at postreload.c:403

I checked in this patch. H.J. --- commit 230fad69a62607b1845bc04f4b33bdad398cc4e4 Author: H.J. Lu Date: Thu Mar 17 18:26:35 2011 -0700 Add ZERO_EXTEND PLUS base support to ix86_simplify_base_disp. diff --git a/gcc/ChangeLog.x32 b/gcc/ChangeLog.x32 index 6d5eda6..8dd7196 100644 --- a/gcc/C

patch to solve PR48380

The following patch is to solve the PR. The reason for the problem is described on http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48380 I also found some problem with function fix_reg_equiv_init (it calls grow_reg_equivs) whose code is never executed anymore because grow_reg_equivs is called be

Re: Scheduler cleanups, 1/N

On Thu, Mar 24, 2011 at 6:07 AM, Bernd Schmidt wrote: > I have a number of patches that will be necessary for a new target. Some > of these can be applied now as cleanups, so I'm submit them now. > > This changes the schedule_block main loop not to move instructions while > computing the schedule.

libgo patch committed: Sort test log files

This patch to libgo sorts the generated libgo.sum and libgo.log files into a consistent order. This will make it easier to compare them over time. It also now reports the net and syslog tests as UNTESTED when GCCGO_RUN_ALL_TESTS is not defined, rather than just skipping them. Bootstrapped and ran

Re: [patch, testsuite, ia64] patch for gcc.dg/mtune.c

On Fri, 1 Apr 2011, Steve Ellcey wrote: > Looking into why this doesn't happen on other systems, it looks like > the difference is that IA64 uses -mtune to set an enum variable and > most (all?) the other platforms use it to set a string variable. As an increasing number of targets will start usi

Add ToLower .opt facility

This patch adds a ToLower .opt feature, for option arguments that are case-insensitive and should be converted to a canonical lowercase form. This is relevant to at least mips, mn10300 and rx. This patch makes rx use it, thereby causing an existing spec that tested only for a lowercase option for

Re: support for dwarf AT_GNAT_descriptive_type

> I suppose this should wait until stage 1 reopens. I've installed the attached slight variant after testing on x86_64-suse-linux. The main change is the default value of LANG_HOOKS_DESCRIPTIVE_TYPE, NULL like other similar debug hooks. -- Eric Botcazou Index: langhooks-def.h =

Re: [RFC PATCH, go]: Port to ALPHA arch - sysinfo.go fixup

Rainer Orth writes: > I've got a similar issue on IRIX 6.5: has > > struct timeval { > #if _MIPS_SZLONG == 64 > __int32_t :32; > #endif > time_t tv_sec; /* seconds */ > longtv_usec;/* and microseconds */ > }; > > which causes the 64-bit libgo build to break

Re: [RFC PATCH, go]: Port to ALPHA arch - sysinfo.go fixup

On Wed, Mar 30, 2011 at 1:12 PM, Uros Bizjak wrote: > > d) The definition of "type Stat_t struct" also includes additional > struct and this confuses compilation. This seems like something that we could teach mksysinfo.sh to handle. What does the struct look like in gen-sysinfo.go? Ian

Re: [RFC PATCH, go]: Port to ALPHA arch

On Wed, Mar 30, 2011 at 12:58 PM, Uros Bizjak wrote: > > Attached ports go to ALPHA architecture. Thanks! Committed. > b) alpha doesn't define "struct user_regs_struct" from which "type > PtraceRegs" is derived. I have manually created PtraceRegs from > pt_regs structure and patched generated

Re: [libgo] Account for 32-bit fds_bits on Solaris 2

Rainer Orth writes: > Ian Lance Taylor writes: > >> Thanks. I'm going to try gambling that every uses the type "long" for >> the fds_bits array. If so, I think this patch will work. Bootstrapped >> and ran Go testsuite on x86_64-unknown-linux-gnu (forcing the use of >> select). Committed to

RE: [Patch,AVR]: Fix PR45263

> -Original Message- > From: Georg-Johann Lay [mailto:a...@gjlay.de] > Sent: Friday, April 01, 2011 4:01 PM > To: Georg-Johann Lay > Cc: gcc-patches@gcc.gnu.org; Denis Chertykov; Anatoly Sokolov; Weddington, > Eric > Subject: Re: [Patch,AVR]: Fix PR45263 > > This is a better fix that doe

Re: [Patch] [C++0x] Support decltype-specifier as start of nested-name-specifier. (Bug 6709)

Now that we're in stage 1 again, I'd like to get your decltype changes integrated. Sorry I didn't respond sooner. On 01/20/2011 11:51 AM, Adam Butcher wrote: The attached patch attempts to resolve bug http://gcc.gnu.org/bugzilla/show_bug.cgi?id=6709 Allowing decltype(expr) to appear as the st

Re: [Patch,AVR]: Fix PR45263

This is a better fix that does not need push/pop and does not increase numer of instructions. r16 takes the role of r20, and the value formerly in r16, i.e. hh8(__dtors_end resp. __ctors_start) is recreated as needed. 2011-04-02 Georg-Johann Lay PR target/45263 * config/avr/

Re: libgo patch committed: Add timeout for tests

On Apr 1, 2011, at 7:21 AM, Rainer Orth wrote: > Ian Lance Taylor writes: > >>> Shouldn't the default match the DejaGnu default of 300, though? >> >> Sure, it could, but why? If any of these tests fail to complete in a >> minute, something is badly wrong. > > Ok, I'll see how long they take on

Re: libgo patch committed: Add timeout for tests

On Apr 1, 2011, at 12:14 AM, Rainer Orth wrote: > Ian Lance Taylor writes: > >> This patch to libgo adds a timeout for the libgo tests. The default is >> 60 seconds. It can be changed by an argument to gotest, which would >> normally be used as, e.g., >> make GOTESTFLAGS="--timeout=120" check-

[PATCH] (partially) fix lto bootstrap (PR bootstrap/48148)

Hi! richi's lto-cgraph.c change actually didn't do anything, because DECL_ABSTRACT_ORIGIN is still NULL there, see PR for details. This patch instead doesn't try to force dies for DECL_ABSTRACT_ORIGIN DECL_EXTERNAL decls, as dwarf2out isn't prepared to handle them. Bootstrapped/regtested on x86_6

[patch, testsuite, ia64] patch for gcc.dg/mtune.c

The gcc.dg/mtune.c test is failing on ia64-*-* because in addition to the expected error message that is output it also prints out a note about the valid values for mtune. $ gcc -mtune=foo mtune.c gcc: error: unrecognized argument in option '-mtune=foo' gcc: note: valid arguments to '-mtune=' are:

Re: [PATCH 1/6] Disallow predicating the prologue

On 04/01/2011 08:59 PM, H.J. Lu wrote: > On Wed, Mar 23, 2011 at 7:45 AM, Bernd Schmidt > wrote: >> With prologues appearing in blocks other than the entry block, ifcvt can >> decide to predicate them. This is not a good idea, as dwarf2out will >> blow up trying to handle predicated frame-related

Re: [cxx-mem-model] bitfield tests

On Fri, Apr 1, 2011 at 9:24 AM, Richard Henderson wrote: > (1) Do we agree that all such cpus have user-level store insns with byte >    granularity.  Honestly the only non-microcontroler I ever heard of >    without this was the original Alpha.  Which is excluded per (0). And SPU which is exclud

Re: [patch, fortran] Fix PR 48352 - regression with ICE with front end optimization

Hi Jerry, Regression-tested. OK for trunk? OK. Do you plan to open a second PR for this TODO? Sendingfortran/ChangeLog Sendingfortran/frontend-passes.c Sendingtestsuite/ChangeLog Adding testsuite/gfortran.dg/function_optimize_3.f90 Transmitting file data

Re: testcase for PR c/36299

On Fri, 11 Mar 2011, Vincent Lefevre wrote: > As asked in PR c/36299[*], here's a testcase for this PR. It checks > the absence of warning for some valid C code, to make sure the warning > no longer reappears in future GCC versions. > > [*] http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36299#c10 I

Re: Split up toplevel library-disabling cases

On Fri, 1 Apr 2011, Michael Eager wrote: > > Target maintainers: I'd like to understand why it is necessary to > > disable libssp for AVR, AIX and Microblaze > > I believe that at some time in the past libssp failed to build for > MicroBlaze, but I don't recall the details. It currently builds >

Re: [PATCH] convert h8300 to constraints.md

On 02/14/2011 04:19 PM, Nathan Froyd wrote: > +(define_constraint "WU" > + "@internal" > + (and (match_code "mem") > + (match_test "satisfies_constraint_U (op)"))) Based on -#define EXTRA_MEMORY_CONSTRAINT(C, STR) \ - ((C) == 'W') This one should be define_memory_constraint. Otherwise

[PATCH] make CONST_DECL a direct subclass of tree_decl_common

Nobody touches CONST_DECL's DECL_RTL, so there's no reason to for CONST_DECL to inherit from tree_decl_with_rtl. This patch adjusts the inheritance structure accordingly. Bootstrapped on x86_64-unknown-linux-gnu; tests in progress. OK to commit once those complete? -Nathan * tree.h (st

Re: [PATCH 1/6] Disallow predicating the prologue

On Wed, Mar 23, 2011 at 7:45 AM, Bernd Schmidt wrote: > With prologues appearing in blocks other than the entry block, ifcvt can > decide to predicate them. This is not a good idea, as dwarf2out will > blow up trying to handle predicated frame-related things. > One of your changes breaks GCC boot

Re: [patch i386]: Set for method-functions default calling-convention to thiscall for 32-bit msabi

On 04/01/2011 11:36 AM, Kai Tietz wrote: > It might be a good thing to extract those call-abi lookups into a > specific function, which returns numbers for the selected on. By this > we can avoid double lookup of attributes here. > But I would like to do this in a separate patch. Sure. r~

[wwwdocs] Buildstat update for 4.6

Please disregard previous patch, it had the entries in the wrong order. First round of results for the new release. -tgc Testresults for 4.6.0: alpha-dec-osf5.1b (2) armv7l-unknown-linux-gnueabi hppa2.0w-hp-hpux11.11 hppa64-hp-hpux11.11 i386-pc-solaris2.8 (5) i386-pc-solaris2.9 (3)

Re: Split up toplevel library-disabling cases

Joseph S. Myers wrote: This patch, relative to a tree with (pending review) applied, continues toplevel configure cleanup by splitting the disabling of some libraries into separate case statements for those libraries. Separating the logic

Re: [patch i386]: Set for method-functions default calling-convention to thiscall for 32-bit msabi

2011/4/1 Richard Henderson : > On 04/01/2011 11:10 AM, Kai Tietz wrote: >>       * i386.c (ix86_is_msabi_thiscall): New helper function. >>       (ix86_is_type_thiscall): New helper function. >>       (ix86_comp_type_attributes): Handle thiscall for method-functions >>       special. >>       (init

Re: Try simplifying memory operands during optabs expansion

On 03/31/2011 03:19 AM, Richard Sandiford wrote: > * recog.h (insn_operand_data): Add an "allows_mem" field. > * genoutput.c (output_operand_data): Initialize it. > * optabs.c (maybe_legitimize_operand_same_code): New function. > (maybe_legitimize_operand): Use it when match

[wwwdocs] Buildstat update for 4.6

First round of results for the new release. -tgc Testresults for 4.6.0: alpha-dec-osf5.1b (2) armv7l-unknown-linux-gnueabi hppa2.0w-hp-hpux11.11 hppa64-hp-hpux11.11 i386-pc-solaris2.8 (5) i386-pc-solaris2.9 (3) i386-pc-solaris2.10 (4) i386-pc-solaris2.11 (4) i686-pc-linux-gnu (3

Re: [patch i386]: Set for method-functions default calling-convention to thiscall for 32-bit msabi

On 04/01/2011 11:10 AM, Kai Tietz wrote: > * i386.c (ix86_is_msabi_thiscall): New helper function. > (ix86_is_type_thiscall): New helper function. > (ix86_comp_type_attributes): Handle thiscall for method-functions > special. > (init_cumulative_args): Likewise. >

Re: [PATCH], PR 48262, fix failing vector support on powerpc

On Thu, Mar 31, 2011 at 2:37 PM, Michael Meissner wrote: > 2011-03-31  Andrew Pinski   >            Michael Meissner   > >        PR target/48262 >        * config/rs6000/vector.md (movmisalign): Allow for memory >        operands, as per the specifications. > >        * config/rs6000/altivec.md

[Patch,AVR]: Fix PR45263

This is straight forward fix for PR45263 (R20 clobbered by constructor/destructor). Using call-saved register like R15 is undesired because upcoming ATtiny10 architecture. 2011-04-01 Georg-Johann Lay PR target/45263 * config/avr/libgcc.S (__do_global_ctors, __do_global_dtors):

Re: [toplevel] remove tentative_cc

* Joseph S. Myers wrote on Fri, Apr 01, 2011 at 07:04:30PM CEST: > I see there's more than just tentative_cc that tries to deal with compiler > settings in ways that should be the responsibility of autoconf. There's > the > > # Make sure that the compiler is able to generate an executable. If

Re: [patch i386]: Set for method-functions default calling-convention to thiscall for 32-bit msabi

2011/4/1 Nathan Froyd : > On Fri, Apr 01, 2011 at 07:39:13PM +0200, Kai Tietz wrote: >> this patch sets for c++ methods, which are not stdarg ones, the >> default calling-convention to thiscall for ms_abi 32-bit, as vendor >> specific compiler does. > > I think it might be worthwhile to pull this p

[PING] [PATCH] convert h8300 to constraints.md

Hi. - Original Message - From: "Nathan Froyd" Sent: Tuesday, February 15, 2011 4:19 AM Subject: [PATCH] convert h8300 to constraints.md Tested with cross to h8300-elf (which doesn't totally build right now, but the patch doesn't make things any worse). OK to commit? http://gcc.gn

Re: [patch i386]: Set for method-functions default calling-convention to thiscall for 32-bit msabi

On Fri, Apr 01, 2011 at 07:39:13PM +0200, Kai Tietz wrote: > this patch sets for c++ methods, which are not stdarg ones, the > default calling-convention to thiscall for ms_abi 32-bit, as vendor > specific compiler does. I think it might be worthwhile to pull this pattern: > @@ -5500,7 +5524,8 @@

[patch i386]: Set for method-functions default calling-convention to thiscall for 32-bit msabi

Hello, this patch sets for c++ methods, which are not stdarg ones, the default calling-convention to thiscall for ms_abi 32-bit, as vendor specific compiler does. ChangeLog 2011-04-01 Kai Tietz * i386.c (ix86_is_msabi_thiscall): New helper function. (ix86_comp_type_attributes):

[google] Add testcase for relative paths for FDO (issue4354042)

This patch should go into google/main. Add testcase to verify that relative profile paths workes. Bootstraped without regression on x86_64. 2011-04-01 Martin Thuresson PR gcov-profile/47793 * gcc/testsuite/gcc.dg/pr47793.c: New test Index: gcc/testsuite/gcc.dg/pr47793.c ===

Re: [PING]RE: [H8300] Hookize target macros

On 04/01/2011 10:25 AM, Anatoly Sokolov wrote: > * config/h8300/h8300.c (print_operand_address): Rename to... > (h8300_print_operand_address): ...this. Make static. Adjust comments. > Call h8300_print_operand and h8300_print_operand_address instead of > print_operand

Re: [PING]RE: [H8300] Hookize target macros

Hi. On 09/17/2010 12:54:36, Naveen H. S wrote: > Can you please commit the patch if it's OK? > Thanks & Regards, > Naveen > -Original Message- > From: Richard Henderson [mailto:r...@redhat.com] > Sent: Friday, September 03, 2010 8:34 PM > To: Naveen H. S > Cc: gcc-patches@gcc.gnu.org; P

[commit] Fix SPU build issue (Re: Remove US_SOFTWARE_GOFAST)

Joseph Myers wrote: > Index: gcc/config/spu/t-spu-elf > === > --- gcc/config/spu/t-spu-elf (revision 167746) > +++ gcc/config/spu/t-spu-elf (working copy) > dp-bit.c: $(srcdir)/config/fp-bit.c $(srcdir)/config/spu/t-spu-elf > -

Re: [toplevel] remove tentative_cc

I see there's more than just tentative_cc that tries to deal with compiler settings in ways that should be the responsibility of autoconf. There's the # Make sure that the compiler is able to generate an executable. If it # can't, we are probably in trouble. We don't care whether we can run t

Re: [cxx-mem-model] bitfield tests

On 03/31/2011 08:28 AM, Richard Guenther wrote: >>> Well, I'm not sure that strict-align targets that provide byte access do >>> not simply hide the issue inside the CPU (thus, perform the >>> read-modify-write >>> there and do not guarantee any atomicity unless you ask for it). >> Certainly some

Re: [PATCH, PR43920, 8/9] Cross-jumping - Extend search scope - test case.

On 04/01/2011 05:34 PM, Richard Earnshaw wrote: > On Fri, 2011-04-01 at 16:47 +0200, Tom de Vries wrote: > Reposting, with ChangeLog. > > +/* { dg-options "-march=armv7-a -mthumb -Os" } */ > > No, use dg-require-effective-target. The above doesn't work properly > with multilib testing. > Changed

Re: [PATCH, PR43920, 2/9] ARM specific part - test case

On 04/01/2011 05:34 PM, Richard Earnshaw wrote: > On Fri, 2011-04-01 at 16:47 +0200, Tom de Vries wrote: >> Reposting, with ChangeLog. > > +/* { dg-options "-march=armv7-a -mthumb -Os" } */ > > No, use dg-require-effective-target. The above doesn't work properly > with multilib testing. > Chan

Re: [PATCH, PR43920, 1/9] ARM specific part.

On 04/01/2011 05:18 PM, Richard Earnshaw wrote: > > On Fri, 2011-04-01 at 16:45 +0200, Tom de Vries wrote: >> Reposting, with ChangeLog. > > #define BRANCH_COST(speed_p, predictable_p) \ > - (TARGET_32BIT ? 4 : (optimize > 0 ? 2 : 0)) > + (TARGET_32BIT ? (TARGET_THUMB2 && optimize_size ? 1 : 4

Re: [PATCH, PR43920] Improve code-size optimizations

> Sorry about that. I reposted the patches with ChangeLog entries. Thanks! -- Eric Botcazou

Re: [PATCH] Fix expansion issues on type changing MEM_REFs on LHS (PR middle-end/48335)

> I'm now bootstrapping/regtesting following adjusted patch, ok if it passes? > > 2011-04-01 Jakub Jelinek > > PR middle-end/48335 > * expr.c (expand_assignment): Handle all possibilities > if TO_RTX is CONCAT. > * expmed.c (store_bit_field_1): Avoid trying to create >

Re: [PATCH] Fix expansion issues on type changing MEM_REFs on LHS (PR middle-end/48335)

On Fri, Apr 01, 2011 at 05:35:53PM +0200, Eric Botcazou wrote: > > result is set there from store_field, and result is only used in > > if (result) > > preserve_temp_slots (result); > > afterwards. store_field might want to preserve_temp_slots perhaps, so > > clearing result afterwar

Re: [debug] Don't add DW_AT_low_pc if the CU has no associated code

On 03/31/2011 02:17 PM, Mark Wielaard wrote: > * dwarf2out.c (dwarf2out_finish): Don't add low_pc and/or > high_pc attribute if the CU has no associated code. Only output > DW_AT_entry_pc for CU if not generating strict dwarf and > dwarf_version < 4. Ok. r~

Re: [PATCH] Fix expansion issues on type changing MEM_REFs on LHS (PR middle-end/48335)

> result is set there from store_field, and result is only used in > if (result) > preserve_temp_slots (result); > afterwards. store_field might want to preserve_temp_slots perhaps, so > clearing result afterwards might be wrong. You're very likely right, but then this should apply

Re: [PATCH, PR43920, 2/9] ARM specific part - test case

On Fri, 2011-04-01 at 16:47 +0200, Tom de Vries wrote: > Reposting, with ChangeLog. +/* { dg-options "-march=armv7-a -mthumb -Os" } */ No, use dg-require-effective-target. The above doesn't work properly with multilib testing. R.

Re: [PATCH, PR43920, 1/9] ARM specific part.

On Fri, 2011-04-01 at 16:45 +0200, Tom de Vries wrote: > Reposting, with ChangeLog. #define BRANCH_COST(speed_p, predictable_p) \ - (TARGET_32BIT ? 4 : (optimize > 0 ? 2 : 0)) + (TARGET_32BIT ? (TARGET_THUMB2 && optimize_size ? 1 : 4) \ + : (optimize > 0 ? 2 : 0)) Don't use opti

Re: [PATCH, PR43920, 8/9] Cross-jumping - Extend search scope - test case.

On 04/01/2011 05:01 PM, Jakub Jelinek wrote: > On Fri, Apr 01, 2011 at 04:56:10PM +0200, Tom de Vries wrote: >> Reposting, with ChangeLog. > >> 2011-04-01 Tom de Vries >> >> PR target/43920 >> * testsuite/gcc.target/arm/pr43920-2.c: New test. > > gcc/testsuite/ has its own ChangeLog,

Re: [PATCH, PR43920, 2/9] ARM specific part - test case

On 04/01/2011 05:01 PM, Jakub Jelinek wrote: > On Fri, Apr 01, 2011 at 04:56:10PM +0200, Tom de Vries wrote: >> Reposting, with ChangeLog. > > gcc/testsuite/ has its own ChangeLog, so the ChangeLog entry > should say just > * gcc.target/arm/pr43920-2.c: New test. > > Jakub Fixed Change

Re: [PATCH, PR43920, 3/9] Cleanup.

On 04/01/2011 05:01 PM, Jakub Jelinek wrote: > On Fri, Apr 01, 2011 at 04:56:10PM +0200, Tom de Vries wrote: >> Reposting, with ChangeLog. > > Similarly one of your patches had > * gcc/cfgcleanup.c (whatever): Whatever. > in the ChangeLog, should be just > * cfgcleanup.c (whatever): Wha

Re: [PATCH] ICE on mainline bootstrap with powerpc64 -mcmodel={medium,large}

On 03/31/2011 10:41 PM, Alan Modra wrote: > * dwarf2out.c (mem_loc_descriptor): Recurse on LO_SUM. Ok. r~

Re: [PATCH, PR43920] Improve code-size optimizations

Hi Eric, On 03/31/2011 11:02 PM, Eric Botcazou wrote: >> Patches will be posted separately. > > Always post a ChangeLog entry with a patch. > Sorry about that. I reposted the patches with ChangeLog entries. Thanks, - Tom

Re: [PATCH] Fix expansion issues on type changing MEM_REFs on LHS (PR middle-end/48335)

On Fri, Apr 01, 2011 at 01:46:55PM +0200, Jakub Jelinek wrote: > Here is the updated patch, without the result = NULL addition but > instead with remembering result also from the only case which was > clearing result. out-of-bounds changed also in the ChangeLog entry. Bootstrapped/regtested now o

Re: [PATCH, PR43920, 8/9] Cross-jumping - Extend search scope - test case.

On Fri, Apr 01, 2011 at 04:56:10PM +0200, Tom de Vries wrote: > Reposting, with ChangeLog. > 2011-04-01 Tom de Vries > > PR target/43920 > * testsuite/gcc.target/arm/pr43920-2.c: New test. gcc/testsuite/ has its own ChangeLog, so the ChangeLog entry should say just * gcc.t

[PATCH] Avoid bitfield stores to clobber adjacent variables (PR middle-end/48124)

Hi! This patch changes get_best_mode, so that it doesn't suggest using larger modes if it means it could clobber variables located after the containing one. On the attached testcase on x86_64, the structure is 12 bytes wide, and the last bitfield occupies 14 bits in the bytes 8 and 9 from the beg

Re: [PATCH, PR43920, 9/9] Cross-jumping - Allow both directions.

Reposting, with ChangeLog. 2011-04-01 Tom de Vries PR target/43920 * cfgcleanup.c (try_crossjump_to_edge): Add dir parameter. Pass dir to flow_find_cross_jump. Swap variables to implement backward replacement. (try_crossjump_bb): Add argument to try_crossjump_to_edge. diff -u gcc/cfgclea

Re: [PATCH, PR43920, 8/9] Cross-jumping - Extend search scope - test case.

Reposting, with ChangeLog. 2011-04-01 Tom de Vries PR target/43920 * testsuite/gcc.target/arm/pr43920-2.c: New test. Index: gcc/testsuite/gcc.target/arm/pr43920-2.c === --- gcc/testsuite/gcc.target/arm/pr43920-2.c (revision 0) +

Re: [PATCH, PR43920, 7/9] Cross-jumping - Extend search scope.

Reposting, with ChangeLog. 2011-04-01 Tom de Vries PR target/43920 * cfgcleanup.c (walk_to_nondebug_insn): New function. (flow_find_cross_jump): Use walk_to_nondebug_insn. Recalculate bb1 and bb2. (try_crossjump_to_edge): Handle case that newpos1 or newpos2 is not src1 or src2. Redirec

Re: [PATCH, PR43920, 6/9] Cross-jumping - Use reg-notes.

On 03/31/2011 11:16 PM, Tom de Vries wrote: > On 03/31/2011 08:52 PM, Jeff Law wrote: > >> On 03/31/11 12:42, Tom de Vries wrote: >>> Uses regnotes to analyze whether we can replace insn a by insn b, even >>> if we cannot replace insn b by insn a. Uses this info in crossjumping. > >> Shouldn't th

Re: [PATCH, PR43920, 5/9] Cross-jumping - Add missing use of return register.

Reposting, with ChangeLog. 2011-04-01 Tom de Vries PR target/43920 * function.c (emit_use_return_register_into_block): New function. (thread_prologue_and_epilogue_insns): Use emit_use_return_register_into_block. Index: gcc/function.c

Re: [PATCH, PR43920, 4/9] Cross-jumping - Don't count use or clobber.

Reposting, with ChangeLog. 2011-04-01 Tom de Vries PR target/43920 * cfgcleanup.c (flow_find_cross_jump): Don't count USE or CLOBBER as insn. Index: gcc/cfgcleanup.c === --- gcc/cfgcleanup.c (revision 170556) +++ gcc/cfgcleanup

Re: [PATCH, PR43920, 3/9] Cleanup.

Reposting, with ChangeLog. 2011-04-01 Tom de Vries PR target/43920 * gcc/cfgcleanup.c (try_crossjump_bb): Remove 2 superfluous variables. diff -u gcc/cfgcleanup.c gcc/cfgcleanup.c --- gcc/cfgcleanup.c (working copy) +++ gcc/cfgcleanup.c (working copy) @@ -1961,7 +1961,6 @@ edge e, e2, fal

Re: [PATCH, PR43920, 2/9] ARM specific part - test case

Reposting, with ChangeLog. 2011-04-01 Tom de Vries PR target/43920 * gcc/testsuite/gcc.target/arm/pr43920-1.c: New test. Index: gcc/testsuite/gcc.target/arm/pr43920-1.c === --- gcc/testsuite/gcc.target/arm/pr43920-1.c (revision

Re: [PATCH, PR43920, 1/9] ARM specific part.

Reposting, with ChangeLog. 2011-04-01 Tom de Vries PR target/43920 * config/arm/arm.h (BRANCH_COST): Set to 1 for Thumb-2 when optimizing for size. Index: gcc/config/arm/arm.h === --- gcc/config/arm/arm.h (revision 293961) +++

Re: [patch mingw]: Enable optional thread-model posix for mingw-targets

2011/3/31 Ralf Wildenhues : > * Kai Tietz wrote on Thu, Mar 31, 2011 at 12:19:51PM CEST: >> --- gcc.orig/gcc/config.gcc   2011-03-23 21:15:32.0 +0100 >> +++ gcc/gcc/config.gcc        2011-03-31 10:50:05.559129000 +0200 > >> @@ -1420,6 +1420,10 @@ i[34567]86-*-mingw* | x86_64-*-mingw*) >>  

Re: libgo patch committed: Add timeout for tests

Ian Lance Taylor writes: >> Shouldn't the default match the DejaGnu default of 300, though? > > Sure, it could, but why? If any of these tests fail to complete in a > minute, something is badly wrong. Ok, I'll see how long they take on my 250 MHz R10k MIPS :-) Rainer -- -

Re: libgo patch committed: Add timeout for tests

Rainer Orth writes: > Ian Lance Taylor writes: > >> This patch to libgo adds a timeout for the libgo tests. The default is >> 60 seconds. It can be changed by an argument to gotest, which would >> normally be used as, e.g., >> make GOTESTFLAGS="--timeout=120" check-target-libgo > > Great, th

Re: [patch, fortran] Extend character optimization to LLE and friends

On 04/01/2011 06:29 AM, Jerry DeLisle wrote: On 03/28/2011 11:50 AM, Thomas Koenig wrote: Hello world, the attached patch extends the character optimizations to the lexical comparison functions (LLE and friends). Regression-tested. OK for trunk? Yes, this patch is OK? Fat fingers, s/?/./

[patch][cprop.c] Allow splitting of critical edges to expose implicit sets

Hi, With this patch, it is OK for find_implicit_sets to split an edge if that makes it possible to record an implicit set on that edge. This is possible and cheap now because CPROP runs in cfglayout mode. Bootstrapped and tested on x86_64-unknown-linux-gnu. Shaves off ~1kb of cc1, which isn't ver

Re: [patch, fortran] Extend character optimization to LLE and friends

On 03/28/2011 11:50 AM, Thomas Koenig wrote: Hello world, the attached patch extends the character optimizations to the lexical comparison functions (LLE and friends). Regression-tested. OK for trunk? Yes, this patch is OK? Thanks, Jerry

[PATCH] macroize some rs6000 call patterns

This patch does as $SUBJECT suggests. It doesn't macroize all the call insns, as iterators don't support substituting CONST_INTs, but it does clean up quite a bit. Tested on powerpc64-unknown-linux-gnu. OK to commit? -Nathan * config/rs6000/rs6000.md (*call_local32): Rename to...

Re: Cleaning up expand optabs code

Richard Sandiford schrieb: > Georg-Johann Lay writes: >> Richard Henderson schrieb: >>> On 03/25/2011 05:41 AM, Georg-Johann Lay wrote: > On 03/22/2011 06:48 PM, Richard Henderson wrote: > >> Ok. Watch out for other target problems this week. libgcc fails to build for avr (SVN 17

Re: Remove old host cases from toplevel configure

* Joseph S. Myers wrote on Fri, Apr 01, 2011 at 02:40:45PM CEST: > OK to commit this patch to remove this code? OK. Thanks, Ralf > 2011-04-01 Joseph Myers > > * configure.ac: Remove code setting CONFIG_SHELL, config_shell and > moveifchange. > * configure: Regenerate. >

Re: More old target toplevel cleanup

On 04/01/2011 02:34 PM, Joseph S. Myers wrote: This patch does some more toplevel configure cleanup related to targets no longer supported by GCC: * The code to enable libgomp on hosted POSIX systems handled *-*-sysv4*; GCC no longer supports any such targets. * The md_exec_prefix code covered

Re: Remove old host cases from toplevel configure

On Fri, 25 Mar 2011, Paolo Bonzini wrote: > On 03/24/2011 08:49 PM, Joseph S. Myers wrote: > > Related to that point, I notice a piece of code starting "we might need to > > use some other shell than /bin/sh for running subshells" that tries to > > determine a shell on Windows hosts. It's autocon

More old target toplevel cleanup

This patch does some more toplevel configure cleanup related to targets no longer supported by GCC: * The code to enable libgomp on hosted POSIX systems handled *-*-sysv4*; GCC no longer supports any such targets. * The md_exec_prefix code covered lots of targets that either are no longer supp

Re: [patch] c-decl.c(grokdeclarator) trivial formating fixes

This patch is OK. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH] Fix expansion issues on type changing MEM_REFs on LHS (PR middle-end/48335)

Hi! Here is the updated patch, without the result = NULL addition but instead with remembering result also from the only case which was clearing result. out-of-bounds changed also in the ChangeLog entry. 2011-04-01 Jakub Jelinek PR middle-end/48335 * expr.c (expand_assignment

Re: [PATCH] Fix expansion issues on type changing MEM_REFs on LHS (PR middle-end/48335)

On Fri, Apr 01, 2011 at 12:52:04PM +0200, Eric Botcazou wrote: > > --- gcc/expr.c.jj 2011-03-23 17:15:55.0 +0100 > > +++ gcc/expr.c 2011-03-30 11:38:15.0 +0200 > > @@ -4278,16 +4278,47 @@ expand_assignment (tree to, tree from, b > >/* Handle expand_expr of a complex v

Re: [PATCH] Fix expansion issues on type changing MEM_REFs on LHS (PR middle-end/48335)

> --- gcc/expr.c.jj 2011-03-23 17:15:55.0 +0100 > +++ gcc/expr.c2011-03-30 11:38:15.0 +0200 > @@ -4278,16 +4278,47 @@ expand_assignment (tree to, tree from, b >/* Handle expand_expr of a complex value returning a CONCAT. */ >else if (GET_CODE (to_rtx) ==

Re: [3/3] Record the number of generator arguments in insn_data

Richard Henderson writes: > On 03/31/2011 01:09 PM, Richard Sandiford wrote: >>> I think the assert should be retained (for now) as >>> >>> gcc_assert (nops == 4 || nops == 6); >>> >>> at least until we add such verification to some genfoo. >> >> After the patch we have: >> >> gcc_assert (no

Re: [libgo] Account for 32-bit fds_bits on Solaris 2

Ian Lance Taylor writes: > Thanks. I'm going to try gambling that every uses the type "long" for > the fds_bits array. If so, I think this patch will work. Bootstrapped > and ran Go testsuite on x86_64-unknown-linux-gnu (forcing the use of > select). Committed to mainline. I fear you lost ;-

Re: [wwwdocs]

Background/Test: The cvs command needs to know the compression level while using the -z option. Hence we need to provide compression level for network traffic ranging from 0 to 9. 0 for default. This value for -z command is not mentioned in the 2nd step. Hence it gives error. On Fri, Apr 1, 2011

[wwwdocs]

Found a bug on the cvs.html web page. How to start the web page patching I followed the instruction on the http://gcc.gnu.org/cvs.html page, downloaded the wwwdocs via cvs, validated the web page to be updated. Now I want to commit the changes to the repo. But for that I need write access(obviously

Re: libgo patch committed: Add timeout for tests

Ian Lance Taylor writes: > This patch to libgo adds a timeout for the libgo tests. The default is > 60 seconds. It can be changed by an argument to gotest, which would > normally be used as, e.g., > make GOTESTFLAGS="--timeout=120" check-target-libgo Great, thanks. Shouldn't the default mat