Re: Random cleanups [2/4]: canonicalize ctor values

On 03/31/2011 03:22 AM, Michael Matz wrote: this came up when looking into why the static ctors contain useless trees (like casts). We can simply canonicalize them while varpool analyzes pending decls. It'll look at initialzers once, where we can "gimplify" them. This requires making canonical

Re: fix rs6000 break

- Original Message From: Jakub Jelinek To: Andreas Tobler Cc: jos...@codesourcery.com, gcc-patches@gcc.gnu.org Subject: Re: fix rs6000 break Date: 31/03/11 08:44 > On Thu, Mar 31, 2011 at 08:30:42AM +0200, Andreas Tobler wrote: > > Hello, > > > > Is this ok to apply? > > That

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

Uros Bizjak writes: > d) The definition of "type Stat_t struct" also includes additional > struct and this confuses compilation. 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 */

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

On Wed, Mar 30, 2011 at 6:05 PM, Jakub Jelinek wrote: > Hi! > > MEM_REFs which can represent type punning on lhs don't force > non-gimple types to be addressable.  This causes various problems > in the expander, which wasn't prepared to handle that. > > This patch tries to fix what I've found and

Re: RFC: PATCH: Remove 26 element limit in vector

On Wed, Mar 30, 2011 at 8:09 PM, H.J. Lu wrote: > On Wed, Mar 30, 2011 at 08:02:38AM -0700, H.J. Lu wrote: >> Hi, >> >> Currently, we limit XVECEXP to 26 elements in machine description >> since we use letters 'a' to 'z' to encode them.  I don't see any >> reason why we can't go beyond 'z'.  This

Re: Fix pass49-frag (mudflap support for varargs)

On Wed, Mar 30, 2011 at 11:17 PM, Michael Matz wrote: > Hi, > > for some reasons the FAIL of pass49-frag now annoyed me enough to look > into it, where it didn't do so for the last 50 years it's failing (give or > take a few).  mudflap has a mean to mark some expressions as not > interesting to ge

Re: [google] Disable guality tests (issue4328047)

On Wed, Mar 30, 2011 at 11:42 PM, Diego Novillo wrote: > > The guality tests are failing/passing intermittently.  This confuses > our builders which expect 0 failures and 0 xpasses.  When they "work", > it's not that they are working but dejagnu failed to launch gdb: > > --

Re: [patch][cprop.c] Clean up hash table building

On Thu, Mar 31, 2011 at 12:39 AM, Steven Bosscher wrote: > Hi, > > This is the first cleanup for cprop.c. These cleanups are only > possible now, thanks to splitting the CPROP code out from gcse.c > > The first change is to not treat unfolded conditions as constant in > gcse_constant_p(). This nev

Re: [debug] Introduce -fno-debug-types-section flag

On Tue, Mar 29, 2011 at 12:20:32PM +0200, Mark Wielaard wrote: > Some dwarf consumers (dwarves, systemtap and dwarflint) grok Dwarf V4, > but not yet the new .debug_types section. To make it easier to use these > tools with -gdwarf-4 this patch introduces a new > -fno-debug-types-section flag and g

[patch] add hp sfc ops DWARF-2 subcodes

Hi, this patch simply adds the DW_LNE_HP_SFC subcodes, to be used by binutils. Ok for trunk ? (Sanity-checked by rebuilding gcc). Tristan. include/ 2011-03-31 Tristan Gingold * dwarf2.h (dwarf_line_number_hp_sfc_ops): New enum. --- a/include/dwarf2.h +++ b/include/dwarf2.h @@ -757,6

Re: Random cleanups [4/4]: Streamlining streamer

On Thu, Mar 31, 2011 at 4:07 AM, Michael Matz wrote: > Hi, > > I fear I wasn't as thorough in also splitting this one into several > patches, but the different cleanups are at least mostly in different > files.  They are: > > * lto-lang remembers all builtin decls in a local list, to be returned >

Re: [PATCH] PR debug/47471 (set prologue_end in .debug_line)

On Thu, 31 Mar 2011 04:59:18 +0200, Richard Henderson wrote: > On 03/30/2011 11:19 AM, Dodji Seketeli wrote: > > First, it avoids emitting two consecutive .loc that are identical. > > Strictly speaking that should fix this issue in this particular case. > > What's the compatibility strategy? I.e.

