Re: [PATCH, i386, PR50766] Fix incorrect mem/reg operands order

2011-10-20 Thread Uros Bizjak
On Thu, Oct 20, 2011 at 6:39 AM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Updated patch is attached. Test fails wihout and passing with the fix. ChangeLog entry: 2011-10-20  Kirill Yukhin  kirill.yuk...@intel.com        PR target/50766        * config/i386/i386.md (bmi_bextr_mode):

[Patch, gcc, testsuite] Adjust optimization levels for some cases.

2011-10-20 Thread Terry Guo
Hello, These four cases check the amount of the desired instructions. At O2 level, some factors like loop unroll will increase the amount of them. This patch is proposing to adjust the optimization level to O1 (the minimal requirement) to avoid such impact. In this way, the cases are more robust.

RE: PING: [PATCH, ARM, iWMMXt][1/5]: ARM code generic change

2011-10-20 Thread Xinyu Qi
Ping http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01100.html * config/arm/arm.c (arm_option_override): Enable use of iWMMXt with VFP. Disable use of iwMMXt and Neon. (arm_expand_binop_builtin): Accept VOIDmode op. * config/arm/arm.md (*arm_movdi): Remove check for

RE: PING: [PATCH, ARM, iWMMXt][3/5]: built in define and expand

2011-10-20 Thread Xinyu Qi
Ping http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01103.html * config/arm/arm.c (enum arm_builtins): Revise built-in fcode. (builtin_description bdesc_2arg): Revise built in declaration. (builtin_description bdesc_1arg): Likewise. (arm_init_iwmmxt_builtins):

RE: PING: [PATCH, ARM, iWMMXt][4/5]: WMMX machine description

2011-10-20 Thread Xinyu Qi
Ping http://gcc.gnu.org/ml/gcc-patches/2011-09/msg00279.html * config/arm/arm.c (arm_output_iwmmxt_shift_immediate): New function. (arm_output_iwmmxt_tinsr): Likewise. * config/arm/arm-protos.h (arm_output_iwmmxt_shift_immediate): Declare.

RE: PING: [PATCH, ARM, iWMMXt][5/5]: pipeline description

2011-10-20 Thread Xinyu Qi
Ping http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01106.html * config/arm/t-arm (MD_INCLUDES): Add marvell-f-iwmmxt.md. * config/arm/marvell-f-iwmmxt.md: New file. * config/arm/arm.md (marvell-f-iwmmxt.md): Include.

[PATCH] Loop IM cost TLC

2011-10-20 Thread Richard Guenther
We've got new tree codes, the following makes Loop IM cost consider those expensive that make sense. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2011-10-19 Richard Guenther rguent...@suse.de * tree-ssa-loop-im.c (stmt_cost): Add WIDEN_*, FMA_EXPR

[PATCH, libcpp] Fix thinko in _cpp_remaining_tokens_num_in_context (PR bootstrap/50801)

2011-10-20 Thread Dodji Seketeli
Hello, The below fixes an embarrassing thinko of mine that breaks bootstrap on SPU and PPC targets (at very least). I am surprised it doesn't break more code. :-( I have lightly tested it on SPU in a cross compiled environment (so I couldn't bootstrap it there) and I have bootstrapped it on

Re: [PATCH PR50572] Tune loop alignment for Atom

2011-10-20 Thread Sergey Ostanevich
Please provide a patch which can be applied.  Cut/paste doesn't create a working patch.  Please attach it. -- H.J. Will that works? Sergos. diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 6c73404..e21cf86 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2011-10-20 Sergey

Re: [PATCH] Account for devirtualization opportunities in inliner

2011-10-20 Thread Richard Guenther
On Wed, Oct 19, 2011 at 11:59 PM, Maxim Kuvyrkov ma...@codesourcery.com wrote: On 28/09/2011, at 4:56 PM, Maxim Kuvyrkov wrote: Jan, The following patch starts a series of patches which improve devirtualization optimizations in GCC. This patch builds on ipa-cp.c and ipa-prop.c

Re: Avoid gcc.dg/tree-prof/val-prof-7.c dependence on strings.h

2011-10-20 Thread Richard Guenther
On Thu, Oct 20, 2011 at 1:25 AM, Joseph S. Myers jos...@codesourcery.com wrote: The testcase gcc.dg/tree-prof/val-prof-7.c includes strings.h to get a declaration of bzero.  This causes it to fail on targets where bzero (a legacy function removed in the latest version of POSIX) is not declared

[PATCH, i386]: Use reciprocal sequences for vectorized SFmode division and sqrtf(x) for -ffast-math

2011-10-20 Thread Uros Bizjak
Hello! This patch builds on recent patch by Michael (that implemented fine-grained control on -mrecip option) and with -ffast-math emits reciprocal sequences with additional NR step for vectorized SFmode division and vectorized sqrtf(x). 2011-10-20 Uros Bizjak ubiz...@gmail.com *

Re: new patches using -fopt-info (issue5294043)

2011-10-20 Thread Richard Guenther
On Thu, Oct 20, 2011 at 1:33 AM, Andi Kleen a...@firstfloor.org wrote: x...@google.com (Rong Xu) writes: After some off-line discussion, we decided to use a more general approach to control the printing of optimization messages/warnings. We will introduce a new option -fopt-info:  *

