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] 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: [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: 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: [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

[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: 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.

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: [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 >>

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

[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, 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

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

[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

[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 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 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 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 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

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

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][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: [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: 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

[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: [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 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] 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 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] 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 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] 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] 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: 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 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: [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 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 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, 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 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 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 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: [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] 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: [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

[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] 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

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 [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: [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

[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.

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 =

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

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 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: 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 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: [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 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: [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, 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] 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: [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 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: [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 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 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 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: 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: [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

<    1   2