Re: Random cleanups [2/4]: canonicalize ctor values

On Thu, Mar 31, 2011 at 9:14 AM, Paolo Bonzini wrote: > On 03/31/2011 03:22 AM, Michael Matz wrote: >> >> this came up when looking into why the static ctors contain useless trees >> (like casts).  We can simply canonicalize them while varpool analyzes >> pending decls.  It'll look at initialzers

RE: [Patch] Bfin: Ensure rotrsi and rotlsi don't accept non-const INTVALS

Seems reasonable to me, thanks! Patch updated. Stu 2011-03-29 Stuart Henderson From Bernd Schmidt * config/bfin/bfin.md (rotrsi, rotlsi): Don't take INTVAL of anything that's not CONST_INT. -Original Message- From: Richard Henderson [mailto:r...@redhat.com] Sent: 30 Marc

Re: PING^2 [PATCH] Support for AMD64 targets running GNU/kFreeBSD

Ping^2 2011/1/26 Robert Millan : > Ping! > > 2011/1/18 Robert Millan : >> 2011/1/14 Robert Millan : >>> 2011/1/12 Robert Millan : > * The headers config/kfreebsd-gnu.h etc. override >  GLIBC_DYNAMIC_LINKER.  But the 64-bit configurations >  x86_64-*-kfreebsd*-gnu and x86_64-*-knetbsd*-

Re: RFC: PATCH: Remove 26 element limit in vector

Richard Guenther writes: > On Wed, Mar 30, 2011 at 8:09 PM, H.J. Lu wrote: >> On Wed, Mar 30, 2011 at 08:02:38AM -0700, H.J. Lu wrote: >>> Hi, >>> >>> Currently, we limit XVECEXP to 26 elements in machine description >>> since we use letters 'a' to 'z' to encode them.  I don't see any >>> reason

Re: Cleaning up expand optabs code

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 171446) >> >> ../../../../../gcc.gnu.org/trunk/libgcc/../gcc/libgcc2.c

Re: [x86] Fix PR target/48142

Hello! > this is a regression present for x86-64 on mainline and 4.6 branch with the > options -Os -mpreferred-stack-boundary=5 -fstack-check > -fno-omit-frame-pointer. > This improbable combination of options is necessary because you need to have > stack checking + stack realignment + !ACCUMULAT

[PATCH] Fix PR48246

The LTO partitioning code asserts that at most the first partition is empty (which should probably be also the last one). It works fine when other partitions are empty so there is no reason to barf on users when they run into such partitioning cases. Thus, committed as obvious. Richard. 2011-0

Re: [x86] Fix PR target/48142

On Thu, Mar 31, 2011 at 11:58:29AM +0200, Uros Bizjak wrote: > + /* Even if the stack pointer isn't the CFA register, we need to correctly > + describe the adjustments made to it, in particular differentiate the > + frame-related ones from the frame-unrelated ones. */ > + if (size > 0) >

Re: [patch, ARM] Fix PR48250, adjust DImode reload address legitimizing

On Thu, 2011-03-31 at 11:33 +0800, Chung-Lin Tang wrote: > On 2011/3/30 05:28 PM, Richard Earnshaw wrote: > > > > On Wed, 2011-03-30 at 15:35 +0800, Chung-Lin Tang wrote: > >> On 2011/3/30 上午 12:23, Richard Earnshaw wrote: > >>> > >>> On Tue, 2011-03-29 at 22:53 +0800, Chung-Lin Tang wrote: > >>>

Re: RFC: PATCH: Remove 26 element limit in vector

On Mar 31, 2011, at 1:41 AM, Richard Guenther wrote: > On Wed, Mar 30, 2011 at 8:09 PM, H.J. Lu wrote: >> On Wed, Mar 30, 2011 at 08:02:38AM -0700, H.J. Lu wrote: >>> Hi, >>> >>> Currently, we limit XVECEXP to 26 elements in machine description >>> since we use letters 'a' to 'z' to encode them.

RE: [Patch] improve bfin conditional move support

