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

From: Andi Kleen 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 scripts for this htt

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

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

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): Revise

RE: PING: [PATCH, ARM, iWMMXt][2/5]: intrinsic head file change

Ping http://gcc.gnu.org/ml/gcc-patches/2011-08/msg01963.html * config/arm/mmintrin.h: Revise.

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

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. (arm_output_iwmmxt_tinsr)

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

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

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 * tree-ssa-loop-im.c (stmt_cost): Add WIDEN_*, FMA_EXPR and rotates to the

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

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 x86

Re: [PATCH PR50572] Tune loop alignment for Atom

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

Re: [PATCH] Account for devirtualization opportunities in inliner

On Wed, Oct 19, 2011 at 11:59 PM, Maxim Kuvyrkov 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 infrastructure for a

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

On Thu, Oct 20, 2011 at 1:25 AM, Joseph S. Myers wrote: > The testcase gcc.dg/tree-prof/val-prof-7.c includes 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 in that header; declaring it

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

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 * config/i386/i386.h (RE

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

On Thu, Oct 20, 2011 at 1:33 AM, Andi Kleen 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: >>  * fopt-info=0 or fno

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

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

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

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 (contex

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

On Thu, Oct 20, 2011 at 2:03 AM, Joseph S. Myers 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 > option on affected targ

Re: Use of vector instructions in memmov/memset expanding

Middle-end part of the patch is attached. On 20 October 2011 12:34, Michael Zolotukhin 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 letters). > > The changes passed b

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

"Ulrich Weigand" 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 Date: Thu, 20 Oct 2011 09:43:49 +0200 Subject: [PATCH] Fix thinko in _cpp_rema

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

On Thu, Oct 20, 2011 at 9:24 AM, Andi Kleen wrote: > From: Andi Kleen > > 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 thi

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

"Ulrich Weigand" 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)

Dodji Seketeli 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

> On Thu, Oct 20, 2011 at 9:24 AM, Andi Kleen wrote: > > From: Andi Kleen > > > > 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 > > (LLV

Re: [Patch ARM] Fix PR target/50106

On 19 October 2011 20:38, Nathan Froyd 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 ((size

Re: [PATCH] Account for devirtualization opportunities in inliner

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 --- a/gcc/i

Re: [Patch ARM] Fix PR target/50106

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 > > PR target/50106 > * config/arm/arm.c (thumb_unexpanded_

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

On Thu, Oct 20, 2011 at 10:56 AM, Jan Hubicka wrote: >> On Thu, Oct 20, 2011 at 9:24 AM, Andi Kleen wrote: >> > From: Andi Kleen >> > >> > 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 tha

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

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 var

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

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 initializati

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

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 di

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

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)

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

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

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 mem_loc_res

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

On Thu, Oct 20, 2011 at 12:14 PM, Jan Kratochvil 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 patch. > > The memor

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

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 (scala

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

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   > > > >        * dwarf2out.c (struct dw_loc_list_struct): Add GTY

Plug some bogus used uninitialized warnings

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

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

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 mainl

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

Tested on i586-suse-linux, applied on the mainline. 2011-10-20 Eric Botcazou * 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

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

Re: PR bootstrap/50709 (bootstrap miscompare)

> @@ -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 tha

Re: [patch] C6X unwinding/exception handling

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 O

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

On 20 October 2011 08:42, Xinyu Qi 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) > +++ gcc/conf

[PATCH, doc] Use @smallexample instead of @quotation in cppopts.texi (PR other/50659)

The script contrib/texi2pod.pl doesn't recognize @quotation/@end quotation constructs, so it was given some errors in while building the part cppopts.texi that I added for -fdebug-cpp. I used @smallexample/@end smallexample instead. Tested and committed to trunk as obvious. gcc/

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

On 20 October 2011 08:35, Xinyu Qi 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 op. >  

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

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 branch-co

Re: regcprop.c bug fix

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

Re: Plug some bogus used uninitialized warnings

-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

[PATCH, PR50763] Fix for ICE in verify_gimple

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 like

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

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 a

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

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 > > > > Stop producing DW_AT_sibling without -gstrict-dwarf. > > * dwarf2out.c (dwarf2out_finish): Remove calls of > > add_sibling

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

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 wanti

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

On Thu, 20 Oct 2011, Andi Kleen wrote: > The logic gcc.c uses to find the files is very complicated. I didn't > try to replicate it 100% and left out some magic. I would be interested > if this simple method works for everyone or if more code needs > to be added. This only needs to support LTO sup

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

On Thu, 20 Oct 2011, Uros Bizjak 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 float division and vectorized sqrtf(x) alr

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

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 call

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

On Thu, Oct 20, 2011 at 4:45 PM, Joseph S. Myers 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 float division and

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

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 (expa

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

On Wed, Oct 19, 2011 at 9:50 PM, Uros Bizjak 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 aligned to 8 byt

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

Basile Starynkevitch 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 per the obvious rule.

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

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

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. (gcc/config/tilepro/mul

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

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. * config/tilegx

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

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 HP

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

Steve Ellcey 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

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 modi

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

On Thu, Oct 20, 2011 at 1:21 AM, Richard Guenther wrote: > On Thu, Oct 20, 2011 at 1:33 AM, Andi Kleen 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 wil

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

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 mul

Breakage with "Update testsuite to run with slim LTO"

> Date: Tue, 27 Sep 2011 19:23:22 +0200 > From: Jan Hubicka > 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 existing

Re: Breakage with "Update testsuite to run with slim LTO"

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: gcc.dg/torture

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

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 wrote: > After some off-line discussion, we decided to use a more general approach > to control the printing of optimization messages/warnings. We will > intro

Remove target.vectorize.builtin_vec_perm

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 ac

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

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 th

Re: [PATCH PR50572] Tune loop alignment for Atom

On Thu, Oct 20, 2011 at 1:05 AM, Sergey Ostanevich 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 1006

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

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

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

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

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

On Thu, Oct 20, 2011 at 12:53 PM, Andi Kleen 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 the profil

Use .exe suffix on LTO test executables

As I noted in , test executables with no .exe or . 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? 2011-10-20

Re: Use .exe suffix on LTO test executables

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

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

On Thu, Oct 20, 2011 at 1:30 AM, Kirill Yukhin 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

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

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 handl

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

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 Joseph

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

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 th

Rename builtin_vec_perm_ok to vec_perm_const_ok

... 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 (TARGET_VECTORIZE_V

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

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

On Thu, Oct 20, 2011 at 17:01, Joseph S. Myers 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 > gcc.dg/lto/20081222_1.c. > > Tes

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

On Thu, 2011-10-20 at 18:23 +0200, Rainer Orth wrote: > Steve Ellcey 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 @@

[RFC PATCH] SLP vectorize calls

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 flo

Re: Breakage with "Update testsuite to run with slim LTO"

> > Date: Tue, 27 Sep 2011 19:23:22 +0200 > > From: Jan Hubicka > > > 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

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

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; y

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

On Thu, Oct 20, 2011 at 3:38 PM, Joseph S. Myers 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. > > Any test that

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

This patch should enable macosx support for and partial support for , 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 disabling all thr

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

From: Andi Kleen 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 scripts for this htt

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

Tested on X86-32 linux. 2011-10-21 Ville Voutilainen 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 --- a/gcc/cp/parser.c +++ b/gcc/

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

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 pe

Re: regcprop.c bug fix

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"

> Date: Fri, 21 Oct 2011 00:19:32 +0200 > From: Jan Hubicka > 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 if ce

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

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 te

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

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 line_table

  1   2   >