Fix 63615 - FAIL: gcc.target/i386/addr-sel-1.c

PR 63615 was caused by r21642 fixing the accounting of "n_constants" in simplify_plus_minus. Previously, any expression handled by simplify_plus_minus that expanded to more than two elements and contained at least one constant would have resulted in "n_constants" being larger than one, even if it

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

Hi, This patch converts the vld[234](q?)_dup intrinsics to use builtin functions instead of the previous inline assembly syntax. It can fix the performance issue on PR63173. Reg-tested with aarch64-linux-gnu-gcc on qemu. OK for the trunk? Thanks Index: gcc/ChangeLog ===

Re: avoid alignment of static variables affecting stack's

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 also not have any effect on the stack > alignment. I noticed the issue first with Linux'es dynamic_pr_debug() > construct using an 8-b

avoid alignment of static variables affecting stack's

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

Patch RFA: Top-level configure patch: disable go on systems where it doesn't work

This patch to the top level GCC configure script disables the go languages on some systems where it is known to not work. Bootstrapped on x86_64-unknown-gnu-linux. OK for mainline? Ian 2014-10-22 Ian Lance Taylor * configure.ac: Disable the Go frontend on systems where it is known to not wo

[PATCH] New debug parameter sched-pressure-factor

Hi, For benchmarking and tuning of register-pressure scheduling I've made the following simple patch to adjust effective number of registers that scheduler has at its disposal. Do you think this would be valuable for others to investigate performance problems or tune register-pressure scheduli

Patch committed: Add missing ChangeLog entry

