[PATCH]: Guard division by zero in ipa-icf dumps

2014-10-24 Thread Uros Bizjak
Hello! 2014-10-24 Martin Liska mli...@suse.cz * ipa-icf.c (sem_item_optimizer::parse_nonsingleton_classes): Guard division by zero in dumps. (sem_item_optimizer::merge_classes): Ditto. Tested on alphaev68-linux-gnu, committed to mainline as obvious. Uros. diff --git

Re: [PATCH] Add zero-overhead looping for xtensa backend

2014-10-24 Thread augustine.sterl...@gmail.com
On Thu, Oct 23, 2014 at 9:12 PM, Yangfei (Felix) felix.y...@huawei.com wrote: Here the key point is we need a general purpose register for the loop instruction. So the question to ask here is, How does this work today, without loop instructions? Somehow--even when it has been spilled--a branch

[PATCH] c++ify sreal

2014-10-24 Thread tsaunders
From: Trevor Saunders tsaund...@mozilla.com Hi, do $subject, and cleanup for always 64 bit hwi. bootstrapped + regtested x86_64-unknown-linux-gnu, ok? Trev gcc/ChangeLog: 2014-10-24 Trevor Saunders tsaund...@mozilla.com * ipa-inline.c (edge_badness): Adjust.

Re: [PATCH] Add zero-overhead looping for xtensa backend

2014-10-24 Thread Yangfei (Felix)
On Thu, Oct 23, 2014 at 9:12 PM, Yangfei (Felix) felix.y...@huawei.com wrote: Here the key point is we need a general purpose register for the loop instruction. So the question to ask here is, How does this work today, without loop instructions? Somehow--even when it has been

Re: [PATCH] Add zero-overhead looping for xtensa backend

2014-10-24 Thread augustine.sterl...@gmail.com
I mean without your patch at all. On Thu, Oct 23, 2014 at 11:30 PM, Yangfei (Felix) felix.y...@huawei.com wrote: On Thu, Oct 23, 2014 at 9:12 PM, Yangfei (Felix) felix.y...@huawei.com wrote: Here the key point is we need a general purpose register for the loop instruction. So the

Re: [PATCH] Add zero-overhead looping for xtensa backend

2014-10-24 Thread Yangfei (Felix)
1. The original xtensa port never generates loop instruction at all. 2. A port doesn't need to implement hwloop_pattern_reg hook if it has no zero-cost loop instruction. Is that clear? I mean without your patch at all. On Thu, Oct 23, 2014 at 11:30 PM, Yangfei (Felix)

Re: [PATCH] Add zero-overhead looping for xtensa backend

2014-10-24 Thread augustine.sterl...@gmail.com
On Thu, Oct 23, 2014 at 11:40 PM, Yangfei (Felix) felix.y...@huawei.com wrote: 1. The original xtensa port never generates loop instruction at all. 2. A port doesn't need to implement hwloop_pattern_reg hook if it has no zero-cost loop instruction. Is that clear? We are talking in circles.

[PATCH] Fix bootstrap/PR63632

2014-10-24 Thread Markus Trippelsdorf
r216566 (r216568 for 4.9 branch) added %{fno-lto} to LINK_COMMAND_SPEC. However the linker doesn't understand -fno-lto and errors out. This causes LTO/PGO bootstrap to fail, because -fno-lto is used during STAGEprofile. Fixed by filtering out -fno-lto in collect2.c. LTO/PGO bootstrapped and

Re: [PATCH] Add zero-overhead looping for xtensa backend

2014-10-24 Thread Yangfei (Felix)
Thanks for the explanation. I think I am clear about what you are thinking now. That's an interesting question. I am not sure about reason why GCC's reload cannot handle a doloop_end insn. I guess maybe the doloop_end pattern is special? I mean it's a branch insn in a parallel form. On

Re: [PATCH] Add zero-overhead looping for xtensa backend

2014-10-24 Thread Andrew Pinski
On Thu, Oct 23, 2014 at 11:51 PM, Yangfei (Felix) felix.y...@huawei.com wrote: Thanks for the explanation. I think I am clear about what you are thinking now. That's an interesting question. I am not sure about reason why GCC's reload cannot handle a doloop_end insn. I guess maybe the

Re: [PATCH 5/5] add libcc1

2014-10-24 Thread Phil Muldoon
On 10/10/14 22:58, Jeff Law wrote: On 10/09/14 03:07, Phil Muldoon wrote: Sorry for taking so long to reply. We've talked, on irc and elsewhere a little (some at the Cauldron too!). I think the consensus is as nobody has explicitly mentioned anything, this is OK to go in? Yes, please go

Re: [PATCH 5/5] add libcc1

2014-10-24 Thread Jakub Jelinek
On Fri, Oct 24, 2014 at 08:15:36AM +0100, Phil Muldoon wrote: On 10/10/14 22:58, Jeff Law wrote: On 10/09/14 03:07, Phil Muldoon wrote: Sorry for taking so long to reply. We've talked, on irc and elsewhere a little (some at the Cauldron too!). I think the consensus is as nobody has

Re: [PATCH] Fix genmatch linking

2014-10-24 Thread Rainer Orth
Richard Biener rguent...@suse.de writes: On Thu, 23 Oct 2014, Richard Biener wrote: Final try for today. And this may work as well and is slightly simpler. Index: gcc/Makefile.in === --- gcc/Makefile.in (revision 216590)