Re: [PATCH, libcpp] Fix thinko in _cpp_remaining_tokens_num_in_context (PR bootstrap/50801)

2011-10-20 Thread Ulrich Weigand
Dodji Seketeli wrote: I have lightly tested it on SPU in a cross compiled environment (so I couldn't bootstrap it there) and I have bootstrapped it on x86_64-unknown-linux-gnu. One person confirmed in the audit trail of the PR that it fixes the issue for him on PPC, so I am proposing the

Re: [PATCH, i386, PR50766] Fix incorrect mem/reg operands order

2011-10-20 Thread Kirill Yukhin
OK. Thanks, Uros. Great, could anybody please commit that? K

Re: [PATCH, libcpp] Fix thinko in _cpp_remaining_tokens_num_in_context (PR bootstrap/50801)

2011-10-20 Thread Ulrich Weigand
Dodji Seketeli wrote: cpp_context *context = pfile-context; if (context-tokens_kind == TOKENS_KIND_DIRECT) -return ((LAST (context).token - FIRST (context).token) - / sizeof (cpp_token)); +return (LAST (context).token - FIRST (context).token); else if

Re: Avoid -mno-accumulate-outgoing-args in tests on Windows target

2011-10-20 Thread Richard Guenther
On Thu, Oct 20, 2011 at 2:03 AM, Joseph S. Myers jos...@codesourcery.com wrote: The -mno-accumulate-outgoing-args option does not work with the stack probing used on Windows targets, giving a warning and so causing tests using that option to fail.  This patch makes three tests not use that

Re: Use of vector instructions in memmov/memset expanding

2011-10-20 Thread Michael Zolotukhin
Middle-end part of the patch is attached. On 20 October 2011 12:34, Michael Zolotukhin michael.v.zolotuk...@gmail.com wrote: I fixed the tests as well as updated my branch and fixed introduced during this process bugs. Here is fixed complete patch (other parts will be sent in consequent

Re: [PATCH, libcpp] Fix thinko in _cpp_remaining_tokens_num_in_context (PR bootstrap/50801)

2011-10-20 Thread Dodji Seketeli
Ulrich Weigand uweig...@de.ibm.com writes: I can confirm that the patch does fix the newlib build failure I was seeing on SPU. Pheew, thank you. Below is a better patch that I am bootstrapping at the moment. From: Dodji Seketeli do...@redhat.com Date: Thu, 20 Oct 2011 09:43:49 +0200 Subject:

Re: [PATCH] Add gcc-ar/nm/ranlib wrappers for slim LTO

2011-10-20 Thread Richard Guenther
On Thu, Oct 20, 2011 at 9:24 AM, Andi Kleen a...@firstfloor.org wrote: From: Andi Kleen a...@linux.intel.com Slim LTO requires running ar/nm/ranlib with the LTO plugin. The most convenient way to get this into existing Makefiles is using small wrappers that pass the plugin. This matches how

Re: [PATCH, libcpp] Fix thinko in _cpp_remaining_tokens_num_in_context (PR bootstrap/50801)

2011-10-20 Thread Dodji Seketeli
Ulrich Weigand uweig...@de.ibm.com writes: B.t.w. isn't the same thinko also present in the else if path: Right. Jakub spotted it as well. Hence the followup patch in the other subthread. Thanks for watching. -- Dodji

Re: [PATCH, libcpp] Fix thinko in _cpp_remaining_tokens_num_in_context (PR bootstrap/50801)

2011-10-20 Thread Dodji Seketeli
Dodji Seketeli do...@redhat.com writes: libcpp/ * lex.c (_cpp_remaining_tokens_num_in_context): Fix computation of number of tokens. Jakub OKed the patch on IRC, so I went ahead and committed to trunk Thanks. -- Dodji

Re: [PATCH] Add gcc-ar/nm/ranlib wrappers for slim LTO

2011-10-20 Thread Jan Hubicka
On Thu, Oct 20, 2011 at 9:24 AM, Andi Kleen a...@firstfloor.org wrote: From: Andi Kleen a...@linux.intel.com Slim LTO requires running ar/nm/ranlib with the LTO plugin. The most convenient way to get this into existing Makefiles is using small wrappers that pass the plugin. This matches

Re: [Patch ARM] Fix PR target/50106

2011-10-20 Thread Ramana Radhakrishnan
On 19 October 2011 20:38, Nathan Froyd nfr...@mozilla.com wrote: On 10/19/2011 3:27 PM, Ramana Radhakrishnan wrote: Index: gcc/config/arm/arm.c -      live_regs_mask |= extra_mask  (size / UNITS_PER_WORD); +      live_regs_mask |= extra_mask  ((size + 3) / UNITS_PER_WORD); IIUC, wouldn't

Re: [PATCH] Account for devirtualization opportunities in inliner

2011-10-20 Thread Jan Hubicka
Hi, sorry for delayed review. I am still trying to get ipa-inline-analysis to behave well on real codebases and make my mind around how to get more advanced hints, like this one, into it. diff --git a/gcc/ipa-inline-analysis.c b/gcc/ipa-inline-analysis.c index bd4d2ea..5e88c2d 100644 ---

Re: [Patch ARM] Fix PR target/50106

2011-10-20 Thread Jakub Jelinek
On Wed, Oct 19, 2011 at 08:27:26PM +0100, Ramana Radhakrishnan wrote: Ok to backport to 4.6 branch given it is branch freeze time ? I'll be Yeah (with the changes Nathan suggested). 2011-10-19 Ramana Radhakrishnan ramana.radhakrish...@linaro.org PR target/50106 *

Re: [PATCH] Add gcc-ar/nm/ranlib wrappers for slim LTO

2011-10-20 Thread Richard Guenther
On Thu, Oct 20, 2011 at 10:56 AM, Jan Hubicka hubi...@ucw.cz wrote: On Thu, Oct 20, 2011 at 9:24 AM, Andi Kleen a...@firstfloor.org wrote: From: Andi Kleen a...@linux.intel.com Slim LTO requires running ar/nm/ranlib with the LTO plugin. The most convenient way to get this into existing

Honnor -fno-topleverl-reorder with whopr for vars and functions

2011-10-20 Thread Jan Hubicka
Hi, this patch makes -fno-toplevel-reorder to work better with WHOPR. The functions and variables comes out in proper order that is needed for Linux kernel to currently boot with LTO because linker order is important there for kernel's initialization code. I also used this code when comparing

Re: Honnor -fno-topleverl-reorder with whopr for vars and functions

2011-10-20 Thread Richard Guenther
On Thu, 20 Oct 2011, Jan Hubicka wrote: Hi, this patch makes -fno-toplevel-reorder to work better with WHOPR. The functions and variables comes out in proper order that is needed for Linux kernel to currently boot with LTO because linker order is important there for kernel's initialization

Re: [PATCH, i386]: Use reciprocal sequences for vectorized SFmode division and sqrtf(x) for -ffast-math

2011-10-20 Thread Michael Matz
Hi, On Thu, 20 Oct 2011, Uros Bizjak wrote: This patch builds on recent patch by Michael (that implemented fine-grained control on -mrecip option) and with -ffast-math emits reciprocal sequences with additional NR step for vectorized SFmode division and vectorized sqrtf(x). FWIW, I

Re: [PATCH] Distribute inliner's size_time data across entries with similar predicates

2011-10-20 Thread Jan Hubicka
Hi, Jan, The following patch started as a one-liner for ipa-inline-analysis.c: account_size_time() to merge predicates when we are adding data to entry[0] (i.e., when space for 32 size_time entries is exhausted): @@ -537,6 +592,9 @@ account_size_time (struct inline_summary }

Re: [PATCH] Extend vect_recog_bool_pattern also to stores into bool memory (PR tree-optimization/50596)

2011-10-20 Thread Richard Guenther
On Wed, 19 Oct 2011, Jakub Jelinek wrote: Hi! Similarly to casts of bool to integer, even stores into bool arrays can be handled similarly. Just we need to ensure tree-vect-data-refs.c doesn't reject vectorization before tree-vect-patterns.c has a chance to optimize it.

[patch] dwarf2out crash: missing GTY? (PR 50806)

2011-10-20 Thread Jan Kratochvil
Hi, with custom patched dwarf2out.c I got a crash on memory mangled by the garbage collector. With patched GTY there the crash no longer happened - but I do not have a reproducer anymore, sorry if it is a bogus patch. The memory corrupted later was initially allocated and stored into

Re: [patch] dwarf2out crash: missing GTY? (PR 50806)

2011-10-20 Thread Richard Guenther
On Thu, Oct 20, 2011 at 12:14 PM, Jan Kratochvil jan.kratoch...@redhat.com wrote: Hi, with custom patched dwarf2out.c I got a crash on memory mangled by the garbage collector.  With patched GTY there the crash no longer happened - but I do not have a reproducer anymore, sorry if it is a bogus

Re: [PATCH] Extend vect_recog_bool_pattern also to stores into bool memory (PR tree-optimization/50596)

2011-10-20 Thread Jakub Jelinek
On Thu, Oct 20, 2011 at 11:42:01AM +0200, Richard Guenther wrote: + if (TREE_CODE (scalar_dest) == VIEW_CONVERT_EXPR + is_pattern_stmt_p (stmt_info)) +scalar_dest = TREE_OPERAND (scalar_dest, 0); if (TREE_CODE (scalar_dest) != ARRAY_REF TREE_CODE (scalar_dest) !=

Re: [patch] dwarf2out crash: missing GTY? (PR 50806)

2011-10-20 Thread Jakub Jelinek
On Thu, Oct 20, 2011 at 12:21:58PM +0200, Richard Guenther wrote: I don't see how it can make any difference. Indeed, I see no changes in gt-dwarf2out.h with the patch. So it doesn't do anything. 2011-10-20  Jan Kratochvil  jan.kratoch...@redhat.com        * dwarf2out.c (struct

Plug some bogus used uninitialized warnings

2011-10-20 Thread Jan Hubicka
Hi, last time I tried profiledbootstrap with LTO I had to plug the following warnings. Will commit it as obvoius later today. Honza * pt.c (unify_pack_expansion): Iniitalize bad_old_arg and bad_new_arg. * parser.c (cp_parser_ctor_initializer_opt_and_function_body):

Re: Plug some bogus used uninitialized warnings

2011-10-20 Thread Jakub Jelinek
On Thu, Oct 20, 2011 at 12:35:39PM +0200, Jan Hubicka wrote: Hi, last time I tried profiledbootstrap with LTO I had to plug the following warnings. Will commit it as obvoius later today. Please use NULL_TREE instead of NULL for tree initializers. Jakub

[Ada] Fix couple of issues with pragma Source_Reference

2011-10-20 Thread Eric Botcazou
The GNAT specific pragma Source_Reference can alter the source line mapping, leading to (logical) line numbers lower than 1 or greater than the maximum number of lines in the file. Dodji's patch shows that we weren't taking it into account in gigi. Tested on i586-suse-linux, applied on the

[Ada] Housekeeping work in gigi (40/n)

2011-10-20 Thread Eric Botcazou
Tested on i586-suse-linux, applied on the mainline. 2011-10-20 Eric Botcazou ebotca...@adacore.com * gcc-interface/trans.c (lhs_or_actual_p): New predicate. (unchecked_conversion_nop): Use it. (gnat_to_gnu): Likewise. -- Eric Botcazou Index: gcc-interface/trans.c

Re: [PATCH, RFA] Pass address space to REGNO_MODE_CODE_OK_FOR_BASE_P

2011-10-20 Thread Georg-Johann Lay
Ulrich Weigand schrieb: Hello, Georg-Johann Lay has proposed a patch to add named address space support to the AVR target here: http://gcc.gnu.org/ml/gcc-patches/2011-10/msg00471.html Since the target needs to make register allocation decisions for address base registers depending on the

Re: PR bootstrap/50709 (bootstrap miscompare)

2011-10-20 Thread Jan Hubicka
@@ -1392,16 +1393,20 @@ inline_small_functions (void) if (!edge-inline_failed) continue; - /* Be sure that caches are maintained consistent. */ #ifdef ENABLE_CHECKING + /* Be sure that caches are maintained conservatively consistent. + This means that cached

Re: [patch] C6X unwinding/exception handling

2011-10-20 Thread Bernd Schmidt
On 10/17/11 16:10, Nicola Pero wrote: I checked the attached patch, test results at http://gcc.gnu.org/ml/gcc-testresults/2011-10/msg01377.html which are the same as with my suggested patch. Ok for the trunk? I probably don't have authority to approve this, but looks OK to me. The

Re: PING: [PATCH, ARM, iWMMXt][5/5]: pipeline description

2011-10-20 Thread Ramana Radhakrishnan
On 20 October 2011 08:42, Xinyu Qi x...@marvell.com wrote: Ping http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01106.html Index: gcc/config/arm/marvell-f-iwmmxt.md === --- gcc/config/arm/marvell-f-iwmmxt.md(revision 0)

Re: PING: [PATCH, ARM, iWMMXt][1/5]: ARM code generic change

2011-10-20 Thread Ramana Radhakrishnan
On 20 October 2011 08:35, Xinyu Qi x...@marvell.com wrote: Ping http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01100.html        * config/arm/arm.c (arm_option_override): Enable use of iWMMXt with VFP.        Disable use of iwMMXt and Neon.        (arm_expand_binop_builtin): Accept VOIDmode

[patch tree-optimization]: allow branch-cost optimization for truth-and/or on mode-expanded simple boolean-operands

2011-10-20 Thread Kai Tietz
Hello, this patch re-enables the branch-cost optimization on simple boolean-typed operands, which are casted to a wider integral type. This happens due casts from boolean-types are preserved, but FE might expands simple-expression to wider mode. I added two tests for already working

Re: regcprop.c bug fix

2011-10-20 Thread Bernd Schmidt
On 10/19/11 23:24, Mike Stump wrote: So while tracking down a hairy address reload for an output reload bug, copyprop_hardreg_forward_1 was faulting because it was trying to extract move patterns that didn't work out, and when it came back to the code, it then tries to access recog_data, but

Re: Plug some bogus used uninitialized warnings

2011-10-20 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/20/11 04:35, Jan Hubicka wrote: Hi, last time I tried profiledbootstrap with LTO I had to plug the following warnings. Will commit it as obvoius later today. Honza * pt.c (unify_pack_expansion): Iniitalize bad_old_arg and bad_new_arg.

[PATCH, PR50763] Fix for ICE in verify_gimple

2011-10-20 Thread Tom de Vries
Richard, I have a fix for PR50763. The second example from the PR looks like this: ... int bar (int i); void foo (int c, int d) { if (bar (c)) bar (c); d = 33; while (c == d); } ... When compiled with -O2 -fno-dominator-opt, the gimple representation before ftree-tail-merge looks

Re: [PATCH] Add gcc-ar/nm/ranlib wrappers for slim LTO

2011-10-20 Thread Andi Kleen
On Thu, Oct 20, 2011 at 10:45:31AM +0200, Richard Guenther wrote: My previous attempt at using shell scripts for this http://gcc.gnu.org/ml/gcc-patches/2010-10/msg02471.html was not approved. Here's another attempt using wrappers written in C. It's only a single wrapper which just adds a

Re: [patch#2] dwarf2out: Drop the size + performance overhead of DW_AT_sibling

2011-10-20 Thread Jan Kratochvil
On Tue, 18 Oct 2011 10:38:23 +0200, Jakub Jelinek wrote: On Tue, Oct 18, 2011 at 10:28:09AM +0200, Jan Kratochvil wrote: 2011-10-12 Jan Kratochvil jan.kratoch...@redhat.com Stop producing DW_AT_sibling without -gstrict-dwarf. * dwarf2out.c (dwarf2out_finish): Remove calls of

Re: [v3] tr2: bool_set, dynamic_bitset, ratio

2011-10-20 Thread Joseph S. Myers
On Wed, 19 Oct 2011, Ed Smith-Rowland wrote: I don't know if there is a paper yet. I also did rational using the gmp library. I'm wondering if rational should be a template class that could take Having things in libstdc++ etc. using GMP runs into the same issues as libquadmath of not

Re: [PATCH] Add gcc-ar/nm/ranlib wrappers for slim LTO

2011-10-20 Thread Joseph S. Myers
On Thu, 20 Oct 2011, Andi Kleen wrote: collect2 does not fully match what gcc.c does I think, so there's already some divergence. collect2 is always called from within the gcc driver, so it can rely on environment variables set by the driver. As I understand it, these wrappers are not

Re: [PATCH, i386]: Use reciprocal sequences for vectorized SFmode division and sqrtf(x) for -ffast-math

2011-10-20 Thread Uros Bizjak
On Thu, Oct 20, 2011 at 4:45 PM, Joseph S. Myers jos...@codesourcery.com wrote: The patch was tested on x86_64-pc-linux-gnu, but I would like Joseph to check if I didn't mess something with options handling. I have no comments on the option handling in this patch. +for vectorized single

Re: [cxx-mem-model] compare_exchange implementation II

2011-10-20 Thread Richard Henderson
On 10/19/2011 05:43 PM, Andrew MacLeod wrote: * optabs.h (direct_optab_index): Replace DOI_atomic_compare_exchange with DOI_atomic_compare_and_swap. (direct_op): Add DOI_atomic_compare_and_swap. * genopinit.c: Set atomic_compare_and_swap_optab. * expr.h

Re: [PATCH, testsuite]: Require non_strict_align effective target for gcc.dg/ipa/ipa-sra-[26].c

2011-10-20 Thread Uros Bizjak
On Wed, Oct 19, 2011 at 9:50 PM, Uros Bizjak ubiz...@gmail.com wrote: These two tests require non_strict_aligned effective target, since IPA fails in tree_non_mode_aligned_mem_p () for cow and calf candidates for STRICT_ALIGNMENT targets. Mode alignment requires 32 bytes, while data is

Re: trunk (rev 180248) not buildable --with-gc=zone: undefined ggc_alloced_size_for_request

2011-10-20 Thread Dodji Seketeli
Basile Starynkevitch bas...@starynkevitch.net a écrit: libbackend.a(ggc-zone.o): In function `ggc_internal_alloc_zone_stat': /usr/src/Lang/gcc-trunk-bstarynk/gcc/ggc-zone.c:1105: undefined reference to `ggc_alloced_size_for_request' This is my fault. I have tested and committed the below as

[patch, testsuite] Fix vect-120.c failure on IA64

2011-10-20 Thread Steve Ellcey
I am going to check this change in as obvious later today, the test includes a conversion from float to int in the loop and if that isn't supported by a target, then the loop is not vectorized. This test has been failing on IA64 and perhaps on ARM too, there was a reference to it in PR 50150. I

Re: [PATCH] New port for TILEPro and TILE-Gx 2/7: changes in contrib

2011-10-20 Thread Walter Lee
Here is a resubmission of the contrib patch, adding the entries to gcc_update to handle the multiply tables. * config-list.mk (tilegx-linux-gnu): Add. (tilepro-linux-gnu): Add. * gcc_update (gcc/config/tilegx/mul-tables.c): New dependencies.

Re: [PATCH] New port for TILEPro and TILE-Gx: 5/7 libgcc port

2011-10-20 Thread Walter Lee
Here is a resubmission of the libgcc patch, using soft-fp as the floating point library. I plan to do the benchmarking between the implementations as suggested, but I'd like to decouple that from the initial submission. * config.host: Handle tilegx and tilepro. *

[patch, testsuite] Patch for gcc.dg/pr49994-3.c on HP-UX

2011-10-20 Thread Steve Ellcey
I am going to check this change in as obvious later today if there are no objections, the test gives warnings on HP-UX because it calls __builtin_return_address with arguments of 0 through 5 but the value 0 is the only valid argument to __builtin_return_address on HP-UX. Tested on IA64 and PA

Re: [patch, testsuite] Patch for gcc.dg/pr49994-3.c on HP-UX

2011-10-20 Thread Rainer Orth
Steve Ellcey s...@cup.hp.com writes: Index: gcc.dg/pr49994-3.c === --- gcc.dg/pr49994-3.c(revision 180233) +++ gcc.dg/pr49994-3.c(working copy) @@ -2,6 +2,7 @@ /* { dg-options -O2 -fsched2-use-superblocks -g }

Re: [PATCH][PING] Vectorize conversions directly

2011-10-20 Thread Richard Henderson
On 10/20/2011 09:24 AM, Dmitry Plotnikov wrote: gcc/ * tree-cfg.c (verify_gimple_assign_unary): Allow vector conversions. * tree-vect-stmts.c (supportable_convert_operation): New function. (vectorizable_conversion): Call it. Change condition and behavior for NONE modifier

Re: new patches using -fopt-info (issue5294043)

2011-10-20 Thread Xinliang David Li
On Thu, Oct 20, 2011 at 1:21 AM, Richard Guenther richard.guent...@gmail.com wrote: On Thu, Oct 20, 2011 at 1:33 AM, Andi Kleen a...@firstfloor.org wrote: x...@google.com (Rong Xu) writes: After some off-line discussion, we decided to use a more general approach to control the printing of

Re: new patches using -fopt-info (issue5294043)

2011-10-20 Thread Rong Xu
Richard, Thanks for the comments. Let me give some background of the patch: The initial intention of the patch is to suppress the verbose warnings and notes emitted in profile-use compilation. This warnings/notes are caused by inconsistent profile due to data race (which is currently common in

Breakage with Update testsuite to run with slim LTO

2011-10-20 Thread Hans-Peter Nilsson
Date: Tue, 27 Sep 2011 19:23:22 +0200 From: Jan Hubicka hubi...@ucw.cz this patch updates testsuite to cover both fat and slim LTO when linker plugin is used and also both linker plugin and collect2 paths. I didn't wanted to slow down testing too much so I just distributes the flags across

Re: Breakage with Update testsuite to run with slim LTO

2011-10-20 Thread Andi Kleen
Looks like this patch broke, for cris-elf with TOT binutils: Running /tmp/hpautotest-gcc1/gcc/gcc/testsuite/gcc.dg/torture/dg-torture.exp ... FAIL: gcc.dg/torture/cris-asm-mof-1.c scan-assembler in-asm: .mof FAIL: gcc.dg/torture/cris-asm-mof-1.c scan-assembler out-asm: .mof FAIL:

Re: new patches using -fopt-info (issue5294043)

2011-10-20 Thread Xinliang David Li
While discussion for trunk version is still going, it is ok for google branches. thanks, David On Wed, Oct 19, 2011 at 4:28 PM, Rong Xu x...@google.com wrote: After some off-line discussion, we decided to use a more general approach to control the printing of optimization messages/warnings.

Remove target.vectorize.builtin_vec_perm

2011-10-20 Thread Richard Henderson
Since the vectorizer has been changed to emit VEC_PERM_EXPR, I've now removed the hook and the implementations of that hook. For the x86 target I also removed the builtins themselves. For the rs6000 and spu targets, I've left that detail to the port maintainers; I don't know what interfaces are

C++ PATCH for c++/41449 (EH cleanup of partially-aggregate-initialized objects)

2011-10-20 Thread Jason Merrill
The C++ standard says that if an exception is thrown during initialization of a class, any fully-constructed subobjects are destroyed. We already handled that properly for objects initialized via constructor, but we weren't handling it properly for aggregate initialization. This patch adds

Re: [PATCH PR50572] Tune loop alignment for Atom

2011-10-20 Thread H.J. Lu
On Thu, Oct 20, 2011 at 1:05 AM, Sergey Ostanevich sergos@gmail.com wrote: Please provide a patch which can be applied.  Cut/paste doesn't create a working patch.  Please attach it. -- H.J. Will that works? Sergos. diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 6c73404..e21cf86

Re: new patches using -fopt-info (issue5294043)

2011-10-20 Thread Andi Kleen
This warnings/notes are caused by inconsistent profile due to data race (which is currently common in multi-thread programs), I never quite understood why the gcov counters are not simply marked __thread. This would make the profiled programs faster too because they wouldn't bounce cache lines

Re: [PATCH] New port for TILEPro and TILE-Gx 3/7: gcc port

2011-10-20 Thread Joseph S. Myers
On Thu, 20 Oct 2011, Walter Lee wrote: +#undef MCOUNT_NAME +#define MCOUNT_NAME mcount For a new target it seems much better to define your ABI to use a name in the reserved namespace for this - that is, starting with two underscores. I've changed it to use _mcount with one

Re: new patches using -fopt-info (issue5294043)

2011-10-20 Thread Xinliang David Li
On Thu, Oct 20, 2011 at 12:53 PM, Andi Kleen a...@firstfloor.org wrote: This warnings/notes are caused by inconsistent profile due to data race (which is currently common in multi-thread programs), I never quite understood why the gcov counters are not simply marked __thread. This would make

Use .exe suffix on LTO test executables

2011-10-20 Thread Joseph S. Myers
As I noted in http://gcc.gnu.org/ml/gcc-patches/2008-09/msg00905.html, test executables with no .exe or .something suffix are problematic for testing on Windows targets. This patch fixes the LTO tests to use such suffixes, like other tests. Tested with cross to i686-mingw32. OK to commit?

Re: Use .exe suffix on LTO test executables

2011-10-20 Thread Diego Novillo
On Thu, Oct 20, 2011 at 16:23, Joseph S. Myers jos...@codesourcery.com wrote: 2011-10-20  Joseph Myers  jos...@codesourcery.com        * lib/lto.exp (lto-execute): Use .exe suffix for test executable        names. OK. Diego.

Re: [PATCH, i386, PR50766] Fix incorrect mem/reg operands order

2011-10-20 Thread H.J. Lu
On Thu, Oct 20, 2011 at 1:30 AM, Kirill Yukhin kirill.yuk...@gmail.com wrote: OK. Thanks, Uros. Great, could anybody please commit that? I checked it in for you. -- H.J.

[cxx-mem-model] expand_atomic_load: Handle an empty target

2011-10-20 Thread Aldy Hernandez
Found this while testing the branch on ia64. The call to expand_val_compare_and_swap() above returns NULL_RTX when it can't find a suitable instruction. OK for branch? * optabs.c (expand_atomic_load): Handle a NULL target. Index: optabs.c

Re: [cxx-mem-model] expand_atomic_load: Handle an empty target

2011-10-20 Thread Andrew MacLeod
On 10/20/2011 04:54 PM, Aldy Hernandez wrote: Found this while testing the branch on ia64. The call to expand_val_compare_and_swap() above returns NULL_RTX when it can't find a suitable instruction. OK for branch? yes. btw, did you audit the other new expand routines to see if they

Fix gcc.dg/lto/pr46940_0.c for assembler name prefixes

2011-10-20 Thread Joseph S. Myers
gcc.dg/lto/pr46940_0.c needs fixing for targets using a prefix on assembler names, similar to the fixes recently made by Joern to some other testcases. This patch fixes it in the same way as Joern fixed gcc.dg/lto/20081222_1.c. Testes with cross to i686-mingw32. OK to commit? 2011-10-20

[cxx-mem-model] Handle x86-64 with -m32

2011-10-20 Thread Aldy Hernandez
These operations don't exist on x86-32 bits, and when running multilibed tests, the target is still x86_64-unknown-linux-gnu but the target is 32-bits when using -m32. The following change checks that we are actually running in 64-bits before assuming sync_int_128 or sync_long_long exist on

Rename builtin_vec_perm_ok to vec_perm_const_ok

2011-10-20 Thread Richard Henderson
... since it no longer applies to a builtin. Tested on x86_64-linux. r~ * target.def (vec_perm_const_ok): Rename from builtin_vec_perm_ok. * optabs.c (can_vec_perm_expr_p): Update to match. (expand_vec_perm_expr): Likewise. * config/i386/i386.c

Re: [cxx-mem-model] expand_atomic_load: Handle an empty target

2011-10-20 Thread Aldy Hernandez
On 10/20/11 15:56, Andrew MacLeod wrote: On 10/20/2011 04:54 PM, Aldy Hernandez wrote: Found this while testing the branch on ia64. The call to expand_val_compare_and_swap() above returns NULL_RTX when it can't find a suitable instruction. OK for branch? yes. btw, did you audit the other new

Re: Fix gcc.dg/lto/pr46940_0.c for assembler name prefixes

2011-10-20 Thread Diego Novillo
On Thu, Oct 20, 2011 at 17:01, Joseph S. Myers jos...@codesourcery.com wrote: gcc.dg/lto/pr46940_0.c needs fixing for targets using a prefix on assembler names, similar to the fixes recently made by Joern to some other testcases.  This patch fixes it in the same way as Joern fixed

Re: [patch, testsuite] Patch for gcc.dg/pr49994-3.c on HP-UX

2011-10-20 Thread Steve Ellcey
On Thu, 2011-10-20 at 18:23 +0200, Rainer Orth wrote: Steve Ellcey s...@cup.hp.com writes: Index: gcc.dg/pr49994-3.c === --- gcc.dg/pr49994-3.c (revision 180233) +++ gcc.dg/pr49994-3.c (working copy) @@ -2,6

[RFC PATCH] SLP vectorize calls

2011-10-20 Thread Jakub Jelinek
Hi! While looking at *.vect dumps from Polyhedron, I've noticed the lack of SLP vectorization of builtin calls. This patch is an attempt to handle at least 1 and 2 operand builtin calls (SLP doesn't handle ternary stmts either yet), where all the types are the same. E.g. it can handle extern

Re: Breakage with Update testsuite to run with slim LTO

2011-10-20 Thread Jan Hubicka
Date: Tue, 27 Sep 2011 19:23:22 +0200 From: Jan Hubicka hubi...@ucw.cz this patch updates testsuite to cover both fat and slim LTO when linker plugin is used and also both linker plugin and collect2 paths. I didn't wanted to slow down testing too much so I just distributes the

Re: [cxx-mem-model] Handle x86-64 with -m32

2011-10-20 Thread Joseph S. Myers
On Thu, 20 Oct 2011, Aldy Hernandez wrote: These operations don't exist on x86-32 bits, and when running multilibed tests, the target is still x86_64-unknown-linux-gnu but the target is 32-bits when using -m32. Any test that only handles one of x86_64-* and i?86-* is automatically wrong; you

Re: [cxx-mem-model] Handle x86-64 with -m32

2011-10-20 Thread H.J. Lu
On Thu, Oct 20, 2011 at 3:38 PM, Joseph S. Myers jos...@codesourcery.com wrote: On Thu, 20 Oct 2011, Aldy Hernandez wrote: These operations don't exist on x86-32 bits, and when running multilibed tests, the target is still x86_64-unknown-linux-gnu but the target is 32-bits when using -m32.

[v3] libstdc++/50196 - enable std::thread, std::mutex etc. on darwin

2011-10-20 Thread Jonathan Wakely
This patch should enable macosx support for thread and partial support for mutex, by defining _GLIBCXX_HAS_GTHREADS on POSIX systems without the _POSIX_TIMEOUTS option, and only disabling the types which rely on the Timeouts option, std::timed_mutex and std::recursive_timed_mutex, instead of

[PATCH] Add gcc-ar/nm/ranlib wrappers for slim LTO v2

2011-10-20 Thread Andi Kleen
From: Andi Kleen a...@linux.intel.com Slim LTO requires running ar/nm/ranlib with the LTO plugin. The most convenient way to get this into existing Makefiles is using small wrappers that pass the plugin. This matches how other compilers (LLVM, icc) do this too. My previous attempt at using shell

[C++ Patch] PR 50811 (rejects class-virt-specifier if class-head-name includes nested-name-specifier)

2011-10-20 Thread Ville Voutilainen
Tested on X86-32 linux. 2011-10-21 Ville Voutilainen ville.voutilai...@gmail.com PR c++/50811 * parser.c (cp_parser_class_head): Parse virt-specifiers regardless of whether an id is present diff --git a/gcc/cp/parser.c b/gcc/cp/parser.c index ea0c4dc..dd2357b 100644 ---

[commit, spu] Fix vec_perm pattern (Re: [rs6000, spu] Add vec_perm named pattern)

2011-10-20 Thread Ulrich Weigand
Richard Henderson wrote: The generic support for vector permutation will allow for automatic lowering to V*QImode, so all we need to add to support for these targets is the single V16QI pattern that represents the base permutation insn. I'm not touching any of the other ways that the

Re: regcprop.c bug fix

2011-10-20 Thread Mike Stump
On Oct 20, 2011, at 6:22 AM, Bernd Schmidt wrote: I found that maximally confusing, so let me try to rephrase it to see if I understood you. The two calls to validate_change clobber the recog_data even if they fail. In case they failed, we want to continue looking at data from the original

[RFA:] fix breakage with Update testsuite to run with slim LTO

2011-10-20 Thread Hans-Peter Nilsson
Date: Fri, 21 Oct 2011 00:19:32 +0200 From: Jan Hubicka hubi...@ucw.cz Yes, if we scan assembler, we likely want -fno-fat-lto-objects. then IIUC you need to patch *all* torture tests that use scan-assembler and scan-assembler-not. Alternatively, patch somewhere else, like not passing it

[pph] Re-organize handling of mergeable nodes (issue5318043)

2011-10-20 Thread Diego Novillo
This patch re-organizes the streaming of mergeable nodes so that we can separate the process of merging the incoming ASTs into the current compilation context from the reading of their contents. This problem occurs when two or more PPH images in the same compilation unit have embedded the same

Re: [pph] Make libcpp symbol validation a warning (issue5235061)

2011-10-20 Thread Gabriel Charette
I just thought about something.. Earlier I said that ALL line_table issues were resolved after this patch (as it ignores the re-included headers that were guarded, as the non-pph compiler does naturally). One problem remains however, I'm pretty sure that re-included non-pph'ed header's

Re: [patch] dwarf2out crash: missing GTY? (PR 50806)

2011-10-20 Thread Laurynas Biveinis
2011/10/20 Jan Kratochvil jan.kratoch...@redhat.com: Hi, with custom patched dwarf2out.c I got a crash on memory mangled by the garbage collector.  With patched GTY there the crash no longer happened - but I do not have a reproducer anymore, sorry if it is a bogus patch. The memory

Re: [PATCH, i386, PR50766] Fix incorrect mem/reg operands order

2011-10-20 Thread Kirill Yukhin
Thanks! K On Fri, Oct 21, 2011 at 12:37 AM, H.J. Lu hjl.to...@gmail.com wrote: On Thu, Oct 20, 2011 at 1:30 AM, Kirill Yukhin kirill.yuk...@gmail.com wrote: OK. Thanks, Uros. Great, could anybody please commit that? I checked it in for you. -- H.J.

Re: [patch] dwarf2out crash: missing GTY? (PR 50806)

2011-10-20 Thread Jan Kratochvil
On Fri, 21 Oct 2011 05:37:09 +0200, Laurynas Biveinis wrote: This patch is a no-op, as already pointed out. If this comes up again, I'd set a conditional breakpoint on ggc_set_mark if (arg == struct dw_loc_list_struct with the field that gets collected) and try to find out why the field does

  1   2   >