Re: [PATCH 3/5] combine: Use insn_cost instead of pattern_cost everywhere

2017-08-02 Thread Segher Boessenkool
On Wed, Aug 02, 2017 at 10:40:32AM -0600, Jeff Law wrote: > On 07/31/2017 05:25 PM, Segher Boessenkool wrote: > > This changes combine to always use insn_cost, not pattern_cost. I don't > > intend to commit it like this: it calls recog more often than necessary, > > and it is very ugly (no, don't

Re: [C++ Patch] PR 71440 ("ICE on invalid C++ code in instantiate_type, at cp/class.c...")

2017-08-02 Thread Jason Merrill
OK. On Wed, Aug 2, 2017 at 5:15 PM, Paolo Carlini wrote: > Hi, > > by and large this issue seems already fixed in mainline, ie, we don't ICE > anymore during error recovery, but I noticed that we try to prettyprint > constructor and destructor as expressions and we end up with ugliness like: > >

Re: [RFC][PR 67336][PING^6] Verify pointers during stack unwind

2017-08-02 Thread Florian Weimer
* Jeff Law: > Something like setup a signal handler when we first start unwinding that > flags the error and tear it down when we're done unwinding?Obviously > we can't do setup/tear down each time for each address. Anyway, just > thinking outloud here... Linux doesn't have per-thread signal

[PATCH, AArch64] Add falkor pipeline description.

2017-08-02 Thread Jim Wilson
This adds a pipeline description for the Qualcomm Falkor core. This was tested with a bootstrap and make check. There were no regressions. This gives about 0.5% performance gain on SPEC CPU2006 on our internal tree, which has a few other patches that aren't in the FSF tree yet. OK? Jim

Re: [patch][Ping #3] PR80929: Realistic PARALLEL cost in seq_cost.

2017-08-02 Thread Segher Boessenkool
On Wed, Aug 02, 2017 at 12:17:44PM -0600, Jeff Law wrote: > So this has probably been hashed to death, but just a couple thoughts. > > I think realistically one has to look at the entirety of the PARALLEL to > get any reasonable costing. > > Summing the individual components is wrong. Taking the

Re: [RFC][PR 67336][PING^6] Verify pointers during stack unwind

2017-08-02 Thread H.J. Lu
On Wed, Aug 2, 2017 at 1:56 PM, Yury Gribov wrote: > On 02.08.2017 21:48, H.J. Lu wrote: >> >> On Wed, Aug 2, 2017 at 1:39 PM, Yury Gribov >> wrote: >>> >>> On 02.08.2017 20:02, Jeff Law wrote: On 08/02/2017 12:47 PM, Jakub Jelinek wrote: > > > On Wed, Aug 02, 2017 at 1

Re: [PATCH 1/1] sparc: support for -mmisalign in the SPARC M8

2017-08-02 Thread David Miller
From: Qing Zhao Date: Wed, 2 Aug 2017 14:41:51 -0500 > so, could you please specify what kind of side effects will have > when set STRICT_ALIGNMENT to true on TARGET_MISALIGN? Why don't you read the code rather than just relying upon what high level description is given by the documentation inst

[PATCH 5/6 v2] [i386] Modify SP realignment in ix86_expand_prologue, et. al.

2017-08-02 Thread Daniel Santos
My first version of this patch inited m->fs.sp_realigned_fp_last with the value of m->fs.sp_offset prior to performing the stack realignment. I had forgotten, however, that when we're saving GP regs using MOV that we delay SP modification as long as possible so that the value of m->fs.sp_offset at

Re: [PATCH, rs6000] Clean up capitalized diagnostic messages

2017-08-02 Thread Segher Boessenkool
Hi Bill, On Wed, Aug 02, 2017 at 10:29:20AM -0500, Bill Schmidt wrote: > I don't anticipate backporting any of this. Good :-) > @@ -6802,7 +6802,7 @@ altivec_resolve_overloaded_builtin (location_t loc > if (unsupported_builtin) >{ > const char *name = rs6000_overloaded_builtin

Re: [PING^4][PATCH v2] Generate reproducible output independently of the build-path

2017-08-02 Thread Ximin Luo
Jeff Law: > On 07/21/2017 10:15 AM, Ximin Luo wrote: >> (Please keep me on CC, I am not subscribed) >> >> >> Proposal >> >> >> This patch series adds a new environment variable BUILD_PATH_PREFIX_MAP. When >> this is set, GCC will treat this as extra implicit >> "-fdebug-prefix-map=$value"

Re: [RFC][PR 67336][PING^6] Verify pointers during stack unwind

2017-08-02 Thread Yury Gribov
On 02.08.2017 23:04, H.J. Lu wrote: On Wed, Aug 2, 2017 at 1:56 PM, Yury Gribov wrote: On 02.08.2017 21:48, H.J. Lu wrote: On Wed, Aug 2, 2017 at 1:39 PM, Yury Gribov wrote: On 02.08.2017 20:02, Jeff Law wrote: On 08/02/2017 12:47 PM, Jakub Jelinek wrote: On Wed, Aug 02, 2017 at 12:3

Re: [PING^4][PATCH v2] Generate reproducible output independently of the build-path

2017-08-02 Thread Yury Gribov
On 03.08.2017 3:06, Ximin Luo wrote: Jeff Law: On 07/21/2017 10:15 AM, Ximin Luo wrote: (Please keep me on CC, I am not subscribed) Proposal This patch series adds a new environment variable BUILD_PATH_PREFIX_MAP. When this is set, GCC will treat this as extra implicit "-fdebug-pref

Re: [PATCH] RTEMS: Add GCC Runtime Library Exception

2017-08-02 Thread Sebastian Huber
On 02/08/17 21:30, Jeff Law wrote: On 07/24/2017 12:03 AM, Sebastian Huber wrote: gcc/ PR libgcc/61152 * aarch64/rtems.h: Add GCC Runtime Library Exception. Format changes. * arm/rtems.h: Likewise. * bfin/rtems.h: Likewise. * i386/rtemself.h: Li

<    1   2