Re: [PATCH] Fix genmatch linking

2014-10-24 Thread Richard Biener
On Thu, 23 Oct 2014, Richard Sandiford wrote: Richard Biener rguent...@suse.de writes: This adds a libcpp host module without NLS and ICONV support and properly links genmatch against the build libcpp instead of the host one. Bootstrap running on x86_64-unknown-linux-gnu (stage1

Re: [PATCH 1/2, x86, PR63534] Fix darwin bootstrap

2014-10-24 Thread Uros Bizjak
On Thu, Oct 23, 2014 at 11:19 PM, Evgeny Stupachenko evstu...@gmail.com wrote: Linux x86 bootstrap/make check passed. Darwin x86 bootstrap passed configured with --with-cpu=intel --with-cpu=corei7 issue refers to PR63620. On Fri, Oct 17, 2014 at 6:08 PM, Evgeny Stupachenko evstu...@gmail.com

Re: PATCH: fix breakage from [PATCH] Fix genmatch linking

2014-10-24 Thread Richard Biener
On Fri, 24 Oct 2014, Hans-Peter Nilsson wrote: From: Richard Biener rguent...@suse.de Date: Thu, 23 Oct 2014 10:47:43 +0200 This adds a libcpp host module without NLS and ICONV support and properly links genmatch against the build libcpp instead of the host one. Bootstrap running

Re: [PATCH] Fix genmatch linking

2014-10-24 Thread Richard Biener
On Fri, 24 Oct 2014, Rainer Orth wrote: Richard Biener rguent...@suse.de writes: On Thu, 23 Oct 2014, Richard Biener wrote: Final try for today. And this may work as well and is slightly simpler. Index: gcc/Makefile.in

Re: [Patch ARM-AArch64/testsuite v3 00/21] Neon intrinsics executable tests

2014-10-24 Thread Marcus Shawcroft
On 21 October 2014 14:02, Christophe Lyon christophe.l...@linaro.org wrote: This patch series is an updated version of the series I sent here: https://gcc.gnu.org/ml/gcc-patches/2014-07/msg00022.html I addressed comments from Marcus and Richard, and decided to skip support for half-precision

[AARCH64, Question] Does AARCH64 GCC support long calls?

2014-10-24 Thread Yangfei (Felix)
Hi, I find that the -mlong-calls option is not there for AARCH64. So can this port generate long calls? Any plan on this option? I would like to have a try on this if it's missing :-) Thanks.

Re: [PATCHv4] Enable -fsanitize-recover for KASan

2014-10-24 Thread Yury Gribov
On 10/23/2014 03:10 PM, Andrey Ryabinin wrote: On 10/23/2014 02:38 PM, Jakub Jelinek wrote: On Thu, Oct 23, 2014 at 02:33:42PM +0400, Yury Gribov wrote: Actually this is a historical artifact. If inlining proves to be significantly faster, they may want to switch. Ok. So, at that point

Re: [PATCH PR63173] [AARCH64, NEON] Improve vld[234](q?)_dup intrinsics

2014-10-24 Thread Marcus Shawcroft
On 24 October 2014 03:21, Yangfei (Felix) felix.y...@huawei.com wrote: Thanks for the comments. I updated the patch with the intrinsic moved to its place. Attached please find the new version of the patch. OK for the trunk? Index: gcc/ChangeLog

Re: [PATCH] c++ify sreal

2014-10-24 Thread Richard Biener
On Fri, Oct 24, 2014 at 8:28 AM, tsaund...@mozilla.com wrote: From: Trevor Saunders tsaund...@mozilla.com Hi, do $subject, and cleanup for always 64 bit hwi. bootstrapped + regtested x86_64-unknown-linux-gnu, ok? Ok. Can you please replace remaining HOST_WIDE_INT vestiges in there with

[committed] MAINTAINERS: add myself to write-after-approval list.

2014-10-24 Thread Daniel Hellstrom
2014-10-24 Daniel Hellstrom dan...@gaisler.com * MAINTAINERS (write-after-approval): Add myself. Index: MAINTAINERS === --- MAINTAINERS(revision 216624) +++ MAINTAINERS(working copy) @@ -408,6 +408,7 @@ Pat Haugen

Re: [libgomp, libiberty, libobjc] Fix gnu11 fallout on Solaris 10+

2014-10-24 Thread Rainer Orth
Richard Henderson r...@redhat.com writes: On 10/22/2014 04:43 AM, Rainer Orth wrote: The gnu11 patch broke Solaris 10 and 11 bootstrap: sys/feature_test.h has /* * It is invalid to compile an XPG3, XPG4, XPG4v2, or XPG5 application * using c99. The same is true for POSIX.1-1990,

[PATCH][match-and-simplify] Fix failed init of force_no_side_effects

2014-10-24 Thread Richard Biener
Committed. Richard. 2014-10-24 Richard Biener rguent...@suse.de * genmatch.c (capture_info::capture_info): Initialize force_no_side_effects. Index: gcc/genmatch.c === --- gcc/genmatch.c (revision 216594)

[PATCH, alpha]: Fix c11 fallout in alpha testsuite

2014-10-24 Thread Uros Bizjak
2014-10-24 Uros Bizjak ubiz...@gmail.com * gcc.target/alpha/980217-1.c (main): Fix implicit int. * gcc.target/alpha/pr19518.c (_mm_setzero_si64): New function. Tested on alphaev68-linux-gnu and committed to mainline SVN. Uros. Index: gcc.target/alpha/980217-1.c

Re: avoid alignment of static variables affecting stack's

2014-10-24 Thread Jan Beulich
On 23.10.14 at 20:13, l...@redhat.com wrote: On 10/23/14 01:09, Jan Beulich wrote: On 23.10.14 at 08:50, ja...@redhat.com wrote: On Thu, Oct 23, 2014 at 07:30:27AM +0100, Jan Beulich wrote: Function (or more narrow) scope static variables (as well as others not placed on the stack) should

Re: Move loop peeling from RTL to gimple

2014-10-24 Thread Uros Bizjak
Hello! Hi, this is update of my 2013 update to 2012 patch to move rtl loop peeling to tree level. This is to expose optimization oppurtunities earlier. Incrementally I think I can also improve profiling to provide a histogram on loop iterations and get more sensible peeling decisions.

Re: avoid alignment of static variables affecting stack's

2014-10-24 Thread Richard Biener
On Fri, Oct 24, 2014 at 11:01 AM, Jan Beulich jbeul...@suse.com wrote: On 23.10.14 at 20:13, l...@redhat.com wrote: On 10/23/14 01:09, Jan Beulich wrote: On 23.10.14 at 08:50, ja...@redhat.com wrote: On Thu, Oct 23, 2014 at 07:30:27AM +0100, Jan Beulich wrote: Function (or more narrow) scope

Re: [PATCH,1/2] Extended if-conversion for loops marked with pragma omp simd.

2014-10-24 Thread Richard Biener
On Tue, Oct 21, 2014 at 4:34 PM, Yuri Rumyantsev ysrum...@gmail.com wrote: Richard, In my initial design I did such splitting but before start real if-conversion but I decided to not perform it since code size for if-converted loop is growing (number of phi nodes is increased). It is worth

Re: [PATCH] Simple improvement for predicate computation in if-convert phase.

2014-10-24 Thread Richard Biener
On Fri, Oct 17, 2014 at 3:08 PM, Yuri Rumyantsev ysrum...@gmail.com wrote: Jeff, I prepared another patch that includes test-case as you requested. Below are answers on your questions. First, for the benefit of anyone trying to understand what you're doing, defining what cd equivalent

Re: avoid alignment of static variables affecting stack's

2014-10-24 Thread Jan Beulich
On 24.10.14 at 11:10, richard.guent...@gmail.com wrote: On Fri, Oct 24, 2014 at 11:01 AM, Jan Beulich jbeul...@suse.com wrote: On 23.10.14 at 20:13, l...@redhat.com wrote: On 10/23/14 01:09, Jan Beulich wrote: On 23.10.14 at 08:50, ja...@redhat.com wrote: On Thu, Oct 23, 2014 at 07:30:27AM

Re: avoid alignment of static variables affecting stack's

2014-10-24 Thread Jakub Jelinek
On Fri, Oct 24, 2014 at 11:10:08AM +0200, Richard Biener wrote: For something in static storage, this seems OK. However, I think a hard register variable ought to be left alone -- even if we can't spill it to a stack slot today, there's a reasonable chance we might add that capability in

Re: [testsuite] Fix gcc.dg/ipa/ipa-icf-2[18].c on Solaris

2014-10-24 Thread Rainer Orth
Jan Hubicka hubi...@ucw.cz writes: I had a terrible time figuring out what march value i686-unknown-linux-gnu uses; only in gcc -v it's possible to see that it uses pentiumpro, unlike pentium4 on Solaris/x86, which explains the difference. What is the reason to use pentium4 for Solaris/x86?

Re: [match-and-simplify] add new lower function

2014-10-24 Thread Richard Biener
On Thu, Oct 23, 2014 at 11:22 AM, Prathamesh Kulkarni bilbotheelffri...@gmail.com wrote: Instead of iterating each time for calling lowering function (lower_opt_convert, etc.), add new lower function and pass lower_opt_convert, lower_for etc. to it as callback. I don't think this is more

Re: [PATCH] Fix and improve avx2 broadcasts (PR target/63594)

2014-10-24 Thread Rainer Orth
Rainer Orth r...@cebitec.uni-bielefeld.de writes: Jakub Jelinek ja...@redhat.com writes: On Thu, Oct 23, 2014 at 02:58:06PM +0200, Rainer Orth wrote: Unfortunately, I see some problems with those tests on Solaris: * On Solaris/x86, I get FAIL: gcc.dg/pr63594-2.c execution test for

[PATCH] Fix for PR63595

2014-10-24 Thread Martin Liška
Hello. Following patch contains addition of PHI result comparison in IPA ICF. Boostrap works on x86_64-linux-pc, no regression observed. Ready for trunk? Thanks, Martin gcc/testsuite/ChangeLog: 2014-10-24 Martin Liska mli...@suse.cz * gcc.dg/ipa/pr63595.c: New test. gcc/ChangeLog:

Re: [PATCHv4] Enable -fsanitize-recover for KASan

2014-10-24 Thread Dmitry Vyukov
On Fri, Oct 24, 2014 at 12:28 PM, Yury Gribov y.gri...@samsung.com wrote: On 10/23/2014 03:10 PM, Andrey Ryabinin wrote: On 10/23/2014 02:38 PM, Jakub Jelinek wrote: On Thu, Oct 23, 2014 at 02:33:42PM +0400, Yury Gribov wrote: Actually this is a historical artifact. If inlining proves to

Re: [PATCH] Fix genmatch linking

2014-10-24 Thread Richard Biener
On Fri, 24 Oct 2014, Richard Biener wrote: On Fri, 24 Oct 2014, Rainer Orth wrote: Richard Biener rguent...@suse.de writes: On Thu, 23 Oct 2014, Richard Biener wrote: Final try for today. And this may work as well and is slightly simpler. Index: gcc/Makefile.in

Re: [PATCHv4] Enable -fsanitize-recover for KASan

2014-10-24 Thread Jakub Jelinek
On Fri, Oct 24, 2014 at 01:44:27PM +0400, Dmitry Vyukov wrote: I am somewhat lost in this thread and probably missing something. But why do we need __asan_load (which is not noabort) at all? Outline instrumentation is non a default mode for both user-space asan and kasan (at least in the

Re: avoid alignment of static variables affecting stack's

2014-10-24 Thread Richard Biener
On Fri, Oct 24, 2014 at 11:18 AM, Jakub Jelinek ja...@redhat.com wrote: On Fri, Oct 24, 2014 at 11:10:08AM +0200, Richard Biener wrote: For something in static storage, this seems OK. However, I think a hard register variable ought to be left alone -- even if we can't spill it to a stack

Re: [PATCHv4] Enable -fsanitize-recover for KASan

2014-10-24 Thread Dmitry Vyukov
On Fri, Oct 24, 2014 at 1:50 PM, Jakub Jelinek ja...@redhat.com wrote: On Fri, Oct 24, 2014 at 01:44:27PM +0400, Dmitry Vyukov wrote: I am somewhat lost in this thread and probably missing something. But why do we need __asan_load (which is not noabort) at all? Outline instrumentation is non a

Re: [PATCHv4] Enable -fsanitize-recover for KASan

2014-10-24 Thread Jakub Jelinek
On Fri, Oct 24, 2014 at 11:50:58AM +0200, Jakub Jelinek wrote: On Fri, Oct 24, 2014 at 01:44:27PM +0400, Dmitry Vyukov wrote: I am somewhat lost in this thread and probably missing something. But why do we need __asan_load (which is not noabort) at all? Outline instrumentation is non a

Re: [PATCHv4] Enable -fsanitize-recover for KASan

2014-10-24 Thread Yury Gribov
On 10/24/2014 01:44 PM, Dmitry Vyukov wrote: On Fri, Oct 24, 2014 at 12:28 PM, Yury Gribov y.gri...@samsung.com wrote: On 10/23/2014 03:10 PM, Andrey Ryabinin wrote: On 10/23/2014 02:38 PM, Jakub Jelinek wrote: On Thu, Oct 23, 2014 at 02:33:42PM +0400, Yury Gribov wrote: Actually this is

Re: [PATCH] Fix for PR63595

2014-10-24 Thread Richard Biener
On Fri, Oct 24, 2014 at 11:35 AM, Martin Liška mli...@suse.cz wrote: Hello. Following patch contains addition of PHI result comparison in IPA ICF. Boostrap works on x86_64-linux-pc, no regression observed. Ready for trunk? Ok. Thanks, Richard. Thanks, Martin

Re: avoid alignment of static variables affecting stack's

2014-10-24 Thread Jan Beulich
On 24.10.14 at 11:52, richard.guent...@gmail.com wrote: On Fri, Oct 24, 2014 at 11:18 AM, Jakub Jelinek ja...@redhat.com wrote: On Fri, Oct 24, 2014 at 11:10:08AM +0200, Richard Biener wrote: For something in static storage, this seems OK. However, I think a hard register variable ought to

Re: [PATCHv4] Enable -fsanitize-recover for KASan

2014-10-24 Thread Dmitry Vyukov
On Fri, Oct 24, 2014 at 1:59 PM, Yury Gribov y.gri...@samsung.com wrote: On 10/24/2014 01:44 PM, Dmitry Vyukov wrote: On Fri, Oct 24, 2014 at 12:28 PM, Yury Gribov y.gri...@samsung.com wrote: On 10/23/2014 03:10 PM, Andrey Ryabinin wrote: On 10/23/2014 02:38 PM, Jakub Jelinek wrote: On

Re: [PATCH,1/2] Extended if-conversion for loops marked with pragma omp simd.

2014-10-24 Thread Yuri Rumyantsev
Richard, Patch containing new core related to extended predication is attached. Here is few comments which explain a main goal of design. 1. I don't want to insert any critical edge splitting since it may lead to less efficient binaries (I remember some performance issue when we designed lazy

Re: [PATCH 2/2] [AARCH64,NEON] Convert arm_neon.h to use new builtins for vld[234](q?)_lane_*

2014-10-24 Thread Marcus Shawcroft
On 23 October 2014 18:51, Charles Baylis charles.bay...@linaro.org wrote: Otherwise this and the previous 1/2 associated patch look good, can you respin with these tidy ups? OK for trunk? OK /Marcus

Re: [patch,avr] tweak sign extensions, take #2

2014-10-24 Thread Georg-Johann Lay
Am 10/23/2014 08:16 PM schrieb Denis Chertykov: This optimization makes most sign-extensions one instruction shorter in the case when the source register may be clobbered and the register numbers are different. Source and destination may overlap. Ok for trunk? Johann gcc/ *

Re: avoid alignment of static variables affecting stack's

2014-10-24 Thread Richard Biener
On Fri, Oct 24, 2014 at 12:12 PM, Jan Beulich jbeul...@suse.com wrote: On 24.10.14 at 11:52, richard.guent...@gmail.com wrote: On Fri, Oct 24, 2014 at 11:18 AM, Jakub Jelinek ja...@redhat.com wrote: On Fri, Oct 24, 2014 at 11:10:08AM +0200, Richard Biener wrote: For something in static

Re: [PATCH][AArch64] Cleanup logic around aarch64_final_prescan

2014-10-24 Thread Marcus Shawcroft
On 22 October 2014 14:57, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: 2014-10-13 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/aarch64/aarch64.h (ADJUST_INSN_LENGTH): Wrap definition in do while (0). * config/aarch64/aarch64.c (is_mem_p): Delete. (is_memory_op): Rename

Re: [PATCH] PR58867 ASan and UBSan tests not run for installed testing.

2014-10-24 Thread Eric Botcazou
some time ago, Andrew wrote a patch that fixes PR58867 (http://patchwork.ozlabs.org/patch/286866/), but for some reasons it wasn't committed to trunk. This is resurrected Andrew's patch, extended to support Tsan testsuite. This patch broke --disable-libsanitizer though, i.e. you now get

Re: [PATCH][AArch64][4.8] Backport Cortex-A53 erratum 835769 workaround

2014-10-24 Thread Marcus Shawcroft
On 17 October 2014 16:55, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: Hi all, This is the 4.8 backport of the Cortex-A53 erratum 835769 workaround. 4.8 doesn't have rtx_insns and the type attributes are different. Other than that there's not much different from the trunk version.

[PATCH][3,4,5/n] Merge from match-and-simplify, fold, fold_stmt and first patterns

2014-10-24 Thread Richard Biener
This combines the already posted 3/n (first simple patterns), 4/n (hook into fold-const.c) and not yet posted 5/n (hook into fold_stmt). Over the first posting this also contains recent improvements to the generator from the branch regarding to TREE_SIDE_EFFECTS and NON_LVALUE_EXPR handling.

Re: [PATCH][AArch64][4.8] Add --enable-fix-cortex-a53-835769 configure option

2014-10-24 Thread Marcus Shawcroft
On 17 October 2014 16:55, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: Hi all, This is the 4.8 backport of the configure option --enable-fix-cortex-a53-835769 to enable the workaround for the Cortex-A53 erratum 835769 by default. The patch is very similar to the trunk version, just some

Re: [PATCH] Fix genmatch linking

2014-10-24 Thread Rainer Orth
Richard Biener rguent...@suse.de writes: Dominique reported that this fails for system libiconv but built libintl. Which might be fixed by the following. Does that still work for you? It does: an i386-pc-solaris2.10 bootstrap has finished by now and make check is running. Rainer --

Re: [PATCH][AArch64] LINK_SPEC changes for Cortex-A53 erratum 835769 workaround

2014-10-24 Thread Marcus Shawcroft
On 22 October 2014 15:20, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: Hi all, This patch contains the LINK_SPEC changes required to pass on the linker option --fix-cortex-a53-835769 when compiling with -mfix-cortex-a53-835769 (or by default when configured with

[match-and-simplify] Merge from trunk

2014-10-24 Thread Richard Biener
2014-10-24 Richard Biener rguent...@suse.de Merge from trunk r216543 through r216631. Brings back second merge piece.

Re: [PATCH][AArch64] LINK_SPEC changes for Cortex-A53 erratum 835769 workaround

2014-10-24 Thread Jakub Jelinek
On Fri, Oct 24, 2014 at 12:04:52PM +0100, Marcus Shawcroft wrote: On 22 October 2014 15:20, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: Hi all, This patch contains the LINK_SPEC changes required to pass on the linker option --fix-cortex-a53-835769 when compiling with

Re: [PATCH] Fix genmatch linking

2014-10-24 Thread Richard Biener
On Fri, 24 Oct 2014, Rainer Orth wrote: Richard Biener rguent...@suse.de writes: Dominique reported that this fails for system libiconv but built libintl. Which might be fixed by the following. Does that still work for you? It does: an i386-pc-solaris2.10 bootstrap has finished by

Re: [PATCH][3,4,5/n] Merge from match-and-simplify, fold, fold_stmt and first patterns

2014-10-24 Thread Marc Glisse
+ /* Same applies to modulo operations, but fold is inconsistent here +and simplifies 0 % x to 0, only preserving literal 0 % 0. */ + (for op (ceil_mod floor_mod round_mod trunc_mod) + /* 0 % X is always zero. */ + (simplify + (trunc_mod integer_zerop@0 @1) + /* But not for 0 % 0 so

Re: [PATCH][3,4,5/n] Merge from match-and-simplify, fold, fold_stmt and first patterns

2014-10-24 Thread Richard Biener
On Fri, 24 Oct 2014, Marc Glisse wrote: + /* Same applies to modulo operations, but fold is inconsistent here +and simplifies 0 % x to 0, only preserving literal 0 % 0. */ + (for op (ceil_mod floor_mod round_mod trunc_mod) + /* 0 % X is always zero. */ + (simplify +

[PATCH][ARM] revert changes on check_effective_target_arm_*_ok

2014-10-24 Thread Jiong Wang
we should not add explicit declaration there. arm_neon.h contains those prototype already. they will be available if the compiler configuration is with related builtin predefine, for example __ARM_FEATURE_CRYPTO. so, actually, if there is any warning when compile these test programs, they are

[PATCH][ARM] gnu11 cleanup for aapcs testcases

2014-10-24 Thread Jiong Wang
a furhter cleanup under aapcs sub-directory. ok for trunk? gcc/testsuite/ * gcc.target/arm/aapcs/abitest.h: Declare memcpy. diff --git a/gcc/testsuite/gcc.target/arm/aapcs/abitest.h b/gcc/testsuite/gcc.target/arm/aapcs/abitest.h index 06a92c3..7bce58b 100644 ---

Re: [PATCH][ARM] gnu11 cleanup for aapcs testcases

2014-10-24 Thread Marek Polacek
On Fri, Oct 24, 2014 at 12:48:24PM +0100, Jiong Wang wrote: a furhter cleanup under aapcs sub-directory. ok for trunk? gcc/testsuite/ * gcc.target/arm/aapcs/abitest.h: Declare memcpy. diff --git a/gcc/testsuite/gcc.target/arm/aapcs/abitest.h

Re: [PATCH] Relax check against commuting XOR and ASHIFTRT in combine.c

2014-10-24 Thread Alan Lawrence
Rainer Orth wrote: However, as a quick first step, does adding the ilp32 / lp64 (and keeping the architectures list for now) solve the immediate problem? Patch attached, OK for trunk? No, as I said this is wrong for biarch targets like sparc and i386. When you say no this does not solve

[PATCH v2 0-6/11] Fix PR/61114, make direct vector reductions endianness-neutral

2014-10-24 Thread Alan Lawrence
This is the first half of my previous patch series (https://gcc.gnu.org/ml/gcc-patches/2014-09/msg01456.html), that is the part making the REDUC_..._EXPR tree codes endian-neutral, and adding a new reduce-to-scalar optab in place of the endianness-dependent reduc_[us](plus|min|max)_optab.

[PATCH 7/11][ARM] Migrate to new reduc_plus_scal_optab

2014-10-24 Thread Alan Lawrence
This migrates ARM from reduc_splus_optab and reduc_uplus optab to a single reduc_plus_optab. Tested, in combination with next patch: bootstrap on arm-none-linux-gnueabihf cross-tested check-gcc on arm-none-eabi. gcc/ChangeLog: config/arm/neon.md (reduc_plus_*): Rename to...

Re: [PATCH][ARM] revert changes on check_effective_target_arm_*_ok

2014-10-24 Thread Ramana Radhakrishnan
On Fri, Oct 24, 2014 at 12:47 PM, Jiong Wang jiong.w...@arm.com wrote: we should not add explicit declaration there. arm_neon.h contains those prototype already. they will be available if the compiler configuration is with related builtin predefine, for example __ARM_FEATURE_CRYPTO. so,

[PATCH 8/11][ARM] Migrate to new reduc_[us](min|max)_scal_optab

2014-10-24 Thread Alan Lawrence
Similarly to last patch. Tested, in combination with previous patch: bootstrap on arm-none-linux-gnueabihf cross-tested check-gcc on arm-none-eabi. gcc/ChangeLog: config/arm/neon.md (reduc_smin_mode *2): Rename to... (reduc_smin_scal_mode *2): ...this; extract scalar result.

Re: [PATCH v2 0-6/11] Fix PR/61114, make direct vector reductions endianness-neutral

2014-10-24 Thread Richard Biener
On Fri, 24 Oct 2014, Alan Lawrence wrote: This is the first half of my previous patch series (https://gcc.gnu.org/ml/gcc-patches/2014-09/msg01456.html), that is the part making the REDUC_..._EXPR tree codes endian-neutral, and adding a new reduce-to-scalar optab in place of the

[PATCH 9/11][i386] Migrate reduction optabs to reduc_..._scal

2014-10-24 Thread Alan Lawrence
Bootstrapped and check-gcc on x86_64-none-linux-gnu. gcc/ChangeLog: * config/i386/i386.c (ix86_expand_reduc): Extract result into scalar. * config/i386/sse.md (reduc_splus_v8df, reduc_code_mode * 3, reduc_umin_v8hi): Rename to... (reduc_plus_scal_v8df,

[PATCH] c11-atomic-exec-5: Avoid dead code where LDBL_MANT_DIG is 106

2014-10-24 Thread Maciej W. Rozycki
Hi, Commit 216437 missed a part of Adhemerval's original change that made `long_double_add_overflow', `complex_long_double_add_overflow', `long_double_sub_overflow' and `complex_long_double_sub_overflow' tests consistently defined only if called. These tests are now only made under the

[Protopatch 11/11][IA64] Migrate to reduc_(plus|min|max)_scal_v2df optab

2014-10-24 Thread Alan Lawrence
This is an attempt to migrate IA64 to the newer optabs, however, I found none of the tests in gcc.dg/vect seemed to touch any of the affected patternsso this is only really tested by building a stage-1 compiler. gcc/ChangeLog: * config/ia64/vect.md (reduc_splus_v2sf): Rename to...

[PATCH 10/11][RS6000] Migrate reduction optabs to reduc_..._scal

2014-10-24 Thread Alan Lawrence
This migrates the reduction patterns in altivec.md and vector.md to the new names. I've not touched paired.md as I wasn't really sure how to fix that (how do I vec_extractv2sf ?), moreover the testing I did didn't seem to exercise any of those patterns (iow: I'm not sure what would be an

Re: [PATCH 10/11][RS6000] Migrate reduction optabs to reduc_..._scal

2014-10-24 Thread Alan Lawrence
Ooops, attached.commit e48d59399722ce8316d4b1b4f28b40d87b1193fa Author: Alan Lawrence alan.lawre...@arm.com Date: Tue Oct 7 15:28:47 2014 +0100 PowerPC v2 (but not paired.md) diff --git a/gcc/config/rs6000/altivec.md b/gcc/config/rs6000/altivec.md index 02ea142..92bb5d0 100644 ---

Re: [Protopatch 11/11][IA64] Migrate to reduc_(plus|min|max)_scal_v2df optab

2014-10-24 Thread Alan Lawrence
Ooops, attached.commit 56296417b9f6795e541b1101dce6e6ac1789de9a Author: Alan Lawrence alan.lawre...@arm.com Date: Wed Oct 8 15:58:27 2014 +0100 IA64 (?!) diff --git a/gcc/config/ia64/vect.md b/gcc/config/ia64/vect.md index e3ce292..45f4156 100644 --- a/gcc/config/ia64/vect.md +++

[v3] Minor atomic tweaks

2014-10-24 Thread Paolo Carlini
Hi, tested x86_64-linux. Thanks, Paolo. /// 2014-10-24 Paolo Carlini paolo.carl...@oracle.com * include/bits/atomic_base.h: Avoid including stdbool.h. * include/std/atomic: When __cplusplus 201103L skip the rest of the header. *

Re: [PATCH] Relax check against commuting XOR and ASHIFTRT in combine.c

2014-10-24 Thread Rainer Orth
Alan Lawrence alan.lawre...@arm.com writes: Rainer Orth wrote: However, as a quick first step, does adding the ilp32 / lp64 (and keeping the architectures list for now) solve the immediate problem? Patch attached, OK for trunk? No, as I said this is wrong for biarch targets like sparc and

[PATCH][6/n] Merge from match-and-simplify, make forwprop fold all stmts

2014-10-24 Thread Richard Biener
This patch makes GIMPLE forwprop fold all statements, following single-use SSA edges only (as suggested by Jeff and certainly how this will regress the least until we replace manual simplification code that does not restrict itself this way). forwprop is run up to 4 times at the moment (once

Re: Patch committed: Don't define TARGET_HAS_F_SETLKW

2014-10-24 Thread Andreas Schwab
Ian Taylor i...@golang.org writes: 2014-10-23 Ian Lance Taylor i...@google.com * config/mep/mep.h (TARGET_HAS_F_SETLKW): Don't define. s/define/undefine/ Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5 And now for

[PATCH] Fix modulo patterns in match.pd

2014-10-24 Thread Richard Biener
As noted by Marc I forgot to actually utilize the iterator variable. Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Richard. PS: How do we want to refer to patterns in ChangeLogs? 2014-10-24 Richard Biener rguent...@suse.de * match.pd (0 % X): Properly use the

Re: [PATCH] Fix modulo patterns in match.pd

2014-10-24 Thread Jakub Jelinek
On Fri, Oct 24, 2014 at 03:27:19PM +0200, Richard Biener wrote: As noted by Marc I forgot to actually utilize the iterator variable. Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Richard. PS: How do we want to refer to patterns in ChangeLogs? Perhaps the syntax should

Re: [PATCH] Fix modulo patterns in match.pd

2014-10-24 Thread Richard Biener
On Fri, 24 Oct 2014, Jakub Jelinek wrote: On Fri, Oct 24, 2014 at 03:27:19PM +0200, Richard Biener wrote: As noted by Marc I forgot to actually utilize the iterator variable. Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Richard. PS: How do we want to refer to

[PATCHv5][Kasan] Allow to override Asan shadow offset from command line

2014-10-24 Thread Yury Gribov
Hi all, On 10/17/2014 11:53 AM, Yury Gribov wrote: On 09/29/2014 09:21 PM, Yury Gribov wrote: Kasan developers has asked for an option to override offset of Asan shadow memory region. This should simplify experimenting with memory layouts on 64-bit architectures. New patch which checks that

Re: [patch,avr] tweak sign extensions, take #2

2014-10-24 Thread Denis Chertykov
2014-10-24 14:37 GMT+04:00 Georg-Johann Lay a...@gjlay.de: Am 10/23/2014 08:16 PM schrieb Denis Chertykov: This optimization makes most sign-extensions one instruction shorter in the case when the source register may be clobbered and the register numbers are different. Source and

Re: [PATCHv5][Kasan] Allow to override Asan shadow offset from command line

2014-10-24 Thread Jakub Jelinek
On Fri, Oct 24, 2014 at 05:56:37PM +0400, Yury Gribov wrote: From 1882c41de6c8ae53b7e199b3cc655b6f4b31e8fb Mon Sep 17 00:00:00 2001 From: Yury Gribov y.gri...@samsung.com Date: Thu, 16 Oct 2014 18:31:10 +0400 Subject: [PATCH 1/2] Add strtoll and strtoull to libiberty. 2014-10-20 Yury

Re: [PATCH][optabs] PR63442 libgcc_cmp_return_mode not always return word_mode

2014-10-24 Thread Jiong Wang
ping~ thanks. Regards, Jiong On 17/10/14 13:04, Jiong Wang wrote: the cause should be one minor bug in prepare_cmp_insn. the last mode parameter pmode of prepare_cmp_insn should match the mode of the first parameter x, while during the recursive call of prepare_cmp_insn, x is with mode of

Re: [PATCH, x86, 63534] Fix '-p' profile for 32 bit PIC mode

2014-10-24 Thread Evgeny Stupachenko
The following patch align stack for mcount and there should be no problems with unwind as ix86_frame_pointer_required is true when crtl-profile is true and flag_fentry is false (we call mcount after function prolog). When flag_fentry is true it is set to false in 32bit PIC mode: if

Unifying std::atomic_int and std::atomicint

2014-10-24 Thread Jonathan Wakely
Our atomic was implemented (by Benjamin IIRC) based on an early C++0x draft when the spec was still trying to be valid for both C and C++. Part of the C compatibility aspect was that std::atomic_int is allowed to be either a typedef for std::atomicint or a base class of it, so that a C library

[PATCH] Fix typedef-name printing (PR c/56980)

2014-10-24 Thread Marek Polacek
Our current C pretty printer output sometimes looks a bit goofy: expected ‘enum F *’ but argument is of type ‘enum F *’. It's because it always prints struct/union/enum even though the type is a typedef name. This patch ought to fix this. We've got a bunch of reports about this over the years...

Re: [PATCH 2/n] OpenMP 4.0 offloading infrastructure: LTO streaming

2014-10-24 Thread Ilya Verbin
On 20 Oct 15:19, Ilya Verbin wrote: On 15 Oct 16:23, Richard Biener wrote: +static bool +initialize_offload (void) +{ + bool have_offload = false; + struct cgraph_node *node; + struct varpool_node *vnode; + + FOR_EACH_DEFINED_FUNCTION (node) +if

Re: [PATCH 2/n] OpenMP 4.0 offloading infrastructure: LTO streaming

2014-10-24 Thread Jakub Jelinek
On Fri, Oct 24, 2014 at 06:16:01PM +0400, Ilya Verbin wrote: We have to set the global have_offload flag in few places in omp-low.c and in FE (c/c-decl.c:c_decl_attributes, fortran/trans-common.c:build_common_decl, fortran/trans-decl.c:add_attributes_to_decl). This way looks for me a bit

Re: [PATCH, x86, 63534] Fix '-p' profile for 32 bit PIC mode

2014-10-24 Thread Jakub Jelinek
On Fri, Oct 24, 2014 at 06:12:15PM +0400, Evgeny Stupachenko wrote: The following patch align stack for mcount and there should be no problems with unwind as ix86_frame_pointer_required is true when crtl-profile is true and flag_fentry is false (we call mcount after function prolog). When

Re: [PATCH 3/4] Add libgomp plugin for Intel MIC

2014-10-24 Thread Jakub Jelinek
On Thu, Oct 23, 2014 at 07:41:12PM +0400, Ilya Verbin wrote: malloc can fail, SIGSEGV in response to that is not desirable. Can't you fallback to alloca, or use just alloca, or use alloca with malloc fallback? I replaced it with alloca. There is a risk if a suid or otherwise priviledge

[PATCH v2] avoid alignment of static variables affecting stack's

2014-10-24 Thread Jan Beulich
Function (or more narrow) scope static variables (as well as others not placed on the stack) should also not have any effect on the stack alignment. I noticed the issue first with Linux'es dynamic_pr_debug() construct using an 8-byte aligned sub-file-scope local variable. According to my checking

Re: [PATCH 3/4] Add libgomp plugin for Intel MIC

2014-10-24 Thread Ilya Verbin
On 24 Oct 16:35, Jakub Jelinek wrote: On Thu, Oct 23, 2014 at 07:41:12PM +0400, Ilya Verbin wrote: malloc can fail, SIGSEGV in response to that is not desirable. Can't you fallback to alloca, or use just alloca, or use alloca with malloc fallback? I replaced it with alloca. There

Re: [PATCH 2/2] [AARCH64,NEON] Convert arm_neon.h to use new builtins for vld[234](q?)_lane_*

2014-10-24 Thread Charles Baylis
On 24 October 2014 11:23, Marcus Shawcroft marcus.shawcr...@gmail.com wrote: On 23 October 2014 18:51, Charles Baylis charles.bay...@linaro.org wrote: Otherwise this and the previous 1/2 associated patch look good, can you respin with these tidy ups? OK for trunk? OK /Marcus Committed to

RE: [PATCH v2 0-6/11] Fix PR/61114, make direct vector reductions endianness-neutral

2014-10-24 Thread Matthew Fortune
Alan Lawrence alan.lawre...@arm.com writes: Patches 7-11 migrate migrate ARM, x86, IA64 (I think), and mostly PowerPC, to the new reduc_(plus|[us](min|max))_scal_optab. I have not managed to work out how to do the same for MIPS (specifically what I need to add to mips_expand_vec_reduc), and

  1   2   >