Re: RFC [1/2] divmod transform

On 27 May 2016 at 17:31, Richard Biener wrote: > On Fri, 27 May 2016, Prathamesh Kulkarni wrote: > >> On 27 May 2016 at 15:45, Richard Biener wrote: >> > On Wed, 25 May 2016, Prathamesh Kulkarni wrote: >> > >> >> On 25 May 2016 at 12:52, Richard Biener

Re: [PATCH 5/5] workaround for PR70427

On Mon, May 30, 2016 at 02:34:03AM +0200, Jan Hubicka wrote: > > diff --git a/gcc/ipa-profile.c b/gcc/ipa-profile.c > > index da17bcd..c7d7792 100644 > > --- a/gcc/ipa-profile.c > > +++ b/gcc/ipa-profile.c > > @@ -201,6 +201,8 @@ ipa_profile_generate_summary (void) > > if

Re: [PATCH 1/5] Add gcc-auto-profile script

On Mon, May 30, 2016 at 02:39:06AM +0200, Jan Hubicka wrote: > > > > Since maintaining the script would be somewhat tedious (needs changes > > every time a new CPU comes out) I auto generated it from the online > > Intel event database. The script to do that is in contrib and can be > > rerun. >

Re: [PATCH 1/5] Add gcc-auto-profile script

Andi, thanks a lot for working on the auto-fdo bootstrap. It is badly needed to have some coverage for this feature. I don't think I can approve the build machinery changes. > From: Andi Kleen > > Using autofdo is currently something difficult. It requires using the >

Re: [PATCH2][PR71252] Fix insertion point of stmt_to_insert

On 28/05/16 01:28, Kugan Vivekanandarajah wrote: Hi Richard, This fix insertion point of stmt_to_insert based on your comments. In insert_stmt_before_use , I now use find_insert_point such that we insert the stmt_to_insert after its operands are defined. This means that we now have to insert

Re: [PATCH 5/5] workaround for PR70427

> From: Andi Kleen > > This makes autofdo bootstrap not crash. > > This is probably not the right fix, but for now it works for me. > Not for submission. > --- > gcc/ipa-profile.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/gcc/ipa-profile.c

Re: [PATCH 2/5] Don't cause ICEs when auto profile file is not found with checking

> From: Andi Kleen > > Currently, on a checking enabled compiler when -fauto-profile does > not find the profile feedback file it errors out with assertation > failures. Add proper errors for this case. > > gcc/: > > 2016-05-21 Andi Kleen > >

[C++ Patch] PR 71109 ("Misleading diagnostic message with 'virtual' used in out-of-line definitions of class template member functions")

Hi, submitter noticed that for wrong uses of 'virtual' outside of template classes (B in the testcase) vs plain classes (A) we wrongly emit the "templates may not be %" error message. Simply checking current_class_type seems enough to solve the problem. Case C in the extended

Re: [PING^2] Re: Updated autofdo bootstrap and testing patches

Andi Kleen writes: Ping^2! > Andi Kleen writes: > > Ping! > >> Here's an updated version of the patchkit to enable autofdo bootstrap >> and testing. It also fixes some autofdo issues. The last patch is more a >> workaround >> (to make autofdo

PING^4: [PATCH] PR target/70454: Build x86 libgomp with -march=i486 or better

On Fri, May 20, 2016 at 8:04 AM, H.J. Lu wrote: > On Mon, May 9, 2016 at 5:52 AM, H.J. Lu wrote: >> On Mon, May 2, 2016 at 6:46 AM, H.J. Lu wrote: >>> On Mon, Apr 25, 2016 at 1:36 PM, H.J. Lu wrote: If

PING: [PATCH] Load external function address via GOT slot

On Mon, May 9, 2016 at 10:37 AM, H.J. Lu wrote: > On Fri, Apr 22, 2016 at 6:03 AM, Uros Bizjak wrote: >> On Fri, Apr 22, 2016 at 2:54 PM, H.J. Lu wrote: >>> For -fno-plt, we load the external function address via the GOT slot >>> so

Re: [PATCH][i386] Add -march=native support for VIA nano CPUs

