Simplify contrib/gcc_update wrt. Java

2017-01-16 Thread Gerald Pfeifer
I committed this straightforward simplification, and while I was at it also adjusted the copyright dates. Gerald 2017-01-17 Gerald Pfeifer * gcc_update: Remove entries related to GCJ and libgcj. Complete copyright years and adjust my e-mail address. Index: gcc_update ===

Re: [PATCH] avoid calling memset et al. with excessively large sizes (PR 79095)

2017-01-16 Thread Jakub Jelinek
On Mon, Jan 16, 2017 at 05:06:40PM -0700, Martin Sebor wrote: > The test case submitted in bug 79095 - [7 regression] spurious > stringop-overflow warning shows that GCC optimizes some loops > into calls to memset with size arguments in excess of the object > size limit. Since such calls will unav

Re: [PATCH] Tree-level fix for PR 69526

2017-01-16 Thread Robin Dapp
Ping. To put it shortly, I'm not sure how to differentiate between: example range of a: [3,3] (ulong)(a + UINT_MAX) + 1 --> (ulong)(a) + (ulong)(-1 + 1), sign extend example range of a: [0,0] (ulong)(a + UINT_MAX) + 1 --> (ulong)(a) + (ulong)(UINT_MAX + 1), no sign extend In this case, there is

[java,i386] Remove LIBGCJ_SONAME for cygwin and mingw32

2017-01-16 Thread Gerald Pfeifer
With libgcj gone, no need to define LIBGCJ_SONAME any more in any of the ports. Applied. Gerald 2017-01-17 Gerald Pfeifer * config/i386/cygwin.h (LIBGCJ_SONAME): Remove. * config/i386/mingw32.h (LIBGCJ_SONAME): Remove. Index: config/i386/cygwin.h

Re: [PATCH 2/6] RISC-V Port: gcc

2017-01-16 Thread Palmer Dabbelt
On Sat, 14 Jan 2017 02:05:27 PST (-0800), mer...@debian.org wrote: > Palmer Dabbelt wrote: > >> diff --git a/gcc/config/riscv/linux.h b/gcc/config/riscv/linux.h >> new file mode 100644 >> index 000..045f6cc >> --- /dev/null >> +++ b/gcc/config/riscv/linux.h >> [...] >> +#define GLIBC_DYNAMIC_L

Re: [PATCH 4/6] RISC-V Port: libsanitizer

2017-01-16 Thread Palmer Dabbelt
On Thu, 12 Jan 2017 15:39:54 PST (-0800), jos...@codesourcery.com wrote: > Have these changes been sent upstream? Although at the present > development stage applying selected changes might be better than a bulk > merge from upstream libsanitizer, they should still go upstream so they > aren't a l

[patch committed] [SH] Fix PR target/78633

2017-01-16 Thread Kaz Kojima
Hi, I've applied the quick fix below for PR target/78633 which results a build failure on the target. Tested on sh4-unknown-linux-gnu. Regards, kaz -- 2017-01-17 Kaz Kojima PR target/78633 * config/sh/sh.md (cmpeqsi_t+1): Call copy_rtx to avoid invalid RTL sha

Re: PR79066, non-PIC code generated for powerpc glibc with -fpic

2017-01-16 Thread Alan Modra
On Mon, Jan 16, 2017 at 01:49:36PM -0600, Segher Boessenkool wrote: > On Mon, Jan 16, 2017 at 03:50:01PM +1030, Alan Modra wrote: > > > > > Okay for trunk if there is nothing unexpected. Thanks! > > > > > > > > I guess I should at least build glibc. > > > > > > Yes exactly, something big that us

Re: [PATCH] Make multiple_target.c aware of LTO (PR lto/66295)

2017-01-16 Thread Jan Hubicka
> Hello. > > Not being expert in multi_target area, however it consists of 2 passes. The > first > one (ipa_target_clone) is responsible for creation of multiple targets for > functions > decorated with __attribute__((target_clones("xxx"))). I guess the pass should > be > called just in LGEN ph

Re: [PATCH] fix integer overflow bugs in gimple-ssa-sprintf.c (PR 78608)

2017-01-16 Thread Martin Sebor
If the FIXME was a future thing, then this is OK with the nits fixed. If the FIXME was a marker for something you intended to address now and just forgot, then we either need another iteration or a follow-up patch depending on the severity of the FIXME in your mind. As we discussed privately, I

[PATCH] avoid calling memset et al. with excessively large sizes (PR 79095)

2017-01-16 Thread Martin Sebor
The test case submitted in bug 79095 - [7 regression] spurious stringop-overflow warning shows that GCC optimizes some loops into calls to memset with size arguments in excess of the object size limit. Since such calls will unavoidably lead to a buffer overflow and memory corruption the attached

