[patch, mips] Modify mips-mti-linux-gnu build

2012-10-30 Thread Steve Ellcey
This patch makes a few changes to the work I have been doing on the mips-mti-linux-gnu target that I recently added. It adds an -mabi=64 build to MULTILIBS and changes the order of MULTILIB_OPTIONS to be consistent with the mips-mti-elf target. It also makes the N32 ABI the default when compiling

Re: [patch] Apply conditional down cast to cgraph.h et.al.

2012-10-30 Thread Lawrence Crowl
On 10/30/12, Diego Novillo wrote: > On 2012-10-29 15:01 , Lawrence Crowl wrote: >> On 10/27/12, Marc Glisse wrote: >>> On Fri, 26 Oct 2012, Lawrence Crowl wrote: 2012-10-26 Lawrence Crowl >> >>> missing '>' >> >> Fixed. >> * is-a.h: New. (is_a (U*)): New. Test for is-a re

[patch] Handle NULL in slim RTL dumps of patterns

2012-10-30 Thread Steven Bosscher
Hello, NULL patterns happen in some REG_NOTEs. Will commit as obvious in a bit... Ciao! Steven * sched-vis.c (print_pattern): Handle NULL patterns. Index: gcc/sched-vis.c === --- gcc/sched-vis.c (revision 192992) +++

Re: GCC 4.8.0 Status Report (2012-10-29), Stage 1 to end soon

2012-10-30 Thread Kenneth Zadeck
jakub, i am hoping to get the rest of my wide integer conversion posted by nov 5. I am under some adverse conditions here: hurricane sandy hit her pretty badly. my house is hooked up to a small generator, and no one has any power for miles around. So far richi has promised to review them.

Re: [patch, mips] Modify mips-mti-linux-gnu build