I committed this patch to the top-level ChangeLog to add a missing entry. I forgot to commit the ChangeLog patch with the real change over a year ago, and just came across it. Ian Index: ChangeLog === --- ChangeLog (revision 216522

Re: [PATCH 3/8] Remove cached_first_cycle_multipass_dfa_lookahead and cached_issue_rate

On Oct 22, 2014, at 4:53 AM, Vladimir Makarov wrote: > On 10/20/2014 11:16 PM, Maxim Kuvyrkov wrote: >> Hi, >> >> This patch cleans up (removes) cached_first_cycle_multipass_dfa_lookahead >> and cached_issue_rate. >> >> These seem to be an artifact from the scheduler refactoring 10+ years ago.

Re: [Patch, libstdc++/63497] Avoid dereferencing invalid iterator in regex_executor

On Wed, Oct 22, 2014 at 4:04 PM, Jonathan Wakely wrote: > OK to commit, thanks! Committed. Thank you too! -- Regards, Tim Shen

Re: [PATCH] Account for prologue spills in reg_pressure scheduling

On Oct 23, 2014, at 3:45 AM, Vladimir Makarov wrote: ... > Richard proposed 2-loops solution instead of 3. So I'd definitely prefer his. > > The patch is ok with Richard's 2-loops proposal. Thanks, Maxim. As I wrote > the heuristic is an interesting one. This is the patch I'm going to commi

Optimize powerpc*-*-linux* 32-bit classic hard/soft float hardfp/soft-fp use

Continuing the cleanups of libgcc soft-fp configuration for powerpc*-*-linux* in preparation for implementing TARGET_ATOMIC_ASSIGN_EXPAND_FENV for soft-float and e500, this patch optimizes the choice of which functions to build for the 32-bit classic hard-float and soft-float cases. (e500 will be

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

On 10/22/14 15:40, Jakub Jelinek wrote: diff --git a/gcc/dce.c b/gcc/dce.c index 5b7d36e..a52a59c 100644 --- a/gcc/dce.c +++ b/gcc/dce.c @@ -127,6 +127,10 @@ deletable_insn_p (rtx_insn *insn, bool fast, bitmap arg_stores) if (HARD_REGISTER_NUM_P (DF_REF_REGNO (def)) && global_reg

Re: [PATCH 6/8] Handle SCRATCH in decompose_address

On 10/22/14 17:01, Maxim Kuvyrkov wrote: On Oct 23, 2014, at 9:02 AM, Jeff Law wrote: On 10/20/14 21:35, Maxim Kuvyrkov wrote: Hi, This patch is a simple fix to allow decompose_address to handle SCRATCH'es during 2nd scheduler pass. This patch is a prerequisite for a scheduler improvement th

Re: sort_heap complexity guarantee

Hi, On 10/22/2014 11:05 PM, François Dumont wrote: + VERIFY( counter_type::less_compare_count <= 2.0 * std::log2(nb_values) ); Nit: log2 isn't in C89, thus we shouldn't use it unconditionally, ie, if the test isn't guarded by { dg-require-cmath "" }. Thus, either the latter, or just express l

Re: sort_heap complexity guarantee

On 22/10/14 23:05 +0200, François Dumont wrote: Then I think we need this patch which also fix other issues. 2014-10-22 François Dumont * testsuite/25_algorithms/make_heap/complexity.cc: Add missing test variable. * testsuite/25_algorithms/sort_heap/complexity.cc: Likewise and use

Re: [Patch, libstdc++/63497] Avoid dereferencing invalid iterator in regex_executor

On 22/10/14 14:53 -0700, Tim Shen wrote: On Wed, Oct 22, 2014 at 7:34 AM, Jonathan Wakely wrote: Only if it's likely to catch problems in future. If you'd be putting it in only to make a testcase fail then it's not worth it. No, since I think with this patch _M_current is clearly valid when b

Re: [PATCH 6/8] Handle SCRATCH in decompose_address

On Oct 23, 2014, at 9:02 AM, Jeff Law wrote: > On 10/20/14 21:35, Maxim Kuvyrkov wrote: >> Hi, >> >> This patch is a simple fix to allow decompose_address to handle >> SCRATCH'es during 2nd scheduler pass. This patch is a prerequisite >> for a scheduler improvement that relies on decompose_addre

Re: [PATCH 5/8] Enable max_issue for AArch32 and AArch64

On Oct 23, 2014, at 11:42 AM, Sebastian Pop wrote: > Sebastian Pop wrote: >> Maxim Kuvyrkov wrote: >>> This change requires benchmarking, which I can't easily do at the moment. I >>> would appreciate any benchmarking results that you can share. >> >> I will run my testsuite on aarch64. >> Do yo

Re: [PATCH 5/8] Enable max_issue for AArch32 and AArch64

Sebastian Pop wrote: > Maxim Kuvyrkov wrote: > > This change requires benchmarking, which I can't easily do at the moment. I > > would appreciate any benchmarking results that you can share. > > I will run my testsuite on aarch64. > Do you need the perf for all the previous patches 1 to 5 togethe

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

>But here, after reload completes, there shouldn't be pseudos in the IL, so >the condition should not trigger anymore. At that point we'll just exit the condition earlier with !reload_completed. On Thu, Oct 23, 2014 at 1:40 AM, Jakub Jelinek wrote: > On Wed, Oct 22, 2014 at 03:34:38PM -0600, Jef

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

>What is ia32 specific on the test? Just the fact the test fails on ia32 now, for 64 bit the test is very simple, however we can test it as well. Full patch with testcase: 2014-10-22 Evgeny Stupachenko PR target/63534 PR target/63618 * cse.c (delete_trivially_dead_insns

Re: [Patch, libstdc++/63497] Avoid dereferencing invalid iterator in regex_executor

On Wed, Oct 22, 2014 at 7:34 AM, Jonathan Wakely wrote: > Only if it's likely to catch problems in future. If you'd be putting > it in only to make a testcase fail then it's not worth it. No, since I think with this patch _M_current is clearly valid when being dereferenced. > You could even simp

Re: [ARM] Fix CLZ_DEFINED_VALUE_AT_ZERO for vector modes

Patch that removes extraneous comment attached. The CLZ_DEFINED_VALUE_AT_ZERO macro is hard coded to return 32. For the vector intrinsic vclz this is incorrect and should return the value eight. The CTZ_DEFINED_VALUE_AT_ZERO has the same issue. Tested on arm-linux-gnueabihf, arm-linux-gnueabi.

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

On Wed, Oct 22, 2014 at 03:34:38PM -0600, Jeff Law wrote: > >--- a/gcc/cse.c > >+++ b/gcc/cse.c > >@@ -6953,6 +6953,11 @@ delete_trivially_dead_insns (rtx_insn *insns, int > >nreg) > >/* If no debug insns can be present, COUNTS is just an array > > which counts how many times each

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

On 10/17/14 08:16, Evgeny Stupachenko wrote: Hi, Some instructions (like one in PR63534) could have hidden use of PIC register. Therefore we need to leave SET_GOT not deleted till reload completed. The patch prevents SET_GOT from deleting while PIC register is pseudo. Is it ok? ChangeLog: 201

Re: [PATCH] Add missing requirement to crossmodule-indircall-1a.c

On 10/21/14 12:21, jb...@gmx.de wrote: "Jeff Law" : On 10/21/14 16:13, Haswell wrote: The additional source must have the same requirement crossmodule-indircall-1.c has. * crossmodule-indircall-1a.c: Add missing requirement. Why? When used by crossmodule-indircall-1.c we'll have alr

Re: [PATCH 3/5] IPA ICF pass

PR 63574 ICE building libjava (segfault) on arm-linux-gnueabihf is caused by this commit. from the backtrace, the ICF pass is trying to compare two label tree node without type info. while looks like "compare_operand" expect the type info always be not empty before invoking "func_checker::compati

Re: [Patch, testsuite] clear wrap_compile_flags before setting it

On 10/17/14 15:05, Christophe Lyon wrote: Hi, In testsuite/lib/wrapper.exp's ${tool}_maybe_build_wrapper, we call set_currtarget_info wrap_compile_flags to override wrap_compile_flags with additional flags when compiling testglue.c. However, dejagnu's set_currtarget_info {entry value} will writ

Re: The nvptx port [11/11] More tools.

On 10/22/2014 10:31 PM, Jeff Law wrote: These tools currently require GNU extensions - something I probably ought to fix if we decide to add them to the gcc build itself. Would these be more appropriate in binutils? I don't think so, given that we don't need any piece of regular binutils. The

Re: sort_heap complexity guarantee

Then I think we need this patch which also fix other issues. 2014-10-22 François Dumont * testsuite/25_algorithms/make_heap/complexity.cc: Add missing test variable. * testsuite/25_algorithms/sort_heap/complexity.cc: Likewise and use log2. * testsuite/25_algorithms/pop_hea

Re: [PATCH][0/n] Merge from match-and-simplify

On 10/20/14 05:42, Richard Biener wrote: That was a conscious decision and the idea was that the caller should do this via its lattice valueization function which could look like tree valueize (tree t) { if (TREE_CODE (t) == SSA_NAME && !has_single_use (t)) return NULL_TREE; re

Re: The nvptx port [11/11] More tools.

On 10/20/14 08:48, Bernd Schmidt wrote: This is a "bonus" optional patch which adds ar, ranlib, as and ld to the ptx port. This is not proper binutils; ar and ranlib are just linked to the host versions, and the other two tools have the following functions: * nvptx-as is required to convert the

Re: [PATCH 6/8] Handle SCRATCH in decompose_address

On 10/20/14 21:35, Maxim Kuvyrkov wrote: Hi, This patch is a simple fix to allow decompose_address to handle SCRATCH'es during 2nd scheduler pass. This patch is a prerequisite for a scheduler improvement that relies on decompose_address to parse insns. Bootstrapped and regtested on x86_64-linux

Re: [build] Link genmatch with $(LIBINTL)

On Wed, 22 Oct 2014, Richard Biener wrote: > On October 22, 2014 7:19:33 PM CEST, "Joseph S. Myers" > wrote: > >On Wed, 22 Oct 2014, Rainer Orth wrote: > > > >> 2014-10-22 Rainer Orth > >> > >>* Makefile.in (build/genmatch(build_exeext)): Add $(LIBINTL) to > >>BUILD_LIBS. > >>Add

Re: [PATCH 2/4] Add liboffloadmic

On 22 Oct 10:54, Jakub Jelinek wrote: > On Tue, Oct 21, 2014 at 09:20:34PM +0400, Ilya Verbin wrote: > > This patch contains liboffloadmic library. > > > > It is used by ICC for offloading. The sources are imported from upstream > > ( https://www.openmprtl.org/sites/default/files/liboffload_oss.t

Re: [build] Link genmatch with $(LIBINTL)

On October 22, 2014 7:19:33 PM CEST, "Joseph S. Myers" wrote: >On Wed, 22 Oct 2014, Rainer Orth wrote: > >> 2014-10-22 Rainer Orth >> >> * Makefile.in (build/genmatch(build_exeext)): Add $(LIBINTL) to >> BUILD_LIBS. >> Add $(LIBINTL_DEP) dependency. > >No, this doesn't look rig

Re: [build] Link genmatch with $(LIBINTL)

On Wed, Oct 22, 2014 at 1:31 PM, Richard Biener wrote: > On October 22, 2014 7:13:41 PM CEST, David Edelsohn wrote: >>I reported the same problem in a separate thread. >> >>On AIX it needs LIBINTL and LIBICONV. > > I wonder what ends up linking those with Linux? Or is all the features > availabl

Re: [PATCH][match-and-simplify] Hande side-effects in GENERIC

On October 22, 2014 5:08:50 PM CEST, Jakub Jelinek wrote: >On Wed, Oct 22, 2014 at 04:20:09PM +0200, Richard Biener wrote: >> 2014-10-22 Richard Biener >> >> * genmatch.c (count_captures): New function. >> (dt_simplify::gen): Handle preserving side-effects for >> GENERIC code ge

Re: [PATCH 1/4] Add mkoffload for Intel MIC

On 22 Oct 09:57, Jakub Jelinek wrote: > On Wed, Oct 22, 2014 at 02:30:44AM +0400, Ilya Verbin wrote: > > This mkoffload is expected to be built only with the offload compiler, > > which is expected to be configured with > > '--enable-as-accelerator-for=... --host=x86_64-*-linux-gnu > > --target=x86

Re: Extract and insert merging patch

On Wed, 22 Oct 2014, Petr Murzin wrote: + && UINTVAL (op2) == 1 << UINTVAL (tem)) With modes like V64QI around, it is better to replace 1 with HOST_WIDE_INT_1U, though we are not consistent about it. -- Marc Glisse

Re: The nvptx port [8/11+] Write undefined decls.

On 10/21/14 16:15, Bernd Schmidt wrote: On 10/22/2014 12:05 AM, Jeff Law wrote: On 10/20/14 14:30, Bernd Schmidt wrote: ptx assembly requires that declarations are written for undefined variables. This adds that functionality. Does this need to happen at the use site, or can it be deferred?

Re: [PATCH 5/8] Enable max_issue for AArch32 and AArch64

Maxim Kuvyrkov wrote: > This change requires benchmarking, which I can't easily do at the moment. I > would appreciate any benchmarking results that you can share. I will run my testsuite on aarch64. Do you need the perf for all the previous patches 1 to 5 together, or just for patch 5? Thanks,

Re: The nvptx port [7/11+] Inform the port about call arguments

On 10/21/14 16:06, Bernd Schmidt wrote: On 10/21/2014 11:53 PM, Jeff Law wrote: So, in the end I'm torn. I don't like adding new hooks when they're not needed, but I have some reservations about relying on the order of stuff in CALL_INSN_FUNCTION_USAGE and I worry a bit that you might end up w

Re: The nvptx port [10/11+] Target files

On 10/20/14 08:33, Bernd Schmidt wrote: These are the main target files for the ptx port. t-nvptx is empty for now but will grow some content with follow up patches. Bernd 010-target.diff * configure.ac: Allow configuring lto for nvptx. * configure: Regenerate. gcc/

Re: [build] Link genmatch with $(LIBINTL)

On October 22, 2014 7:13:41 PM CEST, David Edelsohn wrote: >I reported the same problem in a separate thread. > >On AIX it needs LIBINTL and LIBICONV. I wonder what ends up linking those with Linux? Or is all the features available from glibc? Fixed patch is OK. Thanks, Richard. >Index: Makef

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

>For example, is the pic register saved and restored? Yes, if RA decided that it is better to save it. >If restored, is the code to save it actually better than simply appearing it >out of thin air as did previously? "enabling ebx" gives performance mostly to loops. There still could be some perf

Re: [PATCH] AutoFDO patch for trunk

The patch tested OK. And I think it's a trivial patch, and already committed it to trunk. About the perf parser. I'm syncing the toolchain to head which should already have newer kernel support. Thanks, Dehao On Wed, Oct 22, 2014 at 10:07 AM, Xinliang David Li wrote: > Can someone pre-approve t

Re: Do not build soft-fp code at all for powerpc64-linux-gnu

On Tue, Oct 21, 2014 at 9:06 PM, Joseph S. Myers wrote: > When I added support for using soft-fp in libgcc > , libgcc > configuration was still done in the gcc/ directory, meaning that the > variables set in makefile fragments could not dep

Re: Extract and insert merging patch

On 10/22/14 09:01, Petr Murzin wrote: Hi, Please have a look at updated patch. 2014-10-22 Petr Murzin gcc/ * simplify-rtx.c (simplify_ternary_operation): Simplify vec_merge (vec_duplicate (vec_select)). gcc/testsuite/ * gcc.target/i386/extract-insert-combining.c

Re: [build] Link genmatch with $(LIBINTL)

On Wed, 22 Oct 2014, Rainer Orth wrote: > 2014-10-22 Rainer Orth > > * Makefile.in (build/genmatch(build_exeext)): Add $(LIBINTL) to > BUILD_LIBS. > Add $(LIBINTL_DEP) dependency. No, this doesn't look right. A program built for the build system needs to use build versions

Re: [wwwdocs] Add porting_to.html, describe gnu11 changes

On Wed, Oct 22, 2014 at 09:56:08AM -0700, Mike Stump wrote: > On Oct 22, 2014, at 8:29 AM, Marek Polacek wrote: > > The following adds the porting_to.html document for GCC 5 and > > documents issues arising from moving the default to gnu11. > > So, one way for a person to port a large project, wo

Re: [build] Link genmatch with $(LIBINTL)

I reported the same problem in a separate thread. On AIX it needs LIBINTL and LIBICONV. Index: Makefile.in === --- Makefile.in (revision 216542) +++ Makefile.in (working copy) @@ -2520,7 +2520,7 @@ # These programs need libs over an

Re: [PATCH 2/4] Add liboffloadmic

On Wed, 22 Oct 2014, Jakub Jelinek wrote: > Also, do we really want the messy DOS/Windows '\r' in the messages on > Unix-ish targets? Shouldn't that be dependent on what target is the library > configured for? On platforms where it matters, I think it's still right to use \n only - if in the en

Re: [PATCH] AutoFDO patch for trunk

Can someone pre-approve the patch so that Dehao can check it in after basic testing? David On Wed, Oct 22, 2014 at 10:06 AM, David Edelsohn wrote: >> Rainer Orth writes: > >> As Joseph is repeating over and over again, *nothing* must be included >> before config.h, and auto-profile.c violate

Re: [PATCH] AutoFDO patch for trunk

> Rainer Orth writes: > As Joseph is repeating over and over again, *nothing* must be included > before config.h, and auto-profile.c violates this. > > The following patch at least allows the file to compile without errors; > no idea if this the best order for the headers involved. The patch

Re: [wwwdocs] Add porting_to.html, describe gnu11 changes

On Oct 22, 2014, at 8:29 AM, Marek Polacek wrote: > The following adds the porting_to.html document for GCC 5 and > documents issues arising from moving the default to gnu11. So, one way for a person to port a large project, would be just to select gnu89 as the target language for the project.

Re: [match-and-simplify] Merge from trunk

Richard, It at least needs something like: * Makefile.in (build/genmatch): Link with $(LIBINTL) $(LIBICONV). Index: Makefile.in === --- Makefile.in (revision 216542) +++ Makefile.in (working copy) @@ -2520,7 +2520,7 @@ # These prog

Re: [PATCH PR63530] Fix the pointer alignment in vectorization

Thanks for the review. Following patch has been committed. I will port them to 4.9 branch several days later. 2014-10-22 Guozhi Wei PR tree-optimization/63530 tree-vect-data-refs.c (vect_create_addr_base_for_vector_ref): Set pointer alignment according to DR_MISALIGNMEN

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

On Wed, Oct 22, 2014 at 07:55:57PM +0400, Evgeny Stupachenko wrote: > There is a test for Linux x86 that also fails without the changes in the > patch: > > ChangeLog: > > 2014-10-22 Evgeny Stupachenko > > PR rtl-optimization/63618 > * gcc.target/i386/pr63618.c: New. > > diff

Re: [match-and-simplify] Merge from trunk

This merge has broken bootstrap on AIX. genmatch has a dependency on libintl and libiconv, but is not linked with those libraries. ld: 0711-317 ERROR: Undefined symbol: .libintl_dgettext ld: 0711-317 ERROR: Undefined symbol: .libintl_bindtextdomain ld: 0711-317 ERROR: Undefined symbol: .libiconv

Re: [build] Link genmatch with $(LIBINTL)

On Wed, Oct 22, 2014 at 05:53:22PM +0200, Rainer Orth wrote: > 2014-10-22 Rainer Orth > > * Makefile.in (build/genmatch(build_exeext)): Add $(LIBINTL) to Shouldn't that be build/genmatch$(build_exeext) , both in the ChangeLog and more importantly, in the patch? > BUILD_LIBS. >

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

There is a test for Linux x86 that also fails without the changes in the patch: ChangeLog: 2014-10-22 Evgeny Stupachenko PR rtl-optimization/63618 * gcc.target/i386/pr63618.c: New. diff --git a/gcc/testsuite/gcc.target/i386/pr63618.c b/gcc/testsuite/gcc.target/i386/pr63618.c

[build] Link genmatch with $(LIBINTL)

With the match-and-simply patch, Solaris 10 bootstrap is broken: Undefined first referenced symbol in file libintl_bindtextdomain ../libcpp/libcpp.a(init.o) libintl_dgettext../libcpp/libcpp.a(errors.o) ld: fatal: s

Re: [PATCH] Fix folding of X - X to 0

On 10/22/14 06:58, Richard Biener wrote: The following fixes a bug in fold-const.c which happily drops side-effects when simplifying X - X to 0. I ran into this because on match-and-simplify I cannot simplify expressions with TREE_SIDE_EFFECTS and it happens that TYPE_SIZE for VLAs has side-eff

Re: [wwwdocs] Add porting_to.html, describe gnu11 changes

On Wed, Oct 22, 2014 at 05:29:13PM +0200, Marek Polacek wrote: > The following adds the porting_to.html document for GCC 5 and > documents issues arising from moving the default to gnu11. Now with a typo fixed. --- porting_to.html.mp 2014-10-22 17:25:42.122367884 +0200 +++ porting_to.html 20

[wwwdocs] Add porting_to.html, describe gnu11 changes

The following adds the porting_to.html document for GCC 5 and documents issues arising from moving the default to gnu11. Ok? --- porting_to.html.mp 2014-10-22 17:25:42.122367884 +0200 +++ porting_to.html 2014-10-22 17:23:14.442645393 +0200 @@ -0,0 +1,235 @@ + + + +Porting to GCC 5 + + + +Por

[PATCH, committed] Reset ipa-icf.c:optimizer to NULL when done

After a recent merge of trunk to the jit branch that brought in ipa-icf, the new pass was segfaulting on the second iteration of an in-process compile; e.g. with: test-factorial.exe: internal compiler error: Segmentation fault 0x7f3f20c2301a crash_signal ../../src/gcc/toplev.c:349 0x7f3f20

Re: [PATCH] AutoFDO patch for trunk

Dehao Chen writes: > The updated patch attached. Will commit the patch in 2~3 hours if no > objection is received. Apart from the AIX bootstrap failure your patch introduced, it also breaks Solaris bootstrap: In file included from ./config.h:6:0, from /vol/gcc/src/hg/trunk/loca

Re: RFA: Add libstdc++-v3 support for avr 4/7: fix locale_facets_nonio overloading on struct tm using template

[CCing the libstdc++ list, all libstdc++ patches should go there as well as gcc-patches] On 21/10/14 17:47 +0100, Joern Rennecke wrote: On 21 October 2014 17:29, Jonathan Wakely wrote: >> +typedef typeof (((tm*)0)->tm_sec) __tm_small_int; I think this should probably use __typeof__ to work w

Re: [PATCH][match-and-simplify] Hande side-effects in GENERIC

On Wed, Oct 22, 2014 at 04:20:09PM +0200, Richard Biener wrote: > 2014-10-22 Richard Biener > > * genmatch.c (count_captures): New function. > (dt_simplify::gen): Handle preserving side-effects for > GENERIC code generation. > (decision_tree::gen_generic): Do not reject

[boehm-gc] Fix gnu11 fallout on SPARC

A couple of boehm-gc testcases are FAILing on Solaris/SPARC due to the recent gnu11 changes: FAIL: boehm-gc.c/gctest.c -O2 (test for excess errors) FAIL: boehm-gc.c/thread_leak_test.c -O2 (test for excess errors) FAIL: boehm-gc.lib/staticrootslib.c -O2 (test for excess errors) All of them are ins

Re: Extract and insert merging patch

Hi, Please have a look at updated patch. 2014-10-22 Petr Murzin gcc/ * simplify-rtx.c (simplify_ternary_operation): Simplify vec_merge (vec_duplicate (vec_select)). gcc/testsuite/ * gcc.target/i386/extract-insert-combining.c: New. On Fri, Sep 19, 2014 at 1:43 AM, Je

[testsuite] Fix SPARC testsuite gnu11 fallout

A couple of gcc.target/sparc tests were FAILing on Solaris/SPARC due to the gnu11 change: FAIL: gcc.target/sparc/20001013-1.c (test for excess errors) FAIL: gcc.target/sparc/mnofpu.c (test for excess errors) FAIL: gcc.target/sparc/sparc-ret.c (test for excess errors) FAIL: gcc.target/sparc/ultrasp

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

Two of the new gcc.dg/ipa/ipa-icf-*.c tests were FAILing on Solaris: FAIL: gcc.dg/ipa/ipa-icf-21.c (test for excess errors) UNRESOLVED: gcc.dg/ipa/ipa-icf-21.c scan-ipa-dump icf "Equal symbols: 1" UNRESOLVED: gcc.dg/ipa/ipa-icf-21.c scan-ipa-dump icf "Semantic equality hit:bar->foo" FAIL: gcc.dg/

[gomp4] Wait directive with async and wait clauses.

Hi! This patch adds code to handle the OpenACC wait directive and the wait and async clauses for the wait directive as well as the other directives which support them. OK for gomp4 branch? Thanks! Jim ChangeLog 2014-10-22 James Norris * builtin-types.def (BT_FN_VOID_INT_INT_VAR,

Re: [PATCH] Account for prologue spills in reg_pressure scheduling

On 2014-10-22 2:17 AM, Maxim Kuvyrkov wrote: On Oct 22, 2014, at 4:24 AM, Vladimir Makarov wrote: On 10/20/2014 02:57 AM, Maxim Kuvyrkov wrote: Hi, This patch improves register pressure scheduling (both SCHED_PRESSURE_WEIGHTED and SCHED_PRESSURE_MODEL) to better estimate number of available

Re: [PATCH][match-and-simplify] Hande side-effects in GENERIC

On Wed, 22 Oct 2014, Richard Biener wrote: > > The following auto-handles preserving of side-effects properly > for GENERIC simplification instead of simply rejecting operands > with side-effects. Cases we cannot handle correctly are still > handled that way. > > For example for > > /* (x | CS

Re: [Patch, libstdc++/63497] Avoid dereferencing invalid iterator in regex_executor

On 21/10/14 09:45 -0700, Tim Shen wrote: On Tue, Oct 21, 2014 at 3:25 AM, Jonathan Wakely wrote: Did you manage to produce a testcase that crashed on trunk? Oh I forgot to mention that I've tried my best to make a testcase that crash the trunk, but failed :). I'm not sure if I should directl

[PATCH][match-and-simplify] Hande side-effects in GENERIC

The following auto-handles preserving of side-effects properly for GENERIC simplification instead of simply rejecting operands with side-effects. Cases we cannot handle correctly are still handled that way. For example for /* (x | CST1) & CST2 -> (x & CST2) | (CST1 & CST2) */ (simplify (bit_a

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

Hi all, This is the 4.8 backport of the LINK_SPEC changes to pass down the linker option --fix-cortex-a53-835769 Bootstrapped and tested on aarch64-none-linux-gnu. This depends on the patches under review at: https://gcc.gnu.org/ml/gcc-patches/2014-10/msg01757.html and https://gcc.gnu.org/ml/g

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

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 --enable-fix-cortex-a53-835769). This requires a binutils installation with the patch posted at htt

[jit] Merger of trunk r216524 into jit branch

I've merged svn trunk into the git dmalcolm/jit branch, specifically from trunk r216524 aka d14cac46135326115f0dc589b0b3d2d249d74cf7 (2014-10-21) bringing in all changes since r215958 aka e012cdc775868e9922f5fef9068a764546876d93 (2014-10-06). This required a cleanup of global state "optimizer" in

[PATCH][AArch64] Cleanup logic around aarch64_final_prescan

Hi all, This patch addresses feedback from: https://gcc.gnu.org/ml/gcc-patches/2014-10/msg00933.html and https://gcc.gnu.org/ml/gcc-patches/2014-10/msg00985.html I think the correct way is this approach. We want to return true not only when body is NULL, but also when we know that prev (from wh

[PATCH] Fix folding of X - X to 0

The following fixes a bug in fold-const.c which happily drops side-effects when simplifying X - X to 0. I ran into this because on match-and-simplify I cannot simplify expressions with TREE_SIDE_EFFECTS and it happens that TYPE_SIZE for VLAs has side-effects because it usually contains a SAVE_EXP

Re: [PATCH i386 AVX512] [88/n] Add missed avx512bw/avx512vl cmp intrinsics.

On 22 Oct 16:51, Kirill Yukhin wrote: > gcc/ > * testsuite/gcc.target/i386/avx512bw-vpcmpequb-1.c: New. Please, disregard "testsuite/" in overall ChangeLog entry. -- Thanks, K

Re: [PATCH i386 AVX512] [86/n] Add tests for options.

On 22 Oct 16:37, Kirill Yukhin wrote: > gcc/ > * testsuite/g++.dg/other/i386-2.C: Add new options. Please, disregard "testsuite/gcc.target" in overall ChangeLog entry. -- Thanks, K

Re: [PATCH i386 AVX512] [87/n] Add rest of the tests.

On 22 Oct 16:48, Kirill Yukhin wrote: > Hello, > This patch adds tests for new intrinsics. > > Bootstrapped. > AVX-512* tests on top of patch-set all pass > under simulator. > > Is it ok for trunk? > > gcc/ > * testsuite/gcc.target/i386/avx512bw-check.h: New. Please, disregard "testsuite/g

[PATCH i386 AVX512] [89/n] Improve mask move insn generation.

Hello, This patch fixes kmov* insn generation and adds memory alternative to `movqi_internal' pattern Reg-test included. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/i386.md (movhi_internal): Always detect mskmov.

[PATCH i386 AVX512] [88/n] Add missed avx512bw/avx512vl cmp intrinsics.

Hello, This patch introduces rest of intrinsics (compare). Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/avx512bwintrin.h: Add new intrinsics. * config/i386/avx512vlbwintrin.h: Ditto. * config/i386/avx512

Re: [PATCH] Account for prologue spills in reg_pressure scheduling

Maxim Kuvyrkov writes: > + for (i = 0; i < FIRST_PSEUDO_REGISTER; ++i) > + if (call_used_regs[i]) > + for (c = 0; c < ira_pressure_classes_num; ++c) > + { > + int j; > + enum reg_class cl = ira_pressure_classes[c]; > + > + for (j = 0; j < ira_cl

[PATCH i386 AVX512] [87/n] Add rest of the tests.

Hello, This patch adds tests for new intrinsics. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * testsuite/gcc.target/i386/avx512bw-check.h: New. * testsuite/gcc.target/i386/avx512bw-kunpckdq-1.c: Ditto. * testsuite/gc

[PATCH i386 AVX512] [85/n] Add intrinsics headers.

Hello, This patch introduces necessary AVX-512* intrinsics headers. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/avx512bwintrin.h: New. * config/i386/avx512dqintrin.h: Ditto. * config/i386/avx512vlbwintr

[PATCH i386 AVX512] [86/n] Add tests for options.

Hello, This patch extends sse-* and avx-* tests which checks immediates/options. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * testsuite/g++.dg/other/i386-2.C: Add new options. * testsuite/g++.dg/other/i386-3.C: Ditto.

[PATCH i386 AVX512] [84/n] Add missing immediate checks.

Hello, This tiny patch adds couple of missing immediate checks. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/i386.c (ix86_expand_args_builtin): Handle avx_vpermilv4df_mask, avx_shufpd256_mask, avx_vpermilv2df_ma

[PATCH i386 AVX512] [83/n] Add scatter built-ins.

Hello, This patch adds built-ins for scatter insns. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/i386.c (ix86_init_mmx_sse_builtins): Define __builtin_ia32_gather3siv2df, __builtin_ia32_gather3siv4df,

[PATCH i386 AVX512] [82/n] Add gather built-ins.

Hello, This patch introduces built-ins for gather insns. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/i386.c (ix86_builtins): Add IX86_BUILTIN_GATHER3ALTSIV4DF, IX86_BUILTIN_GATHER3ALTDIV8SF, IX86_BUILTIN_GATHER

PR lto/63603: Fix -fno-lto handling in driver (gcc/gcc.c)

I intent to commit the following patch this evening, which was pre-proved for 4.9/trunk by Richard in the PR. However, I still need to do the bootstrapping and regtesting. Tobias 2014-10-22 Richard Biener Tobias Burnus PR lto/63603 * gcc.c (LINK_COMMAND_SPEC): Ad

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

The gnu11 patch broke Solaris 10 and 11 bootstrap: has /* * It is invalid to compile an XPG3, XPG4, XPG4v2, or XPG5 application * using c99. The same is true for POSIX.1-1990, POSIX.2-1992, POSIX.1b, * and POSIX.1c applications. Likewise, it is invalid to compile an XPG6 * or a POSIX.1-2001

Re: [PATCH i386 AVX512] [81/n] Add new built-ins.

On Wed, Oct 22, 2014 at 03:34:22PM +0400, Kirill Yukhin wrote: > > Can you test with -mavx512 (or whatever enables the builtins?) > Done. > I did: > sync && time for i in `seq 1000` ; do ./build-x86_64-linux/gcc/xgcc > -B./build-x86_64-linux/gcc -O0 -S test.c -mavx512vl ; done > > Here're r

Re: [PATCH i386 AVX512] [81/n] Add new built-ins.

Hello, On 22 Oct 10:09, Richard Biener wrote: > On Tue, Oct 21, 2014 at 5:08 PM, Kirill Yukhin > wrote: > > On 21 Oct 18:47, Kirill Yukhin wrote: > >> On 21 Oct 16:20, Jakub Jelinek wrote: > >> > On Tue, Oct 21, 2014 at 06:08:15PM +0400, Kirill Yukhin wrote: > >> > > --- a/gcc/tree.h > >> > > +++

  1   2   >