RE: [PATCH, MIPS] Target flag and build option to disable indexed memory OPs.

2017-01-16 Thread Moore, Catherine
> -Original Message- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- > ow...@gcc.gnu.org] On Behalf Of Matthew Fortune > Sent: Monday, January 16, 2017 11:25 AM > To: Doug Gilmore ; gcc- > patc...@gcc.gnu.org > Cc: Moore, Catherine > Subject: RE: [PATCH, MIPS] Target flag and

Reverting mingw32 patch from late 2016

2017-01-16 Thread Jeff Law
Your patch for ming32 causes bootstrapping problems for mingw32. Given that mingw32 isn't something I can afford to spend time debugging, I've reverted the patch until it can be fixed. https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78880 Jeff

Add tests for recent dse bugs

2017-01-16 Thread Jeff Law
ACATS already had a test covering the Ada issue, Eric also added a test to the gnat.dg testsuite. So that's well covered. The test for the bootstrap comparison failure was (as expected) trivial to construct (ssa-dse-29.c). The test for the ppc64 big endian failures was easy to extract fro

Re: [PATCH 9f] Add a way for the C frontend to compile __RTL-tagged functions

2017-01-16 Thread David Malcolm
On Mon, 2017-01-16 at 14:54 -0700, Jeff Law wrote: > On 01/09/2017 07:38 PM, David Malcolm wrote: > > The backend is full of singleton state, so we have to compile > > __RTL-functions as soon as we parse them. This means that the > > C frontend needs to invoke the backed. > > > > This patch adds

[PATCH, i386]: Remove LIMIT_RELOAD_CLASS macro

2017-01-16 Thread Uros Bizjak
Hello! This is reload-only macro, not needed after target moved to LRA. 2017-01-17 Uros Bizjak * config/i386/i386.h (LIMIT_RELOAD_CLASS): Remove. Bootstrapped and regression tested on x86_64-linux-gnu {,-m32}. Committed to mainline SVN. Uros. Index: config/i386/i386.h =

Re: [PATCH, rs6000] Add support for vec_rlnm and vec_rlmi

2017-01-16 Thread Bill Schmidt
> On Jan 16, 2017, at 4:24 PM, Segher Boessenkool > wrote: > > Hi Bill, > > A few comments: > > On Mon, Jan 16, 2017 at 12:12:22PM -0600, Bill Schmidt wrote: >> * config/rs6000/rs6000-builtin.def (VRLWNM): New monomorphic >> function entry. > > I had to look up if "monomorphic" is

Re: [PATCH 9g] Extend .md and RTL parsing to support being wired up to cc1

2017-01-16 Thread David Malcolm
On Mon, 2017-01-16 at 15:04 -0700, Jeff Law wrote: > On 01/09/2017 07:38 PM, David Malcolm wrote: > > gcc/ChangeLog: > > * read-md.c (md_reader::read_char): Support filtering > > the input to a subset of line numbers. > > (md_reader::md_reader): Initialize fields > > m_first_line an

Re: [PATCH 1/6] RISC-V Port: gcc/config/riscv/riscv.c

2017-01-16 Thread Andrew Waterman
On Thu, Jan 12, 2017 at 1:38 PM, Joseph Myers wrote: > On Wed, 11 Jan 2017, Palmer Dabbelt wrote: > >> +#include > > This is included in system.h, so don't include it here. OK. > >> + error ("unknown cpu `%s' for -mtune", cpu_string); > > This is using very-old-style `' quotes. Diagnostics sh

Re: [PATCH, bugfix] builtin expansion of strcmp for rs6000

2017-01-16 Thread Segher Boessenkool
On Mon, Jan 16, 2017 at 03:09:35PM -0600, Aaron Sawdey wrote: > Tulio noted that glibc's strncmp test was failing. This turned out to > be the use of signed HOST_WIDE_INT for handling strncmp length. The > glibc test calls strncmp with length 2^64-1, presumably to provoke > exactly this type of bug

Re: [PATCH] Add AVX512 k-mask intrinsics

2017-01-16 Thread Jakub Jelinek
On Tue, Jan 17, 2017 at 01:30:11AM +0300, Andrew Senkevich wrote: > here is one more part of intrinsics for k-mask registers shifts: The software developer manuals describe KSHIFT{L,R}* like: KSHIFTLW COUNT <- imm8[7:0] DEST[MAX_KL-1:0] <- 0 IF COUNT <=15 THEN DEST[15:0] <- SRC1[15:0] << COUNT; FI

Re: [PATCH][PR tree-optimization/79090] Fix two minor DSE bugs