2012-10-30 Thread Richard Sandiford
"Steve Ellcey " writes: > 2012-10-30 Steve Ellcey > > * config/mips/mti-linux.h (SYSROOT_SUFFIX_SPEC): Change order > and add mabi=64. > (DRIVER_SELF_SPECS): Make -n32 the default on mips64* archs. > * config/mips/t-mti-linux (MULTILIB_OPTIONS): Change order. > (MU

Re: [patch] Apply conditional down cast to cgraph.h et.al.

2012-10-30 Thread Diego Novillo
On Tue, Oct 30, 2012 at 4:53 PM, Lawrence Crowl wrote: > On 10/30/12, Diego Novillo wrote: >> >> So, to use these three functions, the user must define this single >> 'is_a_helper' routine? Nothing else? > > You need to distinguish which kind user. Someone just wanting > to convert does not nee

committed: Fix building of targets without SECONDARY_MEMORY_NEEDED

2012-10-30 Thread Joern Rennecke
Build-tested for mmix-knuth-mmixware. Commited as obvious. 2012-10-30 Joern Rennecke * lra-constraints.c (check_secondary_memory_needed_p): Add ATTRIBUTE_UNUSED to parameters. Index: lra-constraints.c === --- lra-

Re: [patch] Apply conditional down cast to cgraph.h et.al.

2012-10-30 Thread Lawrence Crowl
On 10/30/12, Diego Novillo wrote: > On Tue, Oct 30, 2012 at 4:53 PM, Lawrence Crowl wrote: >> On 10/30/12, Diego Novillo wrote: >>> >>> So, to use these three functions, the user must define this single >>> 'is_a_helper' routine? Nothing else? >> >> You need to distinguish which kind user. Som

Re: [patch] Apply conditional down cast to cgraph.h et.al.

2012-10-30 Thread Xinliang David Li
Good cleanup! Simple and consistent interfaces are usually self-documenting, and this one is a good step forward. thanks, David On Tue, Oct 30, 2012 at 2:09 PM, Lawrence Crowl wrote: > On 10/30/12, Diego Novillo wrote: >> On Tue, Oct 30, 2012 at 4:53 PM, Lawrence Crowl wrote: >>> On 10/30/12,

Re: GCC 4.8.0 Status Report (2012-10-29), Stage 1 to end soon

2012-10-30 Thread Lawrence Crowl
On 10/30/12, Diego Novillo wrote: > On Mon, Oct 29, 2012 at 1:56 PM, Jakub Jelinek wrote: >> Status >> == >> >> I'd like to close the stage 1 phase of GCC 4.8 development >> on Monday, November 5th. If you have still patches for new features >> you'd >> like to see in GCC 4.8, please post th

Re: [PATCH] Fix PR 53743 and other -freorder-blocks-and-partition failures (issue6823047)

2012-10-30 Thread Steven Bosscher
Hello Teresa, Could you try this patch for me also? It moves bbpart outside the part of the passes pipeline that works in cfglayout mode.

Re: [PATCH][RFC] Sanity checking for -freorder-blocks-and-partition failures

2012-10-30 Thread Teresa Johnson
On Tue, Oct 30, 2012 at 12:33 PM, Xinliang David Li wrote: > On Tue, Oct 30, 2012 at 12:25 PM, Steven Bosscher > wrote: >> On Tue, Oct 30, 2012 at 5:59 PM, Teresa Johnson wrote: >>> I will try testing your patch on top of mine with our fdo benchmarks. >> >> Thanks. But you should expect a lot of

Re: [PATCH] Fix PR 53743 and other -freorder-blocks-and-partition failures (issue6823047)

2012-10-30 Thread Steven Bosscher
On Tue, Oct 30, 2012 at 10:28 PM, Steven Bosscher wrote: > Hello Teresa, > > Could you try this patch for me also? It moves bbpart outside the part > of the passes pipeline that works in cfglayout mode. where's the "unsend" button if you need it... So, to complete the mail... Could you try this

Re: [PATCH][RFC] Sanity checking for -freorder-blocks-and-partition failures

2012-10-30 Thread Xinliang David Li
There are not many -- see gcc/testsuite/gcc.dg/tree-prof directory. Under that, 5 cases explicitly set the option. David On Tue, Oct 30, 2012 at 2:31 PM, Teresa Johnson wrote: > On Tue, Oct 30, 2012 at 12:33 PM, Xinliang David Li > wrote: >> On Tue, Oct 30, 2012 at 12:25 PM, Steven Bosscher

Re: [PATCH][RFC] Sanity checking for -freorder-blocks-and-partition failures

2012-10-30 Thread Steven Bosscher
On Tue, Oct 30, 2012 at 10:31 PM, Teresa Johnson wrote: > In an attempt to provoke the failures I was seeing on a smaller test > case, I tried enabling -freorder-blocks-and-partition by default in my > common.opt file and running through the regression test suite (without > my fixes). But I didn't

Re: [PATCH] Fix PR 53743 and other -freorder-blocks-and-partition failures (issue6823047)

2012-10-30 Thread Teresa Johnson
On Tue, Oct 30, 2012 at 2:33 PM, Steven Bosscher wrote: > On Tue, Oct 30, 2012 at 10:28 PM, Steven Bosscher wrote: >> Hello Teresa, >> >> Could you try this patch for me also? It moves bbpart outside the part >> of the passes pipeline that works in cfglayout mode. > > where's the "unsend" button i

Re: GCC 4.8.0 Status Report (2012-10-29), Stage 1 to end soon

2012-10-30 Thread Sriraman Tallam
Hi Jakub, My function multiversioning patch is being reviewed and I hope to get this in by Nov. 5. Thanks, -Sri. On Mon, Oct 29, 2012 at 10:56 AM, Jakub Jelinek wrote: > Status > == > > I'd like to close the stage 1 phase of GCC 4.8 development > on Monday, November 5th. If you have st

Re: [1/8] Remove redundant BLKmode test

2012-10-30 Thread Eric Botcazou
> gcc/ > * expmed.c (store_bit_field_1): Remove test for BLKmode values. This looks fine to me. -- Eric Botcazou

Re: [2/8] Remove redundant MAX_MACHINE_MODE tests

2012-10-30 Thread Eric Botcazou
> gcc/ > * expmed.c (store_bit_field_1): Use OP_MODE to check whether an > insv pattern is available. Remove redundant checks for OP_MODE > being MAX_MACHINE_MODE. > (extract_bit_field_1): Remove redundant checks for EXT_MODE being > MAX_MACHINE_MODE. We have had a s

Re: [3/8] Split out insv and ext(z)v code

2012-10-30 Thread Eric Botcazou
> gcc/ > * expmed.c (store_bit_field_using_insv): New function, > split out from... > (store_bit_field_1): ...here. > (extract_bit_field_using_extv): New function, split out from... > (extract_bit_field_1): ...here. No objections to this cleanup. -- Eric Botcazou

Re: [8/8] Rename some powerpc patterns

2012-10-30 Thread David Edelsohn
On Tue, Oct 30, 2012 at 3:44 PM, Richard Sandiford wrote: > The optabs patches that I'm working on treat patterns with names like > "insvsi" and "insvdi" as part of the public target interface. > Those names clash with some existing rs6000.md patterns, so this patch adds > some _internal suffixes

Re: [4/8] Separate reg and mem uses of insv and ext(z)v

2012-10-30 Thread Eric Botcazou
> gcc/ > * expmed.c (store_bit_field_1): Move generation of MEM insvs > to the MEM_P block. > (extract_bit_field_1): Likewise extvs and extzvs. I guess it's in keeping with your earlier changes in these functions. -- Eric Botcazou

Re: [5/8] Add narrow_bit_field_mem

2012-10-30 Thread Eric Botcazou
> This patch splits out a fairly common operation: that of narrowing a MEM > to a particular mode and adjusting the bit number accordingly. > > I've kept with "bit_field" rather than "bitfield" for consistency with > the callers, although we do have "bitfield" in "adjust_bitfield_address". My bad

Re: [6/8] Simplify make_extraction

2012-10-30 Thread Eric Botcazou
> gcc/ > * combine.c (make_extraction): Remove dead wanted_inner_mode- > and pos_rtx-related code. OK, thanks. -- Eric Botcazou

Re: [7/8] BITS_BIG_ENDIAN vs. (zero_extract (const_int ...) ...)

2012-10-30 Thread Eric Botcazou
> gcc/ > * combine.c (simplify_comparison): If BITS_BIG_ENDIAN, always assume > that zero_extracts of const_ints are doing word-sized extractions. This looks plausible. -- Eric Botcazou

Re: Patch: add @direntry for gcov

2012-10-30 Thread Joseph S. Myers
On Tue, 30 Oct 2012, Tom Tromey wrote: > This patch adds a @direntry for gcov. > > I noticed that it was missing today, when I tried to find the gcov > manual from the info "dir" node. Then I found out that I had filed PR > 50899 for this ages ago. > > Ok? OK. -- Joseph S. Myers jos...@codes

[Ada] Fix Makefile dependencies of C files

2012-10-30 Thread Eric Botcazou
This will hopefully fix the build failure reported by Diego. Apart from adding missing dependencies, this also removes redundant command lines. Tested on x86_64-suse-linux, applied on the mainline and 4.7 branch. 2012-10-30 Eric Botcazou * gcc-interface/Make-lang.in: Fix and clean

[PATCH] Update source location for PRE inserted stmt

2012-10-30 Thread Dehao Chen
Hi, This patch aims to improve debugging of optimized code. It ensures that PRE inserted statements have the same source location as the statement at the insertion point, instead of UNKNOWN_LOCATION. Bootstrapped on x86_64, and passed gcc regression tests and gdb regression tests. Is it okay for

Re: dead_debug_local_finish may be used uninit warning

2012-10-30 Thread Alexandre Oliva
On Oct 30, 2012, Roland McGrath wrote: > The only initialization is conditional, but at least the first and last > uses are unconditional. (I don't understand the conditions of the second > use enough to know whether it's always kosher.) Lawrence let me know about that warning earlier today. I

Re: [PR54693] loss of debug info in jump threading and loop ivopts

2012-10-30 Thread Alexandre Oliva
On Oct 30, 2012, Jakub Jelinek wrote: > On Tue, Oct 30, 2012 at 03:51:19PM +0100, Richard Biener wrote: >> + if (!comp) >> + BREAK_FROM_IMM_USE_STMT (imm_iter); >> how does comp magically become NULL_TREE here? > Looks like pasto to me, from the first loop. Mo

Re: GCC 4.8.0 Status Report (2012-10-29), Stage 1 to end soon

2012-10-30 Thread Sharad Singhai
Hi Jakub, My -fopt-info pass filtering patch (http://gcc.gnu.org/ml/gcc-patches/2012-10/msg02704.html) is being reviewed and I hope to get this in by Nov. 5 for inclusion in gcc 4.8.0. Thanks, Sharad On Mon, Oct 29, 2012 at 10:56 AM, Jakub Jelinek wrote: > Status > == > > I'd like to close

Remove left-overs of RTX_UNCHANGING_P

2012-10-30 Thread Eric Botcazou
In the form of a couple of references in comments from cse.c. Tested on x86_64-suse-linux, applied on the mainline and 4.7 branch. 2012-10-30 Eric Botcazou * cse.c (hash_rtx_cb): Replace RTX_UNCHANGING_P with MEM_READONLY_P in head comment. (hash_rtx): Likewise. --

[patch][RFC] Filename based shared library versioning on AIX

2012-10-30 Thread Michael Haubenwallner
Hello! I would like to introduce filename-based shared library versioning (known as the "soname" in ELF world) for AIX, activated by the '--enable-aix-soname' configure flag. Attached patch is for libgcc_s only, patches for libtool have been posted (although without any reply so far) to: http://l

Re: [PATCH] Update source location for PRE inserted stmt

2012-10-30 Thread Steven Bosscher
On Wed, Oct 31, 2012 at 12:00 AM, Dehao Chen wrote: > This patch aims to improve debugging of optimized code. It ensures > that PRE inserted statements have the same source location as the > statement at the insertion point, instead of UNKNOWN_LOCATION. Wrong patch attached. However, is it really

[C++ Patch] PR 54583

2012-10-30 Thread Paolo Carlini
Hi, this diagnostic issue is about spurious "value computed is not used" warnings with VLAs, emitted by convert_to_void (on the TYPE_SIZE, a MULT_EXPR): the warning is already gated by !TREE_NO_WARNING (expr) thus setting it at the end of build_cplus_array_type seems a rather straightforward

Re: [PATCH] Update source location for PRE inserted stmt

2012-10-30 Thread Dehao Chen
Sorry, new patch attached... On Tue, Oct 30, 2012 at 4:38 PM, Steven Bosscher wrote: > On Wed, Oct 31, 2012 at 12:00 AM, Dehao Chen wrote: >> This patch aims to improve debugging of optimized code. It ensures >> that PRE inserted statements have the same source location as the >> statement at the

Re: [patch][RFC] Filename based shared library versioning on AIX

2012-10-30 Thread Joseph S. Myers
On Wed, 31 Oct 2012, Michael Haubenwallner wrote: > I would like to introduce filename-based shared library versioning (known as > the "soname" in ELF world) for AIX, activated by the '--enable-aix-soname' > configure flag. Patches adding new configure options should include the changes to insta

Re: [PATCH] Update source location for PRE inserted stmt

2012-10-30 Thread Xinliang David Li
It will make the location info for the newly synthesized stmt more deterministic, I think. David On Tue, Oct 30, 2012 at 4:38 PM, Steven Bosscher wrote: > On Wed, Oct 31, 2012 at 12:00 AM, Dehao Chen wrote: >> This patch aims to improve debugging of optimized code. It ensures >> that PRE inserte

Re: [Patch] Potential fix for PR55033

2012-10-30 Thread Alan Modra
On Tue, Oct 30, 2012 at 02:45:40PM +0100, Sebastian Huber wrote: > On 10/26/2012 02:22 PM, Sebastian Huber wrote: > >Hello, > > > >here is a test case for PR55033. > > > > Is there something wrong with this test case? It compiles well with Alan's > patch. It looks OK to me if you replace your "

[google] Add attributes: always_patch_for_instrumentation and never_patch_for_instrumentation (issue6821051)

2012-10-30 Thread Harshit Chopra
Adding function attributes: 'always_patch_for_instrumentation' and 'never_patch_for_instrumentation' to always patch a function or to never patch a function, respectively, when given the option -mpatch-functions-for-instrumentation. Additionally, the attribute always_patch_for_instrumentation d

Re: User directed Function Multiversioning via Function Overloading (issue5752064)

2012-10-30 Thread Sriraman Tallam
On Tue, Oct 30, 2012 at 12:10 PM, Jason Merrill wrote: > On 10/27/2012 09:16 PM, Sriraman Tallam wrote: >> >> + /* See if there's a match. For functions that are >> multi-versioned, >> +all the versions match. */ >> if (same_type_p (target_fn_type, static_fn_type (

Go patch committed: Add -fgo-relative-import-path option

2012-10-30 Thread Ian Lance Taylor
The go tool, used by many people to build Go programs, handles relative import paths by passing a special option to the compiler. The option is a prefix to use for any relative import. The go tool does this so that programs can be built in a temporary directory and can still find packages importe

Re: Is libstdcxx-time still experimental?

2012-10-30 Thread Jonathan Wakely
On 26 August 2012 18:20, Jonathan Wakely wrote: >> I'm working on a patch to enable it by default and to define >> this_thread::sleep_xxx and this_thread::yield unconditionally. > > For anyone interested, here's a work-in-progress patch that enables > this_thread::sleep_for() and this_thread::yield

[v3] Fix memory leak in forward_list

2012-10-30 Thread Jonathan Wakely
This fixes a memory leak in the allocator-extended move constructor I added to _Fwd_list_base two weeks ago. When the supplied allocator is not equal to the one in the rvalue parameter the rvalue's head pointer was zeroed out, without freeing the nodes it pointed to. This new version leaves the rv

[wwwdocs,Java] Remove java/libgcj-platforms.html

2012-10-30 Thread Gerald Pfeifer
When trying to address some issues with this page, I realized it is not linked, and hasn't for probably a decade or more. And the contents is not up-to-date either or more or less covered elsewhere. Sooo, gone it is. Gerald 2012-10-31 Gerald Pfeifer * libgcj-platforms.html: Remove.

Re: [patch] move GIMPLE_TRANSACTION expansion to tmmark pass

2012-10-30 Thread Richard Henderson
On 2012-10-30 05:32, Aldy Hernandez wrote: > + // If we have a ``_transaction_cancel [[outer]]'', there is only > + // one abnormal edge: to the transaction marked OUTER. > + tree arg = gimple_call_arg (stmt, 0); > + if (TREE_CODE (arg) == INTEGER_CST) > + { > + if (TR

Re: [patch,libgcc] m32r*rtems* add crtinit.o and crtfinit.o

2012-10-30 Thread Ralf Corsepius
On 10/30/2012 07:03 PM, Ian Lance Taylor wrote: On Tue, Oct 30, 2012 at 12:10 AM, Ralf Corsepius wrote: I would like to apply the patch below to trunk and gcc-4.7-branch. This patch was originalyl submitted by Joel Sherrill back in May (http://gcc.gnu.org/ml/gcc-patches/2012-05/msg01180.html)

committed: Fix build for targets that have SECONDARY_MEMORY_NEEDED ignoring CLASS* arguments.

2012-10-30 Thread Joern Rennecke
Build tested on powerpc64-unknown-linux-gnu X microblaze-elf . Committed as obvious. 2012-10-31 Joern Rennecke * lra-constraints.c (choose_split_class): Add ATTRIBUTE_UNUSED to hard_reg_class. Index: lra-constraints.c ===

Re: [PATCH][RFC] Sanity checking for -freorder-blocks-and-partition failures

2012-10-30 Thread Teresa Johnson
On Tue, Oct 30, 2012 at 2:31 PM, Teresa Johnson wrote: > On Tue, Oct 30, 2012 at 12:33 PM, Xinliang David Li > wrote: >> On Tue, Oct 30, 2012 at 12:25 PM, Steven Bosscher >> wrote: >>> On Tue, Oct 30, 2012 at 5:59 PM, Teresa Johnson wrote: I will try testing your patch on top of mine with

Re: [PR54693] loss of debug info in jump threading and loop ivopts

2012-10-30 Thread Alexandre Oliva
On Oct 30, 2012, Alexandre Oliva wrote: >> BTW, I'd also think that the first loop should set count = 2 if >> the debug stmt already has a non-trivial expression > Good! Will do. Here's the revised patch. Bootstrap completed, test nearly finished. Ok to install if it passes? Replace PHI node

committed: Fix target/52498

2012-10-30 Thread Joern Rennecke
Tested by building cc1 for powerpc64-unknown-linux-gnu X alpha-dec-vms. Committed as obvious. PR target/52498 * vmsdbgout.c (vmsdbgout_write_source_line): Comment out names of last two parameters. Index: vmsdbgout.c

Re: [PATCH] Fix PR 53743 and other -freorder-blocks-and-partition failures (issue6823047)

2012-10-30 Thread Teresa Johnson
On Tue, Oct 30, 2012 at 2:33 PM, Steven Bosscher wrote: > On Tue, Oct 30, 2012 at 10:28 PM, Steven Bosscher wrote: >> Hello Teresa, >> >> Could you try this patch for me also? It moves bbpart outside the part >> of the passes pipeline that works in cfglayout mode. > > where's the "unsend" button i

RE: [PATCH GCC]Improving register pressure directed hoist

2012-10-30 Thread Bin Cheng
> -Original Message- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On > Behalf Of Bin Cheng > Sent: Monday, October 29, 2012 2:59 PM > To: gcc-patches@gcc.gnu.org > Subject: [PATCH GCC]Improving register pressure directed hoist > > Hi, > This patch improves

<    1   2