Hello, On Sun, 2016-05-29 at 21:12 +0200, Uros Bizjak wrote: > Hello! > > > > > When trying to compile using -march=native on a VIA nano CPU, gcc > > selects "-march=core2" "-mtune=i386" then is unable to compile, as > > this > > creates a conflicts between 32 bits and 64 bits compilation

Re: [PATCH] match.pd: optimize unsigned mul overflow check

On Sat, 28 May 2016, Alexander Monakov wrote: For unsigned A, B, 'A > -1 / B' is a nice predicate for checking whether 'A*B' overflows (or 'B && A > -1 / B' if B may be zero). Let's optimize it to an invocation of __builtin_mul_overflow to avoid the divide operation. I forgot to ask earlier:

[RFC PATCH, i386]: Use "lock orl $0, -4(%esp)" in mfence_nosse

Hello! As explained in PR71245, comment #3 [1], it is better to use offset -4 to a %esp to implement a non-SSE memory fence instruction: -q- I guess it costs a code byte for a disp8 in the addressing mode, but it avoids adding a lot of latency to a critical path involving a spill/reload to

Re: [patch] doc/sourcebuild.texi (Directives): Remove extra closing braces.

On Sat, 16 Jan 2016, Jonathan Wakely wrote: > This removes stray closing braces in the docs for dg-error, dg-warning > etc. > > OK for trunk? Yes. Sorry for the delay. I expected someone else to pick this up for review/approval, but now noticed that this patch apparently has not been

[PATCH, i386]: Fix PR71245, atomic load/store bounces the data to the stack using fild/fistp

Hello! As shown in the PR, when moving DFmode value to/from a FP register, we don't need to bounce it with an atomic DImode fild/fistp in case of -march=pentium. DFmode move is atomic by itself. 2016-05-29 Uros Bizjak PR target/71245 * config/i386/sync.md

Re: [PATCH] gcc/config/tilegx/tilegx.c (tilegx_function_profiler): Save r10 to stack before call mcount

On May 29, 2016, at 3:39 AM, cheng...@emindsoft.com.cn wrote: > > r10 may also be as parameter for the nested function, so need save it > before call mcount. mcount can have a special abi where it preserves more registers than one would otherwise expect. I'm wondering if you know what

Re: [PATCH][i386] Add -march=native support for VIA nano CPUs

Hello! > When trying to compile using -march=native on a VIA nano CPU, gcc > selects "-march=core2" "-mtune=i386" then is unable to compile, as this > creates a conflicts between 32 bits and 64 bits compilation modes, as > show by the following test: [...] > ---

Make vectorizer to use likely upper bounds

Hi, this patch makes vectorizer to give up when likely maximal number of iterations is low. boostrapped/regtested x86_64-linux, will commit it once benchmark machines pick up the earlier changes. * tree-vect-loop.c (vect_analyze_loop_2): Use likely_max_stmt_executions_int. Index:

Fix profile updating after complette unrolling

Hi, this patch fixes profile updating after complette unroll. Present code expects that unrolling is done only in the case there is exit which counts iteration and the number of iterations is n_unroll. This is no longer true because we can derive upper bounds on number of iterations from array

Fix maybe_hot_frequency_p

Hi, this patch makes maybe_hot_frequency_p to use multiplication instead of division. This eliminates roundoff errors that are quite serious in this case: HOT_BB_FREQUENCY_FRACTION is 1000 while frequencies are scaled from 0...1. If there is loop in function then most likely no BB is

Contents of PO file 'cpplib-6.1.0.sr.po'

cpplib-6.1.0.sr.po.gz Description: Binary data The Translation Project robot, in the name of your translation coordinator.

New Serbian PO file for 'cpplib' (version 6.1.0)

Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'cpplib' has been submitted by the Serbian team of translators. The file is available at: http://translationproject.org/latest/cpplib/sr.po (This file,

Re: [PATCH] match.pd: optimize unsigned mul overflow check

On Sat, 28 May 2016, Alexander Monakov wrote: For unsigned A, B, 'A > -1 / B' is a nice predicate for checking whether 'A*B' overflows (or 'B && A > -1 / B' if B may be zero). Let's optimize it to an invocation of __builtin_mul_overflow to avoid the divide operation. Hmm, that division by