2017-01-16 Thread Richard Biener
On January 16, 2017 7:27:53 PM GMT+01:00, Jeff Law wrote: >On 01/16/2017 01:51 AM, Richard Biener wrote: >> On Sun, Jan 15, 2017 at 10:34 AM, Jeff Law wrote: >>> >>> At one time I know I had the max_size == size test in >valid_ao_ref_for_dse. >>> But it got lost at some point. This is what cause

Re: Implement -Wduplicated-branches (PR c/64279) (v3)

2017-01-16 Thread Jeff Law
On 01/09/2017 02:21 AM, Marek Polacek wrote: On Thu, Jan 05, 2017 at 04:41:28PM +0100, Jakub Jelinek wrote: On Thu, Jan 05, 2017 at 04:39:40PM +0100, Marek Polacek wrote: Coming back to this... Right, after h0 == h1 is missing && operand_equal_p (thenb, elseb, 0) or so (the exact last operan

Re: [PATCH] Add AVX512 k-mask intrinsics

2017-01-16 Thread Andrew Senkevich
Hi, here is one more part of intrinsics for k-mask registers shifts: gcc/ * config/i386/avx512bwintrin.h: Add k-mask registers shift intrinsics. * config/i386/avx512dqintrin.h: Ditto. * config/i386/avx512fintrin.h: Ditto. * config/i386/i386-builtin-types.def: Add new types. *

Re: [PATCH, rs6000] Add support for vec_rlnm and vec_rlmi

2017-01-16 Thread Segher Boessenkool
Hi Bill, A few comments: On Mon, Jan 16, 2017 at 12:12:22PM -0600, Bill Schmidt wrote: > * config/rs6000/rs6000-builtin.def (VRLWNM): New monomorphic > function entry. I had to look up if "monomorphic" is an existing word in this context. Unfortunately it is, sigh (it clashes hard wi

Re: [ping] libobjc patches and zlib update

2017-01-16 Thread Jeff Law
On 01/12/2017 04:24 AM, Matthias Klose wrote: [CCing some global reviewers] Please review the two libobjc patches: https://gcc.gnu.org/ml/gcc-patches/2016-12/msg02003.html - not a regression, but a simplification of the new configure option. https://gcc.gnu.org/ml/gcc-patches/2016

Re: [PATCH 9g] Extend .md and RTL parsing to support being wired up to cc1

2017-01-16 Thread Jeff Law
On 01/09/2017 07:38 PM, David Malcolm wrote: gcc/ChangeLog: * read-md.c (md_reader::read_char): Support filtering the input to a subset of line numbers. (md_reader::md_reader): Initialize fields m_first_line and m_last_line. (md_reader::read_file_fragment):

Re: [PATCH 9i] testsuite: add aarch64-specific files

2017-01-16 Thread Jeff Law
On 01/09/2017 07:38 PM, David Malcolm wrote: gcc/testsuite/ChangeLog: * gcc.dg/rtl/aarch64/asr_div1.c: New test case. * gcc.dg/rtl/aarch64/pr71779.c: New test case. OK. jeff

Re: [PATCH 9j] testsuite: add x86_64-specific files

2017-01-16 Thread Jeff Law
On 01/09/2017 07:38 PM, David Malcolm wrote: A collection of test cases, capturing the state of various functions at various places within the pass list, and verifying that we can restart at various passes. gcc/testsuite/ChangeLog: * gcc.dg/rtl/x86_64/dfinit.c: New test case. * g

Re: [PATCH 9h] testsuite: add platform-independent files

2017-01-16 Thread Jeff Law
On 01/09/2017 07:38 PM, David Malcolm wrote: This patch adds: - an rtl.exp (to make it easy to run just the tests for __RTL-tagged functions) - a test.c source file I used when generating the various RTL dumps (for reference) - a couple of tests of __RTL parser errors gcc/testsuite/ChangeL

Re: [PATCH 9f] Add a way for the C frontend to compile __RTL-tagged functions

2017-01-16 Thread Jeff Law
On 01/09/2017 07:38 PM, David Malcolm wrote: The backend is full of singleton state, so we have to compile __RTL-functions as soon as we parse them. This means that the C frontend needs to invoke the backed. This patch adds the support needed. Normally this would be a no-no, and including rtl

Re: [PATCH 9e] Update "startwith" logic for pass-skipping to handle __RTL functions

2017-01-16 Thread Jeff Law
On 01/09/2017 07:38 PM, David Malcolm wrote: gcc/ChangeLog: * passes.c: Include "insn-addr.h". (should_skip_pass_p): Add logging. Update logic for running "expand" to be compatible with both __GIMPLE and __RTL. Guard property-provider override so it is only done

[committed] Fix a tree sharing bug during gimplification (PR c/79089)

2017-01-16 Thread Jakub Jelinek
Hi! gimplify_init_constructor sometimes uses object == lhs twice, once in gimple_build_assign and then as the result value, which is wrong if object is something that can't be shared such as COMPONENT_REF. Fixed by unsharing it in that case. Bootstrapped/regtested on x86_64-linux and i686-linux,

Re: [PATCH 9c] callgraph: handle __RTL functions

2017-01-16 Thread Jeff Law
On 01/09/2017 07:38 PM, David Malcolm wrote: The RTL backend code is full of singleton state, so we have to handle functions as soon as we parse them. This requires various special-casing in the callgraph code. gcc/ChangeLog: * cgraph.h (symtab_node::native_rtl_p): New decl. * c

Re: [PATCH 9d] Don't call delete_tree_ssa for __RTL functions

2017-01-16 Thread Jeff Law
On 01/10/2017 06:42 AM, Richard Biener wrote: On Tue, Jan 10, 2017 at 3:38 AM, David Malcolm wrote: gcc/ChangeLog: * final.c (rest_of_clean_state): Don't call delete_tree_ssa for __RTL functions. Heh, so you are lucky that nothing looks at this. MEM_EXPRs can contain SSA name

Re: [PATCH 9d] Don't call delete_tree_ssa for __RTL functions

2017-01-16 Thread Jeff Law
On 01/09/2017 07:38 PM, David Malcolm wrote: gcc/ChangeLog: * final.c (rest_of_clean_state): Don't call delete_tree_ssa for __RTL functions. OK. And just for the record, these patches were submitted prior to stage1 close. I'm hesitant to go forward with them unless the set as

Re: [PATCH, GCC/LRA, gcc-5/6-branch] Fix PR78617: Fix conflict detection in rematerialization

2017-01-16 Thread Vladimir Makarov
On 01/16/2017 02:26 PM, Jeff Law wrote: On 01/13/2017 11:19 AM, Thomas Preudhomme wrote: Ping? I'm not sure if an ok from Valdimir is enough or if I also need RM approval. Vlad's approval is all you need. Thomas, the patch is ok for backporting. It is pretty safe.

Re: [PATCH] Fix as flags for -c xx.s -g0 -g (PR driver/49726)

2017-01-16 Thread Jeff Law
On 01/10/2017 03:51 PM, Jakub Jelinek wrote: Hi! ASM_DEBUG_SPEC uses usually %{g:%{!g0:--gdwarf2}} or something similar. In the past, it used to be %{g:--gdwarf2}. Both have problems (and thus this is also a regression). The problem with the current ASM_DEBUG_SPEC is that if one uses -g0 -g, w

Re: [PATCH, bugfix] builtin expansion of strcmp for rs6000

2017-01-16 Thread Aaron Sawdey
Here is an updated version of this patch.  Tulio noted that glibc's strncmp test was failing. This turned out to be the use of signed HOST_WIDE_INT for handling strncmp length. The glibc test calls strncmp with length 2^64-1, presumably to provoke exactly this type of bug. Fixing the issue require

[PATCH, i386]: Tighten checks in HARD_REGNO_CALLER_SAVE_MODE

2017-01-16 Thread Uros Bizjak
Hello! In addition to mask registers, QImode and HImode fixups in HARD_REGNO_CALLER_SAVE_MODE apply only to general registers. 2017-01-16 Uros Bizjak * config/i386/i386.h (HARD_REGNO_CALLER_SAVE_MODE): Apply HImode and QImode fixups to general and mask registers only. Bootstrapped an

Re: PR79066, non-PIC code generated for powerpc glibc with -fpic

2017-01-16 Thread Segher Boessenkool
On Mon, Jan 16, 2017 at 03:50:01PM +1030, Alan Modra wrote: > > > > Okay for trunk if there is nothing unexpected. Thanks! > > > > > > I guess I should at least build glibc. > > > > Yes exactly, something big that uses pic -- it is pretty obvious it won't > > change anything for non-pic. > > gl

Re: [PATCH, GCC/LRA, gcc-5/6-branch] Fix PR78617: Fix conflict detection in rematerialization

2017-01-16 Thread Jeff Law
On 01/13/2017 11:19 AM, Thomas Preudhomme wrote: Ping? I'm not sure if an ok from Valdimir is enough or if I also need RM approval. Vlad's approval is all you need. jeff

Re: [PATCH] Allow building GCC with PTX offloading even without CUDA being installed (gcc and nvptx-tools patches)

2017-01-16 Thread Jeff Law
On 01/13/2017 11:28 AM, Jakub Jelinek wrote: On Fri, Jan 13, 2017 at 06:19:02PM +, Joseph Myers wrote: --- libgomp/plugin/cuda/cuda.h.jj 2017-01-13 15:58:00.966544147 +0100 +++ libgomp/plugin/cuda/cuda.h 2017-01-13 17:02:47.355817896 +0100 @@ -0,0 +1,174 @@ +/* CUDA API description. +

Re: [PATCH][PR tree-optimization/79090] Fix two minor DSE bugs

2017-01-16 Thread Jeff Law
On 01/16/2017 01:51 AM, Richard Biener wrote: On Sun, Jan 15, 2017 at 10:34 AM, Jeff Law wrote: At one time I know I had the max_size == size test in valid_ao_ref_for_dse. But it got lost at some point. This is what caused the Ada failure. Technically it'd be OK for the potentially dead stor

[PATCH] Fix testcase for PR c/78304

2017-01-16 Thread David Malcolm
On Mon, 2017-01-16 at 13:31 +0100, Rainer Orth wrote: > Hi Christophe, > > > > Successfully bootstrapped®rtested on x86_64-pc-linux-gnu; > > > adds 34 PASS results to gcc.sum. > > > > > These 2 tests fail on arm: > > > > gcc.dg/format/pr78304.c (test for warnings, line 9) > > gcc.dg/form

[PATCH, rs6000] Add support for vec_rlnm and vec_rlmi

2017-01-16 Thread Bill Schmidt
Hi, ISA 3.0 introduces new instructions vrlwmi, vrldmi, vrlwnm, and vrldnm. This patch provides access to them via built-ins, including the vec_rlmi and vec_rlnm built-ins mandated by Appendix A of the ELFv2 ABI document. I also added a vec_vrlnm built-in, which is a more direct translation of the

Re: [driver, doc] Support escaping special characters in specs

2017-01-16 Thread Sandra Loosemore
On 01/16/2017 03:54 AM, Rainer Orth wrote: Hi Sandra, On 01/13/2017 05:59 AM, Rainer Orth wrote: diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi --- a/gcc/doc/invoke.texi +++ b/gcc/doc/invoke.texi @@ -26391,6 +26391,13 @@ be as many clauses as you need. This ma @end table +The swit

RE: [PATCH] MIPS: Fix generation of DIV.G and MOD.G for Loongson targets.

2017-01-16 Thread Maciej W. Rozycki
On Mon, 16 Jan 2017, Toma Tabacu wrote: > After searching through the archives, I have found an interesting bit of > information about DIV.G/MOD.G in the original submission thread: > > > > Ruan Beihong 23 July 2008: > > > > > > I've seen the Loongson 2F manual carefully. The (d)div(u) is > > >

Re: [PATCH][GCC][PATCHv3] Improve fpclassify w.r.t IEEE like numbers in GIMPLE.

2017-01-16 Thread Tamar Christina
Ping. Does this still have a chance or should I table it till Stage 1 opens again? Tamar. From: Tamar Christina Sent: Tuesday, January 3, 2017 9:55:51 AM To: Jeff Law; Joseph Myers Cc: GCC Patches; Wilco Dijkstra; rguent...@suse.de; Michael Meissner; nd S

RE: [PATCH] MIPS: Fix generation of DIV.G and MOD.G for Loongson targets.

2017-01-16 Thread Toma Tabacu
After searching through the archives, I have found an interesting bit of information about DIV.G/MOD.G in the original submission thread: > > Ruan Beihong 23 July 2008: > > > > I've seen the Loongson 2F manual carefully. The (d)div(u) is > > internally splited into one (d)div(u).g and one (d)mod

RE: [PATCH, MIPS] Target flag and build option to disable indexed memory OPs.

2017-01-16 Thread Matthew Fortune
Doug Gilmore > I recently bisected PR78176 to problems introduced with r21650. > > Given the short time until the release, we would like to provide a > target flag and build option to avoid the bug until we are able to > resolve the problem with the commit. Note that as Matthew Fortune has > men

Re: libgo patch committed: Update to Go1.8rc1

2017-01-16 Thread Rainer Orth
Hi Ian, > This seems to need a version of defs_solaris.go that works with the > constants and structs living in syscall. I've made some progress here: I need to check for SIOCGLIF* ioctls, and for the IFT_* constants. I've updated configure.in and sysinfo.c accordingly, and adapted defs_solari

Re: [PATCH] Fix gcc.dg/tree-ssa/scev-[345].c testcases

2017-01-16 Thread Christophe Lyon
On 16 January 2017 at 10:43, Richard Biener wrote: > On Mon, 16 Jan 2017, Christophe Lyon wrote: > >> On 13 January 2017 at 12:16, Bin.Cheng wrote: >> > On Fri, Jan 13, 2017 at 9:46 AM, Richard Biener wrote: >> >> >> >> The following is an attempt to change those testcases to be less dependent >

[PATCH] PR66145 use new ABI for std::ios::failure exceptions

2017-01-16 Thread Jonathan Wakely
This changes the version of std::ios_base::failure thrown by libstdc++.so to be the new __cxx11 ABI one, matching the default for headers that are trying to catch the exception. There's no simple way to do this as an easy transition, but hopefully by now most people are either using the new ABI o

Re: [C++ Patch] PR 71737

2017-01-16 Thread Paolo Carlini
On 16/01/2017 16:11, David Edelsohn wrote: This patch caused a libstdc++ regression on AIX libstdc++-v3/include/tr1/shared_ptr.h:556: internal compiler error: tree check: expected class 'type', have 'exceptional' (error_mark) in build_variant_type_copy, at tree.c:6737 I noticed, patch reverted

Re: [PATCH][AArch64][GCC 6] PR target/79041: Correct -mpc-relative-literal-loads logic in aarch64_classify_symbol

2017-01-16 Thread Kyrill Tkachov
On 13/01/17 16:35, James Greenhalgh wrote: On Wed, Jan 11, 2017 at 04:32:45PM +, Kyrill Tkachov wrote: Hi all, In this PR we generated ADRP/ADD instructions with :lo12: relocations on symbols even though -mpc-relative-literal-loads is used. This is due to the confusing double-negative logi

[PING][PATCH, ARM] PR71607: New approach to arm_disable_literal_pool

2017-01-16 Thread Andre Vieira (lists)
On 28/12/16 09:58, Andre Vieira (lists) wrote: > On 29/11/16 09:45, Andre Vieira (lists) wrote: >> On 17/11/16 10:00, Ramana Radhakrishnan wrote: >>> On Thu, Oct 6, 2016 at 2:57 PM, Andre Vieira (lists) >>> wrote: Hello, This patch tackles the issue reported in PR71607. This patch t

Re: libgo patch committed: Update to Go1.8rc1

2017-01-16 Thread Rainer Orth
Hi Ian, > On Sun, Jan 15, 2017 at 3:12 AM, Andreas Schwab wrote: >> In file included from ../../../libgo/runtime/runtime.h:113:0, >> from ../../../libgo/runtime/go-libmain.c:15: >> ./runtime.inc:650:8: error: redefinition of 'struct siginfo' >> struct siginfo { >> ^~

Re: [C++ Patch] PR 71737

2017-01-16 Thread David Edelsohn
This patch caused a libstdc++ regression on AIX libstdc++-v3/include/tr1/shared_ptr.h:556: internal compiler error: tree check: expected class 'type', have 'exceptional' (error_mark) in build_variant_type_copy, at tree.c:6737

Re: [PATCH][AArch64 - v4] Simplify eh_return implementation

2017-01-16 Thread Wilco Dijkstra
Here is the updated version: This patch simplifies the handling of the EH return value. We force the use of the frame pointer so the return location is always at FP + 8. This means we can emit a simple volatile access in EH_RETURN_HANDLER_RTX without needing md patterns, splitters and frame of

Re: [2/5][DWARF] Generate dwarf information for -msign-return-address by introducing new DWARF mapping hook

2017-01-16 Thread Jiong Wang
On 13/01/17 18:02, Jiong Wang wrote: On 13/01/17 16:09, Richard Earnshaw (lists) wrote: On 06/01/17 11:47, Jiong Wang wrote: This patch is an update on DWARF generation for return address signing. According to new proposal, we simply needs to generate REG_CFA_WINDOW_SAVE annotation. gcc/

Re: [PATCH] Speed-up use-after-scope (re-writing to SSA) (version 2)

2017-01-16 Thread Jakub Jelinek
On Mon, Jan 09, 2017 at 03:58:04PM +0100, Martin Liška wrote: > >> Well, having following sample: > >> > >> int > >> main (int argc, char **argv) > >> { > >> int *ptr = 0; > >> > >> { > >> int a; > >> ptr = &a; > >> *ptr = 12345; > >> } > >> > >> *ptr = 12345; > >> return *ptr

Re: Powerpc bootstrap failure due to duplicate case value

2017-01-16 Thread Alan Modra
On Mon, Jan 16, 2017 at 12:41:29PM +0100, Jakub Jelinek wrote: > Or, as a switch it could be of the form: > switch (INSN_CODE (insn)) > { > #ifdef HAVE_ctrsi_internal1 > case CODE_FOR_ctrsi_internal1: > case CODE_FOR_ctrsi_internal2: > case CODE_FOR_ctrsi_internal3: > case COD

Re: [committed] Don't suppress bogus usage of macros from system headers in -Wformat (PR c/78304)

2017-01-16 Thread Rainer Orth
Hi Christophe, >> Successfully bootstrapped®rtested on x86_64-pc-linux-gnu; >> adds 34 PASS results to gcc.sum. >> > These 2 tests fail on arm: > > gcc.dg/format/pr78304.c (test for warnings, line 9) > gcc.dg/format/pr78304.c -DWIDE (test for warnings, line 9) also on sparc-sun-solari

[PATCH] PR78702 fix accessibility of locale::facet::__shim

2017-01-16 Thread Jonathan Wakely
This fixes a bug that prevents building the library with Clang (though I don't know why anybody's doing that). G++ doesn't notice it due to one of our bugs with access checking in templates. PR libstdc++/78702 * include/bits/locale_classes.h (locale::facet::__shim): Change from

Re: Powerpc bootstrap failure due to duplicate case value

2017-01-16 Thread Jakub Jelinek
On Mon, Jan 16, 2017 at 09:53:17PM +1030, Alan Modra wrote: > Commited as obvious. > > PR target/79098 > * config/rs6000/rs6000.c (rs6000_legitimate_combined_insn): Don't > use a switch. Perhaps it would be useful to write why it can't be written as a switch. Or, as a switch it

Re: [v3 PATCH] PR libstdc++/78389

2017-01-16 Thread Jonathan Wakely
On 16/01/17 11:24 +, Jonathan Wakely wrote: OK for trunk with the additional changes to use better magic numbers in the tests. Oh, and for the branches too.

Re: [1/5][AArch64] Return address protection on AArch64

2017-01-16 Thread James Greenhalgh
On Fri, Jan 13, 2017 at 05:05:43PM +, Jiong Wang wrote: > On 13/01/17 16:04, James Greenhalgh wrote: > >On Fri, Jan 06, 2017 at 11:47:07AM +, Jiong Wang wrote: > >>On 11/11/16 18:22, Jiong Wang wrote: > >>gcc/ > >>2017-01-06 Jiong Wang > >> > >> * config/aarch64/aarch64-opts.h (a

Re: [v3 PATCH] PR libstdc++/78389

2017-01-16 Thread Jonathan Wakely
On 15/01/17 19:07 +0200, Ville Voutilainen wrote: PR libstdc++/78389 Fix backwards size adjustments. I don't think repeating this text here and ... * include/bits/list.tcc (merge(list&&)): Fix backwards size adjustments. ... here is useful. More useful would be a good Git-style

Powerpc bootstrap failure due to duplicate case value

2017-01-16 Thread Alan Modra
Commited as obvious. PR target/79098 * config/rs6000/rs6000.c (rs6000_legitimate_combined_insn): Don't use a switch. diff --git a/gcc/config/rs6000/rs6000.c b/gcc/config/rs6000/rs6000.c index 11394b2..f1d5d9d 100644 --- a/gcc/config/rs6000/rs6000.c +++ b/gcc/config/rs6000/

Re: [PATCH][AArch64 - v3] Simplify eh_return implementation

2017-01-16 Thread James Greenhalgh
On Fri, Jan 13, 2017 at 07:50:48PM +, Wilco Dijkstra wrote: > James Greenhalgh wrote: > > > I've been putting off reviewing this patch for a while now, because I don't > > understand enough about the current eh_return code to understand why what > > you're proposing is correct. > > > > The bes

Re: [driver, doc] Support escaping special characters in specs

2017-01-16 Thread Rainer Orth
Hi Sandra, > On 01/13/2017 05:59 AM, Rainer Orth wrote: >> diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi >> --- a/gcc/doc/invoke.texi >> +++ b/gcc/doc/invoke.texi >> @@ -26391,6 +26391,13 @@ be as many clauses as you need. This ma >> >> @end table >> >> +The switch matching text @code{S

RE: [PATCH] PR79079 Fix __builtin_mul_overflow code gen for !TRULY_NOOP_TRUNCATION target

2017-01-16 Thread Matthew Fortune
Kito Cheng writes: > On Mon, Jan 16, 2017 at 02:42:08PM +0800, Kito Cheng wrote: > > 2017-01-16 Kito Cheng > > Kuan-Lin Chen > > > > PR target/PR79079 > > * gcc/internal-fn.c (expand_mul_overflow): Use convert_modes > instead of > > gen_lowpart. Thanks for t

Re: [committed] Don't suppress bogus usage of macros from system headers in -Wformat (PR c/78304)

2017-01-16 Thread Christophe Lyon
Hi David, On 13 January 2017 at 21:04, David Malcolm wrote: > c-lex.c: lex_string uses cpp_get_token rather than > cpp_get_token_with_location, and hence the C family of frontends > record the physical locations of tokens in string concatenations, rather > than the virtual locations, discarding a

Re: [PATCH] Fix gcc.dg/tree-ssa/scev-[345].c testcases

2017-01-16 Thread Rainer Orth
Hi Richard, > The following is an attempt to change those testcases to be less dependent > on previous passes. The original motivation of the testcases seems to be > testing SCEV capabilities and in turn IVOPTs decisions, thus the testcases > are changed to check the IVO dump, use the GIMPLE FE f

Re: [PATCH] Fix gcc.dg/tree-ssa/scev-[345].c testcases

2017-01-16 Thread Richard Biener
On Mon, 16 Jan 2017, Christophe Lyon wrote: > On 13 January 2017 at 12:16, Bin.Cheng wrote: > > On Fri, Jan 13, 2017 at 9:46 AM, Richard Biener wrote: > >> > >> The following is an attempt to change those testcases to be less dependent > >> on previous passes. The original motivation of the tes

[PATCH] Fix PR71433

2017-01-16 Thread Richard Biener
The following makes VRP deal better with the situation where the same assertion is to be inserted on all predecessors of a BB. That avoids the spurious array-bound warning (and enables some optimization) when jump-threading f**ed up the CFG in such redundant way. (the fully "correct" way to deal

Re: [PATCH] BRIG frontend: request for a global review

2017-01-16 Thread Pekka Jääskeläinen
OK, I'll see into adapting the Jakub's idea and also check if some of the simplest builtins are better expanded directly to tree nodes instead. I'm not sure if lto support is needed though as the assumption now is to have fully linked input to this FE (all necessary BRIG modules fed in at build t

Re: [PATCH] Fix gcc.dg/tree-ssa/scev-[345].c testcases

2017-01-16 Thread Christophe Lyon
On 13 January 2017 at 12:16, Bin.Cheng wrote: > On Fri, Jan 13, 2017 at 9:46 AM, Richard Biener wrote: >> >> The following is an attempt to change those testcases to be less dependent >> on previous passes. The original motivation of the testcases seems to be >> testing SCEV capabilities and in

[PATCH] Make multiple_target.c aware of LTO (PR lto/66295)

2017-01-16 Thread Martin Liška
Hello. Not being expert in multi_target area, however it consists of 2 passes. The first one (ipa_target_clone) is responsible for creation of multiple targets for functions decorated with __attribute__((target_clones("xxx"))). I guess the pass should be called just in LGEN phase and consecutive

Re: [PATCH] BRIG frontend: request for a global review

2017-01-16 Thread Jakub Jelinek
On Mon, Jan 16, 2017 at 09:46:43AM +0100, Richard Biener wrote: > There are 187 of them (well, simple grep of DEF_HSAIL, so probably a bit > less). > They aren't really documented but I guess that __hsail_bitmask_u64 for example > is really equivalent to sth like -1U >> n << m? So I'm not sure wh

Re: [PATCH][PR tree-optimization/79090] Fix two minor DSE bugs

2017-01-16 Thread Richard Biener
On Sun, Jan 15, 2017 at 10:34 AM, Jeff Law wrote: > > At one time I know I had the max_size == size test in valid_ao_ref_for_dse. > But it got lost at some point. This is what caused the Ada failure. > > Technically it'd be OK for the potentially dead store to have a variable > size as long as th

Re: [PATCH] BRIG frontend: request for a global review

2017-01-16 Thread Richard Biener
On Fri, Jan 13, 2017 at 4:54 PM, Pekka Jääskeläinen wrote: > On Fri, Jan 13, 2017 at 2:34 PM, Richard Biener > wrote: >> On Thu, Jan 12, 2017 at 3:55 PM, Pekka Jääskeläinen >> wrote: >>> Hi, >>> >>> A gentle ping... >> >> Looking at 002/ >> >> What's the reason of having brig-builtins.def? I d

Re: [PATCH] system.h: Poison strndup (PR bootstrap/78616)

2017-01-16 Thread Richard Biener
On Fri, Jan 13, 2017 at 10:17 PM, David Malcolm wrote: > This patch poisons strndup (in system.h), as requested in the > discussion of PR bootstrap/78616. > > Successfully bootstrapped®rtested on x86_64-pc-linux-gnu. > > OK for trunk? Ok. Richard. > gcc/ChangeLog: > PR bootstrap/78616 >

[ping] 3 aarch64/arm/rs6000 patches

2017-01-16 Thread Eric Botcazou
aarch64 (Enable descriptors for nested functions in Ada): https://gcc.gnu.org/ml/gcc-patches/2016-11/msg01253.html arm (Enable descriptors for nested functions in Ada): https://gcc.gnu.org/ml/gcc-patches/2016-11/msg01254.html rs6000 (Fix reload failures in 64-bit mode): https://gcc.gnu.org/

Re: [patch,avr]: Increase branch costs after reload.

2017-01-16 Thread Denis Chertykov
2017-01-13 17:28 GMT+04:00 Georg-Johann Lay : > This adds a penalty of 4 to the post-reload branch costs. > > Purpose is reduce the number of out-of-line blocks like in > > unsigned long variant5 (unsigned in) > { > unsigned long out = 0; > if (in & (1 << 0)) out |= 0xful << (4*0); > if