e.g. > - operands[1] = bfin_gen_compare (operands[1], SImode); > + operands[1] = bfin_gen_compare (operands[1], mode); ? Updated patch attached. Thanks, Stu 2011-03-29 Stuart Henderson From Bernd Schmidt: * config/bfin/bfin.md (MOVCC): New mode_macro. (movcc_insn1, movcc_insn2,

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

Hello, This patch enables for native windows targets the posix threading-model by using pthread-library. To build toolchain defaulting to posx-threading model it is necessary to use winpthread library for a complete bootstrap. The pthread-win32 library isn't suitable as it uses for the pthread_

Try simplifying memory operands during optabs expansion

This is the main change that motivated the less-than-successful optabs interface "cleanup". Tree expansion yields MEMs that are legitimiate memory_operands, but it's not uncommon for some optabs to need a more restricted type of address. This patch records whether operands can be memories, and ma

Re: [x86] Fix PR target/48142

On Thu, Mar 31, 2011 at 12:09 PM, Jakub Jelinek wrote: > On Thu, Mar 31, 2011 at 11:58:29AM +0200, Uros Bizjak wrote: >> +  /* Even if the stack pointer isn't the CFA register, we need to correctly >> +     describe the adjustments made to it, in particular differentiate the >> +     frame-related

[Patch] bfin: move loop invariants out of loop

The attached patch attempts to move loop invariants out of loops for Blackfin. 2011-03-31 Stuart Henderson From Bernd Schmidt * config/bfin/bfin.c (bfin_gen_bundles): Don't try to bundle a jump. (bfin_optimize_loops_1, move_loop_constants): New static functions. (bfin_reorg): Call

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

* 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.gcc2011-03-31 10:50:05.559129000 +0200 > @@ -1420,6 +1420,10 @@ i[34567]86-*-mingw* | x86_64-*-mingw*) > *) >

[patch, ARM] Make 128 bits the default vector size for NEON

Hi, This patch changes NEON's default vector size from 64 to 128 bits. The patch doesn't touch mvectorize-with-neon-quad, but removes the uses of TARGET_NEON_VECTORIZE_QUAD. Following Julian's suggestion I added a param preferred-vector-size for testing and debugging purposes. I tested a slightl

Re: RFC: PATCH: Remove 26 element limit in vector

we hit this limit trying to write the explicit semantics for a vec_interleave_evenv32qi. ;;(define_insn "vec_interleave_evenv32qi" ;; [(set (match_operand:V32QI 0 "register_operand" "=r") ;;(vec_select:V32QI ;; (vec_concat:V64QI ;;(match_operand:V32QI 1 "register_operand" "0")

Re: Remove TARGET_VERSION

"Joseph S. Myers" writes: > Bootstrapped with no regressions on x86_64-unknown-linux-gnu. OK to > commit? Ok for mips-tdump.c, mips-tfile.c and config/mips/iris6.h. I had asked about this last year and had it on my agenda since the information included, even if visible to the user, was often o

Re: [patch, ARM] Make 128 bits the default vector size for NEON

On Thu, 31 Mar 2011, Ira Rosen wrote: > +Illegal values are ignored. The default is 128. See the GNU Coding Standards : Please do not use the term "illegal" to refer to erroneous input to a computer program. Please use "invali

Re: Cleaning up expand optabs code

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 171446) >>> >>> ../../../../../gcc.gn

More static checker fixes

-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Whee. The static checkers found another real bug. The tree-ssa-forwprop.c fix eliminates a potential NULL pointer dereference due to a typo in a condition. Basically if DEF_STMT is null, it would be passed to can_propagate_from which would then der

Re: [H8300] Hookize GO_IF_MODE_DEPENDENT_ADDRESS

-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 03/31/11 00:22, Anatoly Sokolov wrote: > Hello. > > This patch removes obsolete GO_IF_MODE_DEPENDENT_ADDRESS macros from H8300 > back end in the GCC and introduces equivalent TARGET_MODE_DEPENDENT_ADDRESS_P > target hook. > > > Regression t

Re: [patch, ARM] Make 128 bits the default vector size for NEON

On 31 March 2011 14:28, Joseph S. Myers wrote: > On Thu, 31 Mar 2011, Ira Rosen wrote: > >> +Illegal values are ignored.  The default is 128. > > See the GNU Coding Standards > : > >   Please do not use the term "illegal" to refer to er

Re: [cxx-mem-model] disallow load data races (1 of some)

-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 03/24/11 17:57, Andrew MacLeod wrote: > My fault for not being specific about it... I tend to just use data race > as a catch all for all these types of things when talking about them > with Aldy. > > the testcase should have for (x=0; x< limi

Re: [patch, ARM] Make 128 bits the default vector size for NEON

On Thu, Mar 31, 2011 at 01:39:05PM +0200, Ira Rosen wrote: > This patch changes NEON's default vector size from 64 to 128 bits. No comments about the patch itself, but this change should be noted in changes.html. -Nathan

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

-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 03/23/11 08:45, 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. OK. je

Re: [PATCH 2/6] Unique return rtx

-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 03/23/11 08:47, Bernd Schmidt wrote: > We'll start putting "return" into JUMP_LABELS in a subsequent patch, so > I've decided to make it unique as a small cleanup. > > There's already another macro called "return_rtx", so the new one goes > by the

Re: [patch, ARM] Make 128 bits the default vector size for NEON

On 31 March 2011 15:11, Nathan Froyd wrote: > On Thu, Mar 31, 2011 at 01:39:05PM +0200, Ira Rosen wrote: >> This patch changes NEON's default vector size from 64 to 128 bits. > > No comments about the patch itself, but this change should be noted in > changes.html. I'll do that. Thanks, Ira > >

Re: [PATCH 5/6] Generate more shrink-wrapping opportunities

-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 03/23/11 08:55, Bernd Schmidt wrote: > The first basic block contains insns to move incoming argument registers > to pseudos. When these pseudos live across calls, they get allocated to > call-saved registers. This in turns disables shrink-wrapping,

Re: More static checker fixes

On Thu, Mar 31, 2011 at 2:55 PM, Jeff Law wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > > Whee.  The static checkers found another real bug.  The > tree-ssa-forwprop.c fix eliminates a potential NULL pointer dereference > due to a typo in a condition.  Basically if DEF_STMT is null,

Re: [cxx-mem-model] disallow load data races (1 of some)

On Thu, Mar 31, 2011 at 3:08 PM, Jeff Law wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > On 03/24/11 17:57, Andrew MacLeod wrote: >> My fault for not being specific about it... I tend to just use data race >> as a catch all for all these types of things when talking about them >> wit

Re: [cxx-mem-model] disallow load data races (1 of some)

-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 03/31/11 07:26, Richard Guenther wrote: > On Thu, Mar 31, 2011 at 3:08 PM, Jeff Law wrote: > On 03/24/11 17:57, Andrew MacLeod wrote: My fault for not being specific about it... I tend to just use data race as a catch all for all these ty

Re: [PATCH 5/6] Generate more shrink-wrapping opportunities

On 03/31/2011 03:23 PM, Jeff Law wrote: > On 03/23/11 08:55, Bernd Schmidt wrote: >> The first basic block contains insns to move incoming argument registers >> to pseudos. When these pseudos live across calls, they get allocated to >> call-saved registers. This in turns disables shrink-wrapping, s

Add an array_mode_supported_p target hook

This patch adds an array_mode_supported_p hook, which says whether MAX_FIXED_MODE_SIZE should be ignored for a given type of array. It follows on from the discussion here: http://gcc.gnu.org/ml/gcc/2011-03/msg00342.html The intended use of the hook is to allow small arrays of vectors to have

Re: Scheduler cleanups, 3/N

-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 03/24/11 07:11, Bernd Schmidt wrote: > This caches dependency costs. The target adjust_cost macro can be quite > expensive. This may not make much of a difference now, but in the final > backtracking scheduler patches (which I'm omitting for the mom

Re: Add an array_mode_supported_p target hook

On Thu, Mar 31, 2011 at 3:32 PM, Richard Sandiford wrote: > This patch adds an array_mode_supported_p hook, which says whether > MAX_FIXED_MODE_SIZE should be ignored for a given type of array. > It follows on from the discussion here: > >    http://gcc.gnu.org/ml/gcc/2011-03/msg00342.html > > The

Re: PING^2 [PATCH] Support for AMD64 targets running GNU/kFreeBSD

I advise CC:ing the relevant target maintainers on such patch submissions. Since there is no *-kfreebsd-gnu maintainer (you might wish to volunteer to be such), and no *-linux* maintainer for the linux*.h changes, this means the people listed as maintainers of the i386 port. -- Joseph S. Myer

[Patch] bfin: protect FDPIC reg on function entry

The attached patch adds the FDPIC reg to TARGET_EXTRA_LIVE_ON_ENTRY for Blackfin fdpic. 2011-03-31 Stuart Henderson From Jie Zhang *config/bfin/bfin.c (bfin_extra_live_on_entry): New. (TARGET_EXTRA_LIVE_ON_ENTRY): Define. Thanks, Stu upstream.patch Description: upstream.patc

Re: [patch] add hp sfc ops DWARF-2 subcodes

OK. Jason

RE: [Patch] bfin: move loop invariants out of loop

New version of previous patch. Apologies. Stu -Original Message- From: Henderson, Stuart Sent: 31 March 2011 12:20 To: gcc-patches@gcc.gnu.org Subject: [Patch] bfin: move loop invariants out of loop The attached patch attempts to move loop invariants out of loops for Blackfin. 2011-03

Re: [patch, ARM] Fix PR48250, adjust DImode reload address legitimizing

On 2011/3/31 06:14 PM, Richard Earnshaw wrote: > > On Thu, 2011-03-31 at 11:33 +0800, Chung-Lin Tang wrote: >> On 2011/3/30 05:28 PM, Richard Earnshaw wrote: >>> >>> On Wed, 2011-03-30 at 15:35 +0800, Chung-Lin Tang wrote: On 2011/3/30 上午 12:23, Richard Earnshaw wrote: > > On Tue, 201

Re: [Patch] bfin: protect FDPIC reg on function entry

On 03/31/2011 03:56 PM, Henderson, Stuart wrote: > The attached patch adds the FDPIC reg to TARGET_EXTRA_LIVE_ON_ENTRY for > Blackfin fdpic. > > > 2011-03-31 Stuart Henderson > > From Jie Zhang > *config/bfin/bfin.c (bfin_extra_live_on_entry): New. > (TARGET_EXTRA_LIVE_ON_ENTRY):

Re: [PATCH] PR debug/47471 (set prologue_end in .debug_line)

Richard, Thank you for the crystal clear explanation. Now I understand why we were not using the end_prologue debug hook before :-). >From what you say and from what Jan said, I think we could just keep the part (of my earlier patch) that avoids emitting two consecutive redundant .loc directives

Re: Add an array_mode_supported_p target hook

Richard Guenther writes: > On Thu, Mar 31, 2011 at 3:32 PM, Richard Sandiford > wrote: >> This patch adds an array_mode_supported_p hook, which says whether >> MAX_FIXED_MODE_SIZE should be ignored for a given type of array. >> It follows on from the discussion here: >> >>    http://gcc.gnu.org/m

Re: [cxx-mem-model] bitfield tests

-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 03/30/11 08:19, 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

[patch, ARM] Fix PR48325, support POST_INC/PRE_DEC for NEON struct modes

This PR doesn't exactly trigger currently on trunk; a REG_DEAD note that occurs in trunk, but not in the 4.5-based compilers which this bug was reported for, currently blocks auto-inc-dec from doing its job, and just happens to avoid this ICE. The problem is vldmia/db, which do exist, are currentl

Re: PING: PATCH: PR target/46770: Use .init_array/.fini_array sections

On Mon, Mar 21, 2011 at 11:40 AM, H.J. Lu wrote: > On Mon, Mar 14, 2011 at 12:28 PM, H.J. Lu wrote: >> On Thu, Jan 27, 2011 at 2:40 AM, Richard Guenther >> wrote: >>> On Thu, Jan 27, 2011 at 12:12 AM, H.J. Lu wrote: On Tue, Dec 14, 2010 at 05:20:48PM -0800, H.J. Lu wrote: > This patch

Re: Scheduler cleanups, 1/N

-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 03/24/11 07:07, 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 instruction

[testsuite] Only XFAIL gfortran.dg/g77/cabs.f execution at -O0 on IRIX 6.5

I noticed that on IRIX 6.5, gfortran.dg/g77/cabs.f XPASSes at all optimization levels except -O0. The following patch reflects this. Tested with the appropriate runtest invocations, installed on mainline and 4.6 branch. The 4.5 branch, which still support the O32 ABI, needs the slightly differen

Re: [Patch] bfin: move loop invariants out of loop

On 03/31/2011 04:16 PM, Henderson, Stuart wrote: > The attached patch attempts to move loop invariants out of loops for Blackfin. > > > 2011-03-31 Stuart Henderson > >From Bernd Schmidt >* config/bfin/bfin.c (bfin_gen_bundles): Don't try to bundle a jump. >(bfin_optimize_loops_1,

Re: [cxx-mem-model] bitfield tests

On Thu, Mar 31, 2011 at 4:47 PM, Jeff Law wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > On 03/30/11 08:19, 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-mod

[patch] Fix failure of ACATS cc3601a

Hi, the ACATS test cc3601a started to fail recently at -O2 after the various tweaks to the optimizer. This is an assertion failure in the gimplifier invoked from PRE because of a dangling PLACEHOLDER_EXPR in operand #2 of an ARRAY_REF. CCP2 turns the ARRAY_REF: D.2774_394 = MEM[(boolean[(si

Re: [patch] Fix failure of ACATS cc3601a

On Thu, Mar 31, 2011 at 5:25 PM, Eric Botcazou wrote: > Hi, > > the ACATS test cc3601a started to fail recently at -O2 after the various > tweaks > to the optimizer.  This is an assertion failure in the gimplifier invoked from > PRE because of a dangling PLACEHOLDER_EXPR in operand #2 of an ARRAY

Re: [PATCH] PR debug/47471 (set prologue_end in .debug_line)

On 03/31/2011 07:35 AM, Dodji Seketeli wrote: > redundant .loc directives. It seems to me that in the case of direct > output (i.e when we the underlying assembler doesn't support the .loc > directive) we already avoid the duplication. And that avoidance fixes > the immediate issue GDB is facing,

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

While debugging why several libgo tests on Solaris 2/SPARC were hanging in select (cf. PR go/48242, go/48243), I found that fd_set is typedef struct fd_set { longfds_bits[__howmany(FD_SETSIZE, FD_NFDBITS)]; } fd_set; The current implementation of the FD* funcs in sysfile_posix.go assu

Re: [Patch] Bfin: Ensure rotrsi and rotlsi don't accept non-const INTVALS

> (rotate:SI (match_operand:SI 1 "register_operand" "") > -(match_operand:SI 2 "immediate_operand" "")))] > +(match_operand:SI 2 "const_int_operand" "")))] >"" > { > - if (INTVAL (operands[2]) != 16) > + if (GET_CODE (operands[2]) != CONST_INT || INTVAL

Re: [libgo] Improve Solaris 2/SPARC support

Ian Lance Taylor writes: > Thanks, I just committed this patch to mainline. > > I'm working on the other ones. Great, thanks. With [libgo] Support Solaris 8/9 http://gcc.gnu.org/ml/gcc-patches/2011-03/msg01641.html and now [libgo] Account for 32-bit fds_bits on Solari

Re: [libgo] Replace wait4 by waitpid (PR go/47515)

Ian Lance Taylor writes: > Thanks for the patch, but I wasn't entirely happy with the approach > because it removes the os.Waitmsg.Rusage field. That field is > inherently system dependent but I would rather not get rid of it. I > wrote this patch instead, which I hope will also solve the probl

Re: Move reg_equiv* arrays into a single VEC structure

-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 03/30/11 10:51, Richard Sandiford wrote: > Nice cleanup thanks. Just noticed a couple of things: > > Jeff Law writes: >> *** struct reload >> *** 100,106 >> int inc; >> /* A reg for which reload_in is the equivalent. >>

Re: Random cleanups [2/4]: canonicalize ctor values

On Thu, 31 Mar 2011, Richard Guenther wrote: > > canonicalize_constructor_val may be doing useful things on ADDR_EXPR > > too, but you don't call it in that case because you only added your > > code in the default case.  You only reach it when the ADDR_EXPR is > > wrapped in a cast. > > > > Any

Re: [PATCH, ARM] PR47855 Compute attr "length" for some thumb2 insns

Hi Carrot, How about adding an alternative only enabled for T2 that uses the `l' constraint and inventing new constraints for some of the constant values that are valid for 16 bit instructions since the `I' and `L' constraints have different meanings depending on whether TARGET_32BIT

Re: Random cleanups [2/4]: canonicalize ctor values

On Thu, Mar 31, 2011 at 5:57 PM, Michael Matz wrote: > On Thu, 31 Mar 2011, Richard Guenther wrote: > >> > canonicalize_constructor_val may be doing useful things on ADDR_EXPR >> > too, but you don't call it in that case because you only added your >> > code in the default case.  You only reach it

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

Rainer Orth writes: > While debugging why several libgo tests on Solaris 2/SPARC were hanging > in select (cf. PR go/48242, go/48243), I found that fd_set is > > typedef struct fd_set { > longfds_bits[__howmany(FD_SETSIZE, FD_NFDBITS)]; > } fd_set; > > The current implementation of th

Re: [Patch] Bfin: Ensure rotrsi and rotlsi don't accept non-const INTVALS

On 03/31/2011 05:42 PM, Richard Henderson wrote: >> (rotate:SI (match_operand:SI 1 "register_operand" "") >> - (match_operand:SI 2 "immediate_operand" "")))] >> + (match_operand:SI 2 "const_int_operand" "")))] >>"" >> { >> - if (INTVAL (operands[2]) != 16) >>

Re: [libgo] Improve Solaris 2/SPARC support

Rainer Orth writes: > I'm especially suffering from hangs on Solaris 8/x86 and Solaris 8 and > 9/SPARC (not yet filed or investigated), which hangs the whole build (PR > go/48242). If I get around to it, I'll probably replace gotest by a > dg-based testsuite. Argh, no, I am trying to fight agai

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

Ian Lance Taylor writes: > What an annoying problem. Sorry about that. But why don't we just > change to byte? If this causes the right bits to be set for big and little-endian hosts, certainly fine with me. I haven't tried yet, but the less different code paths, the better. Rainer

libgo patch committed: Generate DejaGNU style .sum and .log files

This patch to libgo makes the testsuite generate a libgo.sum and libgo.log file which are much like the ones that DejaGNU would generate. My hope is that this will cause the various scripts we have that parse DejaGNU files do something useful with the libgo test results. Bootstrapped and ran Go tes

Re: [libgo] Improve Solaris 2/SPARC support

Ian Lance Taylor writes: > Rainer Orth writes: > >> I'm especially suffering from hangs on Solaris 8/x86 and Solaris 8 and >> 9/SPARC (not yet filed or investigated), which hangs the whole build (PR >> go/48242). If I get around to it, I'll probably replace gotest by a >> dg-based testsuite. >

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

Following the difficulties with the setmem pattern, this patch series makes the number of generator arguments (as opposed to rtx insn operands) available in insn_data. The first patch adds a new "gather statistics" function to gensupport, and uses it in genemit.c Bootstrapped & regression-tested

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

This second patch makes genoutput.c use the new get_pattern_stats function, and adds the extra field to insn_data. Bootstrapped & regression-tested on x86_64-linux-gnu. OK to install? Richard gcc/ * recog.h (insn_data_d): Add n_generator_args. * genoutput.c (data): Likewise.

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

This patch makes use of the new n_generator_args field. Bboostrapped & regressions-tested on x86_64-linux-gnu. Also tested by building cc1 for arm-linux-gnueabi, mips-elf, spu-elf. Richard gcc/ * expr.c (emit_block_move_via_movmem): Use n_generator_args instead of n_operands.

Re: [libgo] Improve Solaris 2/SPARC support

Hello, * Rainer Orth wrote on Thu, Mar 31, 2011 at 06:12:31PM CEST: > Ian Lance Taylor writes: > > Argh, no, I am trying to fight against that as long as possible. We > > should be moving away from DejaGNU, not toward it. > > Do you have a decent alternative? I've no idea what happened to QMTe

Re: [libgo] Improve Solaris 2/SPARC support

Rainer Orth writes: > Ian Lance Taylor writes: > >> Rainer Orth writes: >> >>> I'm especially suffering from hangs on Solaris 8/x86 and Solaris 8 and >>> 9/SPARC (not yet filed or investigated), which hangs the whole build (PR >>> go/48242). If I get around to it, I'll probably replace gotest

go patch committed: Taking slice of array moves it to heap

This patch to the Go frontend fixes a bug: if you take a slice of an array, you are in effect taking the address of the array, which in Go requires moving the array to the heap. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r b9df7566fd57 go/exp

Go patch committed: Fix typo in test case name

This patch to the libgo Makefile fixes a typo in a test case name. Ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r 0158b3ae4e51 libgo/Makefile.am --- a/libgo/Makefile.am Thu Mar 31 09:45:16 2011 -0700 +++ b/libgo/Makefile.am Thu Mar 31 09:47:33 2011 -0700 @@ -20

Re: [libgo] Improve Solaris 2/SPARC support

On Thu, 31 Mar 2011, Ralf Wildenhues wrote: > Hello, > > * Rainer Orth wrote on Thu, Mar 31, 2011 at 06:12:31PM CEST: > > Ian Lance Taylor writes: > > > Argh, no, I am trying to fight against that as long as possible. We > > > should be moving away from DejaGNU, not toward it. > > > > Do you h

Re: [Patch] Bfin: Ensure rotrsi and rotlsi don't accept non-const INTVALS

On 03/31/2011 09:01 AM, Bernd Schmidt wrote: > I have a dim memory of the problem being that something didn't check the > predicate. Sure enough, with the patch below applied to a 4.3 tree, I get > > /local/src/egcs/gcc-4_3-branch/gcc/testsuite/gcc.c-torture/execute/20020226-1.c:43: > internal com

Re: [libgo] Improve Solaris 2/SPARC support

On Thu, 31 Mar 2011, Rainer Orth wrote: > Ian Lance Taylor writes: > > > Rainer Orth writes: > > > >> I'm especially suffering from hangs on Solaris 8/x86 and Solaris 8 and > >> 9/SPARC (not yet filed or investigated), which hangs the whole build (PR > >> go/48242). If I get around to it, I'll

Re: [C++0x] Range-based for statements and ADL

On 03/28/2011 08:28 PM, Rodrigo Rivas wrote: A few comments: 1. I'm not sure about what should happen if the begin/end found in class scope are not ordinary functions. Whatever range.begin() would mean if written explicitly. My guess is that if it is a function (static or non-static) it is ca

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

On 03/31/2011 06:22 PM, Richard Sandiford wrote: > Bootstrapped & regression-tested on x86_64-linux-gnu. OK to install? Ok all three. Bernd

Re: [patch, ARM] Fix PR48250, adjust DImode reload address legitimizing

On Thu, 2011-03-31 at 22:18 +0800, Chung-Lin Tang wrote: > On 2011/3/31 06:14 PM, Richard Earnshaw wrote: > > > > On Thu, 2011-03-31 at 11:33 +0800, Chung-Lin Tang wrote: > >> On 2011/3/30 05:28 PM, Richard Earnshaw wrote: > >>> > >>> On Wed, 2011-03-30 at 15:35 +0800, Chung-Lin Tang wrote: > >>>

[PATCH, PR43920] Improve code-size optimizations

This patch set fixes PR 43920 - 'Choosing conditional execution over conditional branches for code size in some cases'. The patch set was tested on ARM and x86_64. The codesize changes were benchmarked for ARM Thumb-2. For SPEC2000 and EEMBC CoreMark, a reduction of 1.1% in the geomean of the benc

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

arm-size-branch_cost.patch reduces BRANCH_COST for thumb2 -Os to 1. The lower branch cost makes expand choose branches to expand code like '(a == b || c == d)'. The impact of arm-size-branch_cost.patch on the example from the bug report for ARM Thumb-2 -Os is a size reduction of 15%, from 68 to 58

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

A testcase for the code in arm-size-branch_cost.patch. Thanks, - Tom Index: gcc/testsuite/gcc.target/arm/pr43920-1.c === --- gcc/testsuite/gcc.target/arm/pr43920-1.c (revision 0) +++ gcc/testsuite/gcc.target/arm/pr43920-1.c (revision

[PATCH, PR43920, 3/9] Cleanup.

Cleans up some code. Thanks, - Tom 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, fallthru; bool changed; unsigned max, ix, ix2; - basic_block ev, ev2; /* Nothing to do if there is n

[PATCH, PR43920, 4-9/9] Cross-jumping.

The impact of patches 4-9 of the patch set on the example from the bug report for ARM Thumb-2 -Os is a further size reduction of 7%, from 58 to 54 bytes. This size reduction is illustrated in this diff of the assembly (left, without patches, size 58. right, with patches, size 54): ... push{r3,

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

Currently uses and clobbers are counted as insns in crossjumping, which can cause undesirable crossjumping. The patch fixes this. Thanks, - Tom Index: gcc/cfgcleanup.c === --- gcc/cfgcleanup.c (revision 170556) +++ gcc/cfgcleanup.c (

  1   2   >