Re: libiberty: Don't needlessly clear xmemdup allocated memory

Ok then. Thanks!

Re: libiberty: Don't needlessly clear xmemdup allocated memory

On Sat, May 28, 2016 at 10:12:19PM -0400, DJ Delorie wrote: > > Alan Modra writes: > > * xmemdup.c (xmemdup): Use xmalloc rather than xcalloc. > > In glibc at least, calloc can be faster than memset if the kernel is > pre-zero-ing pages. Thus, in those cases, your change

Re: [v3 PATCH] Protect allocator-overloads of tuple-from-tuple constructors from cases that would create dangling references.

On 28 May 2016 at 21:25, Ville Voutilainen wrote: > The fix to avoid binding dangling references to temporaries for tuple's > constructors that take tuples of different type didn't include the fix > for allocator overloads. That was just lazy, and I should feel

[PING] [PATCH] Fix ICE with x87 asm operands (PR inline-asm/68843)

Hi, ping for the RTL optimization stuff. The problem here is that the code in reg-stack.c pretty much everywhere assumes that the stack registers do not have gaps. IMHO it is not worth to fix the register allocation in a way that would be necessary for that configuration to work correctly. So

Re: Revert gcc r227962

On 5/23/2016 16:56, JonY wrote: > On 5/20/2016 06:36, JonY wrote: >> On 5/20/2016 02:11, Jeff Law wrote: >>> So if we make this change (revert 227962), my understanding is that >>> cygwin bootstraps will fail because they won't find kernel32 and perhaps >>> other libraries. >>> >>> Jeff >>> >> >>

[PATCH, PR69067] Remove assert in get_def_bb_for_const

Hi, this patch fixes graphite PR69067, a 6/7 regression. I. Consider the following test-case, compiled with -O1 -floop-nest-optimize -flto: ... int a1, c1, cr, kt; int aa[2]; int ce (void) { while (a1 < 1) { int g8; for (g8 = 0; g8 < 3; ++g8) if (c1 != 0) cr = aa[a1

[PATCH] gcc/config/tilegx/tilegx.c (tilegx_function_profiler): Save r10 to stack before call mcount

From: Chen Gang r10 may also be as parameter for the nested function, so need save it before call mcount. 2016-05-29 Chen Gang * config/tilegx/tilegx.c (tilegx_function_profiler): Save r10 to stack before call mcount. ---

[wwwdocs] readings.html -- fix two links to renesas.com

Admittedly the new addresses are nicer (I never appreciated the original ones); just why a compentent webmaster would not add a redirect escapes my understanding. Applied. Gerald Index: htdocs/readings.html === RCS file:

[wwwdocs] Remove broken link to old Intel handook in projects/tree-ssa

This one tries to redirect on the Intel side, and then even leads to a "Server not found" Committed. Gerald Index: htdocs/projects/tree-ssa/vectorization.html === RCS file:

[PATCH,libstdc++] Adjust link in doc/xml/manual/backwards_compatibility.xml

There is quite a bit more to fix in the libstdc++ manuals, I'll try to make some progress the coming weeks. Applied. Gerald 2016-05-29 Gerald Pfeifer * doc/xml/manual/backwards_compatibility.xml: Adjust lists.debian.org link to https. *

Re: [C++ Patch] PR 71105 ("lambdas with default captures improperly have function pointer conversions")

Hi, On 29/05/2016 05:04, Jason Merrill wrote: OK for trunk and 6. Thanks. The regression is from another issue; this bug just prevents a simple workaround for that bug. Ah now I see, I confused the test attached to the bug and the test provided inline at then end of Comment #0. I'm also

[PATCH][i386] Add -march=native support for VIA nano CPUs

When trying to compile using -march=native on a VIA nano CPU, gcc selects "-march=core2" "-mtune=i386" then is unable to compile, as this creates a conflicts between 32 bits and 64 bits compilation modes, as show by the following test: # echo 'int main(){return 0;}' > test.c && gcc -march=native