Re: [RFA 2/2]: --enable-explicit-exception-frame-registration compatibility option

2014-09-18 Thread Jeff Law
On 09/12/14 11:51, Hans-Peter Nilsson wrote: Ping! From: Hans-Peter Nilsson Date: Thu, 4 Sep 2014 23:42:28 +0200 This adds an option to allow programs and libraries built *without* inhibit_libc to stay compatible with system librarie

Re: Ping: [RFA 1/2]: Don't ignore target_header_dir when deciding inhibit_libc

2014-09-18 Thread Jeff Law
On 09/12/14 11:49, Hans-Peter Nilsson wrote: Ping! From: Hans-Peter Nilsson Date: Thu, 4 Sep 2014 23:40:40 +0200 The directory at $target_header_dir is already inspected in gcc/configure, for e.g. glibc version and stack protector su

Re: [PATCH 3/5] Use simplify_subreg_regno in combine.c:subst

2014-09-18 Thread Jeff Law
On 09/18/14 04:13, Richard Sandiford wrote: combine.c:subst should refuse to substitute a hard register into a subreg if the new subreg would not be simplified to a simple hard register, since the result would have to be reloaded. This is more for optimisation than correctness, since in theory th

Re: [PATCH 2/5] Tweak subreg_get_info documentation

2014-09-18 Thread Jeff Law
On 09/18/14 04:10, Richard Sandiford wrote: Try to clarify what subreg_get_info does and doesn't check. gcc/ * rtl.h (subreg_info): Expand commentary * rtlanal.c (subreg_get_info): Likewise. OK independently of the other patches as well. jeff

Re: [PATCH 0/5] Fix handling of word subregs of wide registers

2014-09-18 Thread Jeff Law
On 09/18/14 04:07, Richard Sandiford wrote: This series is a cleaned-up version of: https://gcc.gnu.org/ml/gcc/2014-03/msg00163.html The underlying problem is that the semantics of subregs depend on the word size. You can't have a subreg for byte 2 of a 4-byte word, say, but you can have

Re: [PATCH 1/5] Allow *_HARD_REG_SET arguments to be const

2014-09-18 Thread Jeff Law
On 09/18/14 04:09, Richard Sandiford wrote: Patch 4 needs to pass a const HARD_REG_SET to AND/COPY_HARD_REG_SET. This patch allows that for all intent-in arguments. gcc/ * hard-reg-set.h (COPY_HARD_REG_SET, COMPL_HARD_REG_SET) (AND_HARD_REG_SET, AND_COMPL_HARD_REG_SET, IOR_HARD_

Re: [patch i386]: Sibcall tail-call improvement and partial fix PR/60104

2014-09-18 Thread Jeff Law
On 09/18/14 15:26, Kai Tietz wrote: Hi, it isn't true that I didn't replied to Iant. I did this on IRC. As I explained there already, this hunk about thunks is more consolidation of code-paths in that function, and not really part of a feature. As this code-path isn't prominent mark being Dar

Re: [C++ Patch] PR 62232

2014-09-18 Thread Gerald Pfeifer
Hi Paolo, On Wed, 17 Sep 2014, Paolo Carlini wrote: > clang recently, in 3.5.0 if I remember correctly, stopped -Wnon-virtual-dtor > warning for final classes. I think it makes sense to do the same. mind adding a note to wwwdocs/htdocs/gcc-5/changes.html ? Thanks, Gerald

Re: [PATCH] Do not allow non-allocatable registers in scratch_operand

2014-09-18 Thread Jeff Law
On 09/15/14 18:58, Segher Boessenkool wrote: Currently, scratch_operand allows all hard registers, also those that cannot be allocated and are only generated explicitly by the backend. This causes problems. Consider the case where combine combines instructions A and B, where B clobbers such a n

Re: [PATCH, 2/2] shrink wrap a function with a single loop: split live_edge

2014-09-18 Thread Jeff Law
On 09/16/14 00:55, Zhenqiang Chen wrote: -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Jiong Wang Sent: Monday, September 15, 2014 11:28 PM To: Zhenqiang Chen Cc: gcc-patches@gcc.gnu.org; Jeff Law Subject: Re: [PATCH, 2/2] s

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

2014-09-18 Thread Jeff Law
On 09/18/14 03:35, Alan Lawrence wrote: Moreover, I think we both agree that if result_mode==shift_mode, the transformation is correct. "Just" putting that check in, achieves what I'm trying for here, so I'd be happy to go with the attached patch and call it a day. However, I'm a little concerned

Re: [patch i386]: Sibcall tail-call improvement and partial fix PR/60104

2014-09-18 Thread Jeff Law
On 09/18/14 15:35, FX wrote: Could you give a link to the patch? I’m not finding it. Has it been tested on darwin? If not, I can do it. That would be greatly appreciated. Jeff

Re: [PATCHv4] Vimrc config with GNU formatting

2014-09-18 Thread Yury Gribov
On 09/18/2014 09:20 PM, Segher Boessenkool wrote: > On Thu, Sep 18, 2014 at 12:40:08PM +0400, Yury Gribov wrote: >> When typing 'make .local.vimrc' in GCC build directory one would expect >> .local.vimrc to be created at the root of build directory, not srcdir. > > Yes, you would not expect it to

Re: [patch i386]: Sibcall tail-call improvement and partial fix PR/60104

2014-09-18 Thread Jeff Law
On 09/18/14 16:20, Iain Sandoe wrote: 1. There has been a change made "to make the upper path like the lower path" (as you said on IRC). - apparently (from our conversation) you don't expect this to be a general optimisation improvement. - but it doesn't seem to be either "obvious" or "a

Re: Fix pr61848, linux kernel miscompile

2014-09-18 Thread Jeff Law
On 09/16/14 00:03, Alan Modra wrote: gcc testsuite additions? I decline. It is too soon. If you had read my patch submission you'll see that at some stage gcc was supposed to warn on conflicting section attributes, but hasn't done so for a very long time. There needs to be some agreement on w

Re: [PATCH 2/2] Add patch for debugging compiler ICEs.

2014-09-18 Thread Yury Gribov
On 09/19/2014 02:16 AM, Joseph S. Myers wrote: On Thu, 11 Sep 2014, Maxim Ostapenko wrote: In general, when cc1 or cc1plus ICE-es, we try to reproduce the bug by running compiler 3 times and comparing stderr and stdout on each attempt with respective ones that were gotten as the result of previ

Re: [patch] avoid ICE due to NULL pointer dereference in ipa-comdats.c

2014-09-18 Thread Jeff Law
On 09/17/14 13:59, Sebastian Pop wrote: Hi, I got an ICE while building libstdc++ of a cross compiler x86 to aarch64. I have a testcase that ICEs on current GCC trunk. I was trying to painfully reduce it with creduce, and it is still several thousand lines of c++. Frustrated that it does not

Re: [PATCH, i386, Pointer Bounds Checker 32/x] Pointer Bounds Checker hooks for i386 target

2014-09-18 Thread Jeff Law
On 09/18/14 13:34, Uros Bizjak wrote: 2014-06-11 18:00 GMT+04:00 Ilya Enkovich : Hi, This patch adds i386 target hooks for Pointer Bounds Checker. Bootstrapped and tested on linux-x86_64. Thanks, Ilya -- gcc/ 2014-06-11 Ilya Enkovich * config/i386/i386.c: Include tree-iterator.h

Fix ODR checking ICE

2014-09-18 Thread Jan Hubicka
Hi, this patch fixes thinko in a condition guarding the ODR comparing patch. Boostrapped/regtested x86_64-linux, comitted. Honza PR lto/63298 * ipa-devirt.c (odr_subtypes_equivalent_p): Fix thinko in a condition. Index: ipa-devirt.c =

[jit] Add a libgccjit.pc file for use by pkg-config

2014-09-18 Thread David Malcolm
Committed to branch dmalcolm/jit: gcc/ChangeLog.jit: * Makefile.in (pkgconfigdir): New. (installdirs): Add creation of $(DESTDIR)$(pkgconfigdir). * configure.ac (gcc_version): Expose this value for use via AC_SUBST, since we need it within the new file libgccjit.pc.

Re: [PATCH][PING] Enable -fsanitize-recover for KASan

2014-09-18 Thread Joseph S. Myers
On Thu, 18 Sep 2014, Jakub Jelinek wrote: > Seems for -fdelete-null-pointer-checks we got it wrong too, > IMHO for -fsanitize={null,{,returns-}nonnull-attribute,undefined} > we want to disable it unconditionally, regardless of whether > that option appears on the command line or not. > And we hand

Re: PATCH [testsuite]: Update gcc.dg/pr61053.c for x32

2014-09-18 Thread Jeff Law
On 09/15/14 15:03, H.J. Lu wrote: Hi, gcc.dg/pr61053.c fails on x32, which has the same alignments for floating point types and the integer types with the same size as x86-64. This patch is needed for x32. Tested on ia32, x32 and x86-64. OK for trunk and 4.8 branch? Thanks. H.J. --- 2014-0

Re: Remove LIBGCC2_TF_CEXT target macro

2014-09-18 Thread Jeff Law
On 09/16/14 15:41, Joseph S. Myers wrote: This patch removes the (undocumented) LIBGCC2_TF_CEXT target macro, replacing it by -fbuilding-libgcc predefines (and thereby gets rid of another LIBGCC2_LONG_DOUBLE_TYPE_SIZE conditional, though some more patches are needed before that target macro can b

Re: [patch] flag .persistent.bss sections as bss

2014-09-18 Thread Jeff Law
On 09/16/14 10:54, Olivier Hainque wrote: Hello, Some target loaders (on VxWorks 653 for example) handle the ".persistent.bss" section as an area to reset during the initial startup of applications but not during warm restarts. We need these to be treated as bss sections by the compiler, in par

[PATCH]Add aarch64 to list of targets that support gold

2014-09-18 Thread Jing Yu
Hi, This patch changes top level configure to add aarch64 to list of targets that support gold. Have tested binutils with this patch on x86_64 and aarch64 platforms. OK for trunk? 2014-09-18 Jing Yu * configure.ac: Add aarch64 to list of targets that support gold. * configure: Reg

Re: [PATCH v2 AArch64]: Re: [PATCH AArch64]: Add constraint letter for stack_protect_test pattern.

2014-09-18 Thread Matthias Klose
Am 17.09.2014 um 15:14 schrieb Matthias Klose: > Am 17.09.2014 um 00:03 schrieb James Greenhalgh: >> If you have any other suggestions, or if "=&r" is actually correct and >> I am misreading the documentation please let me know. > > with this patch I see a lot of ICEs in the testsuite for test cas

Re: [PATCH] Add header guard to several header files.

2014-09-18 Thread Joseph S. Myers
On Fri, 19 Sep 2014, Kito Cheng wrote: > Hi Joseph: > > Here is updated patch and ChangeLog, > > However I don't have commit write yet, can you help me to commit it? thanks Committed. -- Joseph S. Myers jos...@codesourcery.com

Re: [Ping] Port of VTV for Cygwin and MinGW

2014-09-18 Thread Patrick Wollgast
Added Benjamin De Kosnik as a c++ runtime libs maintainer and Kai Tietz as Windows/Cygwin/MinGW maintainer. >> In changes to gcc/config/i386/cygwin.h mingw-w64.h and mingw32.h, you >> forgot to handle the "fvtable-verify=preinit" options. >> fvtable-veriy=preinit should cause vtv_start_preinit

Re: [patch i386]: Sibcall tail-call improvement and partial fix PR/60104

2014-09-18 Thread Iain Sandoe
Hello Kai, all, I am not concerned solely about Darwin here, certainly we can make a patch to "fix" the problem there. My concern is for the general state of this code: On 18 Sep 2014, at 22:26, Kai Tietz wrote: > it isn't true that I didn't replied to Iant. ( iains ;) ) > I did this on IRC

Re: [PATCH 2/2] Add patch for debugging compiler ICEs.

2014-09-18 Thread Joseph S. Myers
On Thu, 11 Sep 2014, Maxim Ostapenko wrote: > In general, when cc1 or cc1plus ICE-es, we try to reproduce the bug by running > compiler 3 times and comparing stderr and stdout on each attempt with > respective ones that were gotten as the result of previous compiler run (we > use temporary dump fi

Re: [patch i386]: Sibcall tail-call improvement and partial fix PR/60104

2014-09-18 Thread Kai Tietz
Here it is. Hope I didn't made here typos. Kai Index: config/i386/predicates.md === --- config/i386/predicates.md (Revision 215364) +++ config/i386/predicates.md (Arbeitskopie) @@ -73,8 +73,15 @@ ;; Return true if OP is a memor

Re: [PATCH, ARM] Tune thumb1_size_rtx_costs

2014-09-18 Thread Ramana Radhakrishnan
On Wed, Sep 10, 2014 at 6:30 AM, Zhenqiang Chen wrote: > Hi, > > Currently most thumb1_size_rtx_costs are "guessed" from performance view, > not size. > > The patch adjusts some of them according the instruction patterns defined in > thumb1.md. It also replaces several hard coded "4" to COSTS_N_IN

Re: [patch i386]: Sibcall tail-call improvement and partial fix PR/60104

2014-09-18 Thread FX
> See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61387 comment #9. It > doesn't apply anymore due current version of predicate was altered. > Nevertheless I could easily update patch for current trunk version. Sure, please send that and I’ll test it on darwin. Thanks, FX

Re: [patch i386]: Sibcall tail-call improvement and partial fix PR/60104

2014-09-18 Thread Kai Tietz
2014-09-18 23:35 GMT+02:00 FX : > Dear Kai, > >> it isn't true that I didn't replied to Iant. I did this on IRC. > > Good. I simply did not see any recent comment from you on the list, or > bugzilla. > >> As >> this code-path isn't prominent mark being Darwin-code - and please >> don't take me wr

Re: [PATCH] RTEMS: Update contrib/config-list.mk

2014-09-18 Thread Joel Sherrill
On 9/18/2014 6:51 AM, Jan-Benedict Glaw wrote: > On Wed, 2014-09-17 10:52:34 -0500, Joel Sherrill > wrote: >> On 9/17/2014 10:41 AM, Sebastian Huber wrote: >>> On 09/17/2014 04:45 PM, Jan-Benedict Glaw wrote: On Wed, 2014-09-17 15:37:32 +0200, Sebastian Huber wrote: >> contrib/Ch

Re: Patch to fix PR61360

2014-09-18 Thread Richard Sandiford
Vladimir Makarov writes: > On 09/18/2014 01:36 PM, Richard Sandiford wrote: >> Jakub Jelinek writes: >>> On Thu, Sep 18, 2014 at 12:04:30PM -0400, Vladimir Makarov wrote: The following patch fixes the PR. The details can be found on https://gcc.gnu.org/bugzilla/show_bug.cgi?id=613

Re: Extract and insert merging patch

2014-09-18 Thread Jeff Law
On 09/16/14 13:40, Andrew Pinski wrote: On Tue, Sep 16, 2014 at 4:40 AM, Petr Murzin wrote: Hi, This patch allows merging of extract and insert. Please have a look. 2014-09-16 Petr Murzin * simplify-rtx.c (simplify_ternary_operation): Allow extract and insert merging. Besides no testcase

Re: [Patch] Teach genrecog/genoutput that scratch registers require write constraint modifiers

2014-09-18 Thread Jeff Law
On 09/18/14 04:19, James Greenhalgh wrote: Hi, As discussed in https://gcc.gnu.org/ml/gcc-patches/2014-09/msg01334.html The construct (clobber (match_scratch 0 "r")) is invalid - operand 0 must be marked either write or read/write. Likewise (match_* 0 "&r") is invalid, marking an ope

Re: [patch] powerpc-vxworksmils port, variant of powerpc-vxworksae

2014-09-18 Thread Jeff Law
On 09/18/14 04:36, Olivier Hainque wrote: Hello, We have been maintaining a port to VxWorks MILS for powerpc for a while now and thought others might be interested. VxWorksMILS is very close to VxWorksAE, so the patch is pretty small. The main noticeable difference is that only the vthreads env

Re: [patch i386]: Sibcall tail-call improvement and partial fix PR/60104

2014-09-18 Thread FX
Dear Kai, > it isn't true that I didn't replied to Iant. I did this on IRC. Good. I simply did not see any recent comment from you on the list, or bugzilla. > As > this code-path isn't prominent mark being Darwin-code - and please > don't take me wrong, but it seems to be until now the only tar

Re: [patch] normalize the x86-vxworks port organization

2014-09-18 Thread Jeff Law
On 09/18/14 11:16, Olivier Hainque wrote: Hello, VxWorks ports typically come in two flavors: regular VxWorks and VxWorksAE (653). In most cases, /vxworks.h is used as a common configuration file for the two flavors and /vxworksae.h overrides/adds on top of that. There are also config/vx*.h shar

Re: [patch i386]: Sibcall tail-call improvement and partial fix PR/60104

2014-09-18 Thread Kai Tietz
Hi, it isn't true that I didn't replied to Iant. I did this on IRC. As I explained there already, this hunk about thunks is more consolidation of code-paths in that function, and not really part of a feature. As this code-path isn't prominent mark being Darwin-code - and please don't take me wr

Re: [PATCH] Fix PR63152

2014-09-18 Thread FX
> The following fixes PR63152 zeroing the data field only for allocatables, not > pointers. The benefit of the patch is a small speedup, and it avoids that > code starts to rely on behavior that is undefined in the standard. With this > patch, something like > > INTEGER, DIMENSION(:), POINTER :

Re: [patch i386]: Sibcall tail-call improvement and partial fix PR/60104

2014-09-18 Thread FX
> What I think we need is folks with an understanding of those systems to chime > in with the information Kai needs to fix the problem. I don't recall seeing > that, so if I missed it, feel free to point me to it. > > I'd rather not start going backwards and reverting because we simply haven't

Re: [Patch] Teach genrecog/genoutput that scratch registers require write constraint modifiers

2014-09-18 Thread Jeff Law
On 09/18/14 04:19, James Greenhalgh wrote: Hi, As discussed in https://gcc.gnu.org/ml/gcc-patches/2014-09/msg01334.html The construct (clobber (match_scratch 0 "r")) is invalid - operand 0 must be marked either write or read/write. Likewise (match_* 0 "&r") is invalid, marking an ope

[jit] Split docs out more formally into "Tutorial" vs "Topic Reference"

2014-09-18 Thread David Malcolm
Committed to branch dmalcolm/jit: gcc/jit/ChangeLog.jit: * docs/index.rst: Split index out into two new files... * docs/intro/index.rst: New file. * docs/topics/index.rst: New file. * docs/_build/texinfo/libgccjit.texi: Regenerate. --- gcc/jit/ChangeLog.jit

Re: Small fix for walking constructors

2014-09-18 Thread Jeff Law
On 09/18/14 13:01, Bernd Schmidt wrote: This fixes an issue on ptx where we fail to output a declaration for a variable. The testcase is c-torture/compile/pr34856.c, and the cause of the problem is that the variable g is never inserted into the varpool, which is where a future patch will look for

Re: [PR libfortran/62768] Handle filenames with embedded nulls

2014-09-18 Thread Hans-Peter Nilsson
On Thu, 18 Sep 2014, Janne Blomqvist wrote: > > If you look back at the patch I posted, there's a > > typo. :-} Duly warned about, but I'd rather expect the build to > > fail. > > Yes, strange that it didn't fail. There's no prototype for cf_fstrcpy, > and since we use std=gnu11 prototypes should

Re: [PATCH][MIPS] Fix ICE in bitmap routines with LRA and inline assembly language

2014-09-18 Thread Eric Christopher
On Thu, Sep 18, 2014 at 1:11 PM, Eric Christopher wrote: > > > > On Wed, Sep 10, 2014 at 3:39 AM, Matthew Fortune > wrote: >> >> > The patch is ok to commit. I think I made a typo as other analogous >> > code contains '|='. >> > >> > Thanks for fixing this, Robert. >> >> The LRA part of this is

Re: [debug-early] Disable removal of DW_tags* while generating subprogram DIEs

2014-09-18 Thread Jason Merrill
On 09/18/2014 03:25 PM, Aldy Hernandez wrote: This patch disables removing of DW_AT_{declaration,object_pointer,formal_parameter} tags while generating a DIE for subprograms. Now that we generate dwarf info early, we will always have an old_die the second time around. I see no need to remove t

Re: [PATCH, i386, Pointer Bounds Checker 33/x] MPX ABI

2014-09-18 Thread Uros Bizjak
Hello! > 2014-06-11 Ilya Enkovich > > * config/i386/i386.c (ix86_option_override_internal): Do not > support x32 with MPX. > is not available. > (init_cumulative_args): Init stdarg, bnd_regno, bnds_in_bt > and force_bnd_pass. > (function_arg_advan

Patch to fix PR63285

2014-09-18 Thread Vladimir Makarov
The following patch fixes PR63285. The details can be found on https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63285 The patch was bootstrapped and tested on x86/x86-64. Committed as rev 215364. 2014-09-18 Vladimir Makarov PR debug/63285 * haifa-sched.c (schedule_block): Adv

[PATCH 4/4] [AARCH64,NEON] Fix unnecessary moves in vst[234]q_* intrinsics

2014-09-18 Thread Charles Baylis
This patch improves code generation of vst[234]q_* intrinsics by avoiding use of the __builtin_aarch64_set_qreg_* builtins to generate a temporary __builtin_aarch64_simd_XX variable. Instead, a union is used for type-punning, which avoids generation of some unnecessary move instructions. This idiom

[PATCH 3/4] [AARCH64,NEON] Fix unnecessary moves in vld[234]q_* intrinsics

2014-09-18 Thread Charles Baylis
This patch improves code generation of vld[234]q_* intrinsics by avoiding use of the __builtin_aarch64_get_qreg_* builtins to generate a temporary result variable. Instead, a union is used for type-punning, which avoids generation of some unnecessary move instructions. This idiom is already used in

[PATCH 1/4] [AARCH64,NEON] Add patterns + builtins for vld[234](q?)_lane_* intrinsics

2014-09-18 Thread Charles Baylis
This patch adds new patterns and builtins to represent single lane structure loads instructions, which will be used to implement the vld[234](q?)_lane_* intrinsics. Tested (with the rest of the patch series) with make check on aarch64-oe-linux with qemu, and also causes no regressions in clyon's N

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

2014-09-18 Thread Charles Baylis
This patch replaces the inline assembler implementations of the vld[234](q?)_lane_* intrinsics with new versions which exploit the new builtin functions added in patch 1. Tested (with the rest of the patch series) with make check on aarch64-oe-linux with qemu, and also causes no regressions in cly

[PATCH 0/4] [AARCH64,NEON] Improve various NEON load/store intrinsics

2014-09-18 Thread Charles Baylis
This patch series improves the code generation for NEON structure loads and stores. Tested with make check on aarch64-oe-linux with qemu, and also passes clyon's NEON intrinsics tests. Charles Baylis (4): [AARCH64,NEON] Add patterns + builtins for vld[234](q?)_lane_* intrinsics [AARCH64,N

Re: [patch] rename DECL_ABSTRACT to DECL_ABSTRACT_P

2014-09-18 Thread Aldy Hernandez
Yeah, sure, either way it's a good cleanup ;). No strong opinions. Though I think true/false are the way we want folks to write new code. Given that's the long term direction, might as well fix that nit for DECL_ABSTRACT_P. Alright... fixed. OK? commit 276bf40161769e0485854576073aff9f35b16

Re: [PATCH, i386, Pointer Bounds Checker 32/x] Pointer Bounds Checker hooks for i386 target

2014-09-18 Thread Uros Bizjak
2014-06-11 18:00 GMT+04:00 Ilya Enkovich : > Hi, > > This patch adds i386 target hooks for Pointer Bounds Checker. > > Bootstrapped and tested on linux-x86_64. > > Thanks, > Ilya > -- > gcc/ > > 2014-06-11 Ilya Enkovich > > * config/i386/i386.c: Include tree-iterator.h. > (ix86_f

[debug-early] Disable removal of DW_tags* while generating subprogram DIEs

2014-09-18 Thread Aldy Hernandez
This patch disables removing of DW_AT_{declaration,object_pointer,formal_parameter} tags while generating a DIE for subprograms. Now that we generate dwarf info early, we will always have an old_die the second time around. I see no need to remove the aforementioned tags, only to create them a

Re: [patch] rename DECL_ABSTRACT to DECL_ABSTRACT_P

2014-09-18 Thread Jeff Law
On 09/18/14 10:33, Marek Polacek wrote: On Thu, Sep 18, 2014 at 10:30:30AM -0600, Aldy Hernandez wrote: - DECL_ABSTRACT (fn) = 1; + DECL_ABSTRACT_P (fn) = 1; It'd probably make sense to use 'true' now. I thought about it, but I wanted to change as little as possible, plus I wanted to fol

Re: [gomp4] various OpenACC/PTX built-ins and a reduction tweak

2014-09-18 Thread Cesar Philippidis
On 09/18/2014 11:43 AM, Thomas Schwinge wrote: > Hi Cesar! > > On Tue, 16 Sep 2014 17:32:54 -0700, Cesar Philippidis > wrote: >> The patch introduces the following OpenACC/PTX-specific built-ins: >> >> * GOACC_ntid >> * GOACC_tid >> * GOACC_nctaid >> * GOACC_ctaid >> * acc_on_device >>

Small fix for walking constructors

2014-09-18 Thread Bernd Schmidt
This fixes an issue on ptx where we fail to output a declaration for a variable. The testcase is c-torture/compile/pr34856.c, and the cause of the problem is that the variable g is never inserted into the varpool, which is where a future patch will look for references to variables not defined i

Re: ptx preliminary rtl patches [3/4]

2014-09-18 Thread Bernd Schmidt
On 09/12/2014 10:04 AM, Richard Biener wrote: On Thu, Sep 11, 2014 at 6:36 PM, Bernd Schmidt wrote: I strongly disagree. It's the same as for any other integer - there's one sign bit, and since there aren't any other bits, the number of sign bit copies is always exactly 1. I agree about that.

[jit] Add the generated texinfo to the source tree

2014-09-18 Thread David Malcolm
Committed to branch dmalcolm/jit: Sphinx is not a blessed dependency, so checkin a copy of the built .texi file for people who want .texi docs without needing to install sphinx. This is generated by running "make texinfo" in the gcc/jit/docs directory. gcc/jit/ChangeLog.jit: * docs/_buil

Re: [PATCH] Add v850-rtems to contrib/config-list.mk

2014-09-18 Thread Joel Sherrill
Thanks. Committed to 4.9 and head. --joel On 9/18/2014 1:28 PM, Jeff Law wrote: > On 09/18/14 09:31, Joel Sherrill wrote: >> OK to commit? >> >> 2014-09-18 Joel Sherrill >> >> * config-list.mk (LIST): Add v850-rtems. > Yes. > > jeff > -- Joel Sherrill, Ph.D. Director of Re

Re: parallel check output changes?

2014-09-18 Thread Segher Boessenkool
On Thu, Sep 18, 2014 at 07:36:09PM +0200, Jakub Jelinek wrote: > Segher on IRC mentioned that changing result_re in dg-extract-results.py > should help here (or disabling the python version, *.sh version should > sort everything). I am testing a patch that is just diff --git a/contrib/dg-extract

Re: [gomp4] various OpenACC/PTX built-ins and a reduction tweak

2014-09-18 Thread Thomas Schwinge
Hi Cesar! On Tue, 16 Sep 2014 17:32:54 -0700, Cesar Philippidis wrote: > The patch introduces the following OpenACC/PTX-specific built-ins: > > * GOACC_ntid > * GOACC_tid > * GOACC_nctaid > * GOACC_ctaid > * acc_on_device > * GOACC_get_thread_num > * GOACC_get_num_threads > > Of

Re: [PATCH] Add v850-rtems to contrib/config-list.mk

2014-09-18 Thread Jeff Law
On 09/18/14 09:31, Joel Sherrill wrote: OK to commit? 2014-09-18 Joel Sherrill * config-list.mk (LIST): Add v850-rtems. Yes. jeff

Re: [patch] update comments on *_ultimate_origin

2014-09-18 Thread Jeff Law
On 09/18/14 10:42, Aldy Hernandez wrote: output_inline_function was removed in tree-ssa times, no sense referencing it a decade later. I still see DECL_ABSTRACT_ORIGIN pointing to itself in some instances, though I haven't tracked down where, so I assume we still need the functionality, just not

Re: [PATCH] Put all constants last in tree_swap_operands_p, remove odd -Os check

2014-09-18 Thread Andrew Pinski
On Thu, Sep 18, 2014 at 9:44 AM, Alan Lawrence wrote: > We've been seeing errors using aarch64-none-linux-gnu gcc to build the > 403.gcc benchmark from spec2k6, that we've traced back to this patch. The > error looks like: > > /home/alalaw01/bootstrap_richie/gcc/xgcc > -B/home/alalaw01/bootstrap_r

[jit] Use the "pyramid" theme for generated HTML docs

2014-09-18 Thread David Malcolm
Committed to branch dmalcolm/jit: The default Sphinx theme is perhaps a little dated; switch to a non-default one. The "pyramid" one is clean and attractive IMHO. I've updated the prebuilt docs currently at: https://dmalcolm.fedorapeople.org/gcc/libgccjit-api-docs/ to use the new theme. gcc/j

Re: Patch to fix PR61360

2014-09-18 Thread Vladimir Makarov
On 09/18/2014 01:36 PM, Richard Sandiford wrote: > Jakub Jelinek writes: >> On Thu, Sep 18, 2014 at 12:04:30PM -0400, Vladimir Makarov wrote: >>> The following patch fixes the PR. The details can be found on >>> >>> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61360 >>> >>> The patch was bootstra

[gomp4] OpenACC acc_on_device (was: various OpenACC/PTX built-ins and a reduction tweak)

2014-09-18 Thread Thomas Schwinge
Hi! Here is my OpenACC acc_on_device patch, in a more complete form, with test cases and all that. Thanks, Cesar, for getting the ball rolling! On Wed, 17 Sep 2014 10:49:54 +0200, Jakub Jelinek wrote: > On Wed, Sep 17, 2014 at 10:44:12AM +0200, Tobias Burnus wrote: > > Cesar Philippidis wrote:

Re: [PATCH][PING] Keep patch file permissions in mklog

2014-09-18 Thread Diego Novillo
On Thu, Sep 18, 2014 at 10:56 AM, Yury Gribov wrote: > > On 08/04/2014 12:14 PM, Tom de Vries wrote: >> >> On 04-08-14 08:45, Yury Gribov wrote: >>> >>> Thanks! My 2 (actually 4) cents below. >>> >> >> Hi Yuri, >> >> thanks for the review. >> >>> > +if ($#ARGV == 1 && ("$ARGV[0]" eq "-i" || "$ARG

Re: Patch to fix PR61360

2014-09-18 Thread Richard Sandiford
Jakub Jelinek writes: > On Thu, Sep 18, 2014 at 12:04:30PM -0400, Vladimir Makarov wrote: >> The following patch fixes the PR. The details can be found on >> >> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61360 >> >> The patch was bootstrapped and tested on x86/x86-64. >> >> Committed as rev.

Re: parallel check output changes?

2014-09-18 Thread Jakub Jelinek
On Thu, Sep 18, 2014 at 07:32:00PM +0200, Bernd Schmidt wrote: > >hum. My 3rd run (which has no compilation change from the 2nd one) is > >different from both other runs :-P. I did tweak my -j parameter in > >the make check, but that is it. > > I'm also seeing this. Python 3.3.5 here. Segher o

Re: [PATCH, i386, Pointer Bounds Checker 31/x] Pointer Bounds Checker builtins for i386 target

2014-09-18 Thread Uros Bizjak
On Thu, Sep 18, 2014 at 3:47 PM, Ilya Enkovich wrote: > Thanks for your comments. Below is a fixed verison. > > Ilya > -- > 2014-09-17 Ilya Enkovich > > * config/i386/i386-builtin-types.def (BND): New. > (ULONG): New. > (BND_FTYPE_PCVOID_ULONG): New. > (VOID_FT

[jit] Markup fixes within documentation

2014-09-18 Thread David Malcolm
Committed to branch dmalcolm/jit: gcc/jit/ChangeLog.jit: * docs/intro/install.rst: Markup fixes. * docs/intro/tutorial01.rst: Likewise. * docs/intro/tutorial02.rst: Likewise. * docs/topics/contexts.rst: Likewise. * docs/topics/expressions.rst: Likewise.

Re: parallel check output changes?

2014-09-18 Thread Bernd Schmidt
On 09/18/2014 05:03 PM, Andrew MacLeod wrote: On 09/18/2014 09:05 AM, Andrew MacLeod wrote: On 09/18/2014 09:01 AM, Jakub Jelinek wrote: On Thu, Sep 18, 2014 at 08:56:50AM -0400, Andrew MacLeod wrote: Has the changes that have gone into the check parallelization made the .sum file non-determin

Re: [PATCH, i386, Pointer Bounds Checker 30/x] Size relocation

2014-09-18 Thread Uros Bizjak
On Thu, Sep 18, 2014 at 4:00 PM, Ilya Enkovich wrote: > On 17 Sep 20:51, Uros Bizjak wrote: >> On Wed, Sep 17, 2014 at 8:35 PM, Ilya Enkovich >> wrote: >> > On 16 Sep 12:22, Uros Bizjak wrote: >> >> On Tue, Sep 16, 2014 at 11:37 AM, Ilya Enkovich >> >> wrote: >> >> > 2014-09-16 13:08 GMT+04:00

Re: [PATCHv4] Vimrc config with GNU formatting

2014-09-18 Thread Segher Boessenkool
On Thu, Sep 18, 2014 at 12:40:08PM +0400, Yury Gribov wrote: > When typing 'make .local.vimrc' in GCC build directory one would expect > .local.vimrc to be created at the root of build directory, not srcdir. Yes, you would not expect it to do anything to your source dir, ever :-) > >> +" To enabl

Re: [PATCH i386 AVX512] [45/n] Add vshufpd insn patterns.

2014-09-18 Thread Uros Bizjak
On Thu, Sep 18, 2014 at 2:02 PM, Kirill Yukhin wrote: > Hello, > This patch supports AVX-512's vshufpd insns. > > Bootstrapped. > AVX-512* tests on top of patch-set all pass > under simulator. > > Is it ok for trunk? > > gcc/ > * config/i386/sse.md > (define_expand "avx_shufpd256")

Re: [PATCH i386 AVX512] [44/n] Add vsgufps insn patterns.

2014-09-18 Thread Uros Bizjak
On Thu, Sep 18, 2014 at 1:59 PM, Kirill Yukhin wrote: > Hello, > Patch in the bottom extends AVX-512 shufps. > > Bootstrapped. > AVX-512* tests on top of patch-set all pass > under simulator. > > Is it ok for trunk? > > gcc/ > * config/i386/sse.md > (define_expand "avx_shufps256"):

[patch] normalize the x86-vxworks port organization

2014-09-18 Thread Olivier Hainque
Hello, VxWorks ports typically come in two flavors: regular VxWorks and VxWorksAE (653). In most cases, /vxworks.h is used as a common configuration file for the two flavors and /vxworksae.h overrides/adds on top of that. There are also config/vx*.h shared by everybody. The x86 port departs from

Re: [PATCH i386 AVX512] [43/n] Add rest of vunpck[lh]ps.

2014-09-18 Thread Uros Bizjak
On Thu, Sep 18, 2014 at 1:54 PM, Kirill Yukhin wrote: > Hello, > This patch adds rest of unpack insn patterns. > > Bootstrapped. > AVX-512* tests on top of patch-set all pass > under simulator. > > Is it ok for trunk? > > gcc/ > * config/i386/sse.md > (define_insn "avx_unpckhps256"

Re: [PATCH i386 AVX512] [42/n] Add masked vunpck[lh]pd.

2014-09-18 Thread Uros Bizjak
On Thu, Sep 18, 2014 at 1:47 PM, Kirill Yukhin wrote: > Hello, > Patch in the bottom extends/adds patterns for masked unpack > instructions. > > Bootstrapped. > AVX-512* tests on top of patch-set all pass > under simulator. > > Is it ok for trunk? > > gcc/ > * config/i386/sse.md >

Re: Patch to fix PR61360

2014-09-18 Thread Vladimir Makarov
On 09/18/2014 12:10 PM, Jakub Jelinek wrote: > On Thu, Sep 18, 2014 at 12:04:30PM -0400, Vladimir Makarov wrote: >> The following patch fixes the PR. The details can be found on >> >> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61360 >> >> The patch was bootstrapped and tested on x86/x86-64. >> >

[committed] Fix Crash when OpenMP target's array section handling is used with templates (PR c++/63248)

2014-09-18 Thread Jakub Jelinek
On Wed, Sep 10, 2014 at 05:21:07PM +0200, Thomas Schwinge wrote: > Hi! > > On Wed, 10 Sep 2014 12:23:04 +0200, Jakub Jelinek wrote: > > On Wed, Sep 10, 2014 at 12:12:03PM +0200, Thomas Schwinge wrote: > > > Are the following issues known? > > > > No, please file a PR. > > Will do tomorrow. Her

[PATCH, rs6000] Rename GCC version in warning messages

2014-09-18 Thread Ulrich Weigand
Hello, the ABI warning messages I introduced in recent patches refer to a GCC version "4.10". As GCC has since adopted a new version naming scheme, this patch updates those messages to refer to "GCC 5" instead. Tested on powerpc64le-linux. OK for mainline? Bye, Ulrich ChangeLog: * c

Re: [PATCH] Put all constants last in tree_swap_operands_p, remove odd -Os check

2014-09-18 Thread Alan Lawrence
We've been seeing errors using aarch64-none-linux-gnu gcc to build the 403.gcc benchmark from spec2k6, that we've traced back to this patch. The error looks like: /home/alalaw01/bootstrap_richie/gcc/xgcc -B/home/alalaw01/bootstrap_richie/gcc -O3 -mcpu=cortex-a57.cortex-a53 -DSPEC_CPU_LP64

[patch] update comments on *_ultimate_origin

2014-09-18 Thread Aldy Hernandez
output_inline_function was removed in tree-ssa times, no sense referencing it a decade later. I still see DECL_ABSTRACT_ORIGIN pointing to itself in some instances, though I haven't tracked down where, so I assume we still need the functionality, just not the comment :). OK for mainline? Al

Re: [patch] rename DECL_ABSTRACT to DECL_ABSTRACT_P

2014-09-18 Thread Marek Polacek
On Thu, Sep 18, 2014 at 10:30:30AM -0600, Aldy Hernandez wrote: > > >>- DECL_ABSTRACT (fn) = 1; > >>+ DECL_ABSTRACT_P (fn) = 1; > > > >It'd probably make sense to use 'true' now. > > I thought about it, but I wanted to change as little as possible, plus I > wanted to follow the same style as wh

Re: [patch] rename DECL_ABSTRACT to DECL_ABSTRACT_P

2014-09-18 Thread Aldy Hernandez
- DECL_ABSTRACT (fn) = 1; + DECL_ABSTRACT_P (fn) = 1; It'd probably make sense to use 'true' now. I thought about it, but I wanted to change as little as possible, plus I wanted to follow the same style as what we've been doing for a lot of the _P macros: DECL_HAS_VALUE_EXPR_P (t) = 1;

Re: [PATCH] Add header guard to several header files.

2014-09-18 Thread Kito Cheng
Hi Joseph: Here is updated patch and ChangeLog, However I don't have commit write yet, can you help me to commit it? thanks btw, I has already signed FSF agreement:) 2014-09-19 Kito Cheng except.h: Fix header guard. addresses.h: Add missing header guard. cfghooks.h: Likewise.

Re: [patch] rename DECL_ABSTRACT to DECL_ABSTRACT_P

2014-09-18 Thread Marek Polacek
On Thu, Sep 18, 2014 at 10:11:24AM -0600, Aldy Hernandez wrote: > Similarly named DECL_ABSTRACT, DECL_ABSTRACT_ORIGIN, and DECL_ORIGIN are > somewhat confusing to my poor brain. Particularly annoying is DECL_ABSTRACT > which is actually a boolean, unlike the other two. > > Would it be OK to renam

[patch] rename DECL_ABSTRACT to DECL_ABSTRACT_P

2014-09-18 Thread Aldy Hernandez
Similarly named DECL_ABSTRACT, DECL_ABSTRACT_ORIGIN, and DECL_ORIGIN are somewhat confusing to my poor brain. Particularly annoying is DECL_ABSTRACT which is actually a boolean, unlike the other two. Would it be OK to rename it something more sensible like DECL_ABSTRACT_P? I know this is a l

Re: Patch to fix PR61360

2014-09-18 Thread Jakub Jelinek
On Thu, Sep 18, 2014 at 12:04:30PM -0400, Vladimir Makarov wrote: > The following patch fixes the PR. The details can be found on > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61360 > > The patch was bootstrapped and tested on x86/x86-64. > > Committed as rev. 215358. What effect does this

[jit] Build the example files from the documentation when running the testsuite

2014-09-18 Thread David Malcolm
Doing this caught missing "return" statements in the examples. This brings the result I see in jit.sum to: # of expected passes 4286 Committed to branch dmalcolm/jit: gcc/jit/ChangeLog.jit: * docs/examples/install-hello-world.c (main): Fix missing "return". * docs/exampl

Patch to fix PR61360

2014-09-18 Thread Vladimir Makarov
The following patch fixes the PR. The details can be found on https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61360 The patch was bootstrapped and tested on x86/x86-64. Committed as rev. 215358. 2014-09-18 Vladimir Makarov PR target/61360 * lra.c (lra): Call recog_init. 2014-

  1   2   >