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

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

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

* 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

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

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

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

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 =

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

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

Re: [PATCH] Add missing edge probability in simd_clone_adjust

On 08/02/2017 06:29 PM, Jakub Jelinek wrote: On Wed, Aug 02, 2017 at 06:15:31PM +0200, Tom de Vries wrote: 2017-08-02 Tom de Vries * omp-simd-clone.c (simd_clone_adjust): Add missing edge probability. I think that is not the right probability, when using masked

Re: [PATCH] fix the handling of string precision in pretty printer (PR 81586)

On 07/27/2017 05:29 PM, Martin Sebor wrote: > The pretty printer treats precision in %s directives as a request > to print exactly as many characters from the string argument when > what precision normally (in C) means is the maximum number of > characters to read from the string. It doesn't mean

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

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

[PATCH, AArch64] Add falkor pipeline description.

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 1/1] sparc: support for -mmisalign in the SPARC M8

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

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

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] Make mempcpy more optimal (PR middle-end/70140).

On 08/01/2017 09:50 PM, Jakub Jelinek wrote: > On Thu, Jul 20, 2017 at 08:59:29AM +0200, Martin Liška wrote: >> Hello. >> >> Following patch does sharing of expansion for mem{p,}cpy and also strpcy >> (with a known constant as source) >> so that we use same type of expansion (direct insns

Re: [PATCH] Make mempcpy more optimal (PR middle-end/70140).

On 08/02/2017 09:16 AM, Jakub Jelinek wrote: > On Wed, Aug 02, 2017 at 09:13:40AM +0200, Martin Liška wrote: >> On 08/01/2017 09:50 PM, Jakub Jelinek wrote: >>> On Thu, Jul 20, 2017 at 08:59:29AM +0200, Martin Liška wrote: Hello. Following patch does sharing of expansion for

[C++ committed] Fix ICE with -Wshadow=compatible-local (PR c++/81640)

Hi! The r250440 change removed two spots that ensured that for !CLASS_TYPE_P (type) it returned -1 (from the idx routines that are gone now) and eventually NULL from the non-idx ones. I've bootstrapped/regtested a change with various assertions that showed that except for this testcase and spot

Re: [PATCH 1/2] add unique_ptr header

On Tue, Aug 1, 2017 at 4:40 PM, David Malcolm wrote > On Mon, 2017-07-31 at 19:46 -0400, tbsaunde+...@tbsaunde.org wrote: >> From: Trevor Saunders >> >> For most of the history of this see >>

[PATCH] More PRE TLC

Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2017-08-02 Richard Biener * tree-ssa-pre.c (bitmap_insert_into_set_1): Remove and inline into ... (bitmap_insert_into_set): ... this. Index: gcc/tree-ssa-pre.c

Re: [PATCH] Make mempcpy more optimal (PR middle-end/70140).

On Wed, Aug 02, 2017 at 09:13:40AM +0200, Martin Liška wrote: > On 08/01/2017 09:50 PM, Jakub Jelinek wrote: > > On Thu, Jul 20, 2017 at 08:59:29AM +0200, Martin Liška wrote: > >> Hello. > >> > >> Following patch does sharing of expansion for mem{p,}cpy and also strpcy > >> (with a known constant

Re: [PATCH] Fix PR81354 (rewrite gimple_split_edge)

On Tue, Aug 1, 2017 at 5:56 PM, Bill Schmidt wrote: > On Aug 1, 2017, at 8:50 AM, Bill Schmidt wrote: >> >> On Aug 1, 2017, at 7:44 AM, Bill Schmidt wrote: >>> On Aug 1, 2017, at 3:46 AM, Richard

Re: [PATCH] Fix target attribute handling (PR c++/81355).

On 07/28/2017 05:13 AM, Martin Liška wrote: > Hello. > > Following patch skips empty strings in 'target' attribute. > Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. > > Ready to be installed? > Martin > > gcc/ChangeLog: > > 2017-07-13 Martin Liska

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

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:38:13PM -0600, Jeff Law wrote: On 07/17/2017 01:23 AM, Yuri Gribov wrote: I've rebased the previous patch to trunk per Andrew's suggestion. Original patch

Re: [PATCH 1/2] Introduce testsuite support to run Python tests

On 07/26/2017 10:48 AM, David Malcolm wrote: > On Wed, 2017-07-26 at 18:35 +0200, Pierre-Marie de Rodat wrote: >> On 07/26/2017 06:25 PM, David Malcolm wrote: >>> str.format was introduced in Python 2.6, so presumably the minimum >>> python 2 version here is at least 2.6+; for Python 3 I believe

Re: [PATCH] Add missing edge probability in simd_clone_adjust

On Wed, Aug 02, 2017 at 08:19:41PM +0200, Tom de Vries wrote: > 2017-08-02 Tom de Vries > > * omp-simd-clone.c (simd_clone_adjust): Add missing edge probability. Ok, thanks. > diff --git a/gcc/omp-simd-clone.c b/gcc/omp-simd-clone.c > index a1a563e..fbb122c

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

On Wed, Aug 02, 2017 at 12:38:13PM -0600, Jeff Law wrote: > On 07/17/2017 01:23 AM, Yuri Gribov wrote: > > I've rebased the previous patch to trunk per Andrew's suggestion. > > Original patch description/motivation/questions are in > > https://gcc.gnu.org/ml/gcc-patches/2017-06/msg01869.html > Is

Re: [PATCH][2/2] early LTO debug, main part

On Wed, Aug 2, 2017 at 6:35 AM, Richard Biener wrote: > On Wed, 2 Aug 2017, Jason Merrill wrote: > >> On 05/19/2017 06:42 AM, Richard Biener wrote: >> > + /* ??? In some cases the C++ FE (at least) fails to >> > +set DECL_CONTEXT properly. Simply globalize stuff

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

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: Likewise. > * lm32/rtems.h:

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

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:38:13PM -0600, Jeff Law wrote: On 07/17/2017 01:23 AM, Yuri Gribov wrote: >

Re: GCC 7.2 Status Report (2017-08-02)

On Wed, Aug 2, 2017 at 4:17 AM, Richard Biener wrote: > > Status > == > > The GCC 7 branch is now frozen for the upcoming release candidate > and release. All changes require release manager approval. Whoops, I committed a patch to the GCC 7 branch this morning (my time)

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

On Aug 2, 2017, at 11:04 AM, Joseph Myers wrote: > > On Wed, 2 Aug 2017, Jakub Jelinek wrote: > >> On Wed, Aug 02, 2017 at 10:29:20AM -0500, Bill Schmidt wrote: >>> --- gcc/config/rs6000/rs6000.c (revision 250791) >>> +++ gcc/config/rs6000/rs6000.c (working

Re: [PATCH v2][RFC] Canonize names of attributes.

OK. On Thu, Jul 13, 2017 at 9:48 AM, Martin Liška wrote: > On 07/11/2017 05:52 PM, Jason Merrill wrote: >> On Tue, Jul 11, 2017 at 9:37 AM, Martin Liška wrote: >>> On 07/03/2017 11:00 PM, Jason Merrill wrote: On Mon, Jul 3, 2017 at 5:52 AM, Martin Liška

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

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: error: taking address of constructor ‘((C*)this)->*A::__ct ’

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

On 07/17/2017 01:23 AM, Yuri Gribov wrote: > Hi all, > > I've rebased the previous patch to trunk per Andrew's suggestion. > Original patch description/motivation/questions are in > https://gcc.gnu.org/ml/gcc-patches/2017-06/msg01869.html Is his stuff used for exception handling? If so, doesn't

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

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" > command-line

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

Hi, David, thanks a lot for your comment. see my reply below > STRICT_ALIGNMENT has a lot of implications. from the definition of STRICT_ALIGNMENT: /* Set this nonzero if move instructions will actually fail to work when given unaligned data. */ #define STRICT_ALIGNMENT 1 for

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

On 06/29/2017 02:51 AM, Georg-Johann Lay wrote: > On 28.06.2017 22:18, Wilco Dijkstra wrote: >> Georg-Johann Lay wrote: >> >> @@ -5300,6 +5300,9 @@ seq_cost (const rtx_insn *seq, bool spee >> set = single_set (seq); >> if (set) >> cost += set_rtx_cost (set, speed); >> +

Re: [PATCH 1/3] Use BUILD_PATH_PREFIX_MAP envvar for debug-prefix-map

On 07/21/2017 10:15 AM, Ximin Luo wrote: > Define the BUILD_PATH_PREFIX_MAP environment variable, and treat it as > implicit > -fdebug-prefix-map CLI options specified before any explicit such options. > > Much of the generic code for applying and parsing prefix-maps is implemented > in >

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

On 08/02/2017 12:47 PM, Jakub Jelinek wrote: > On Wed, Aug 02, 2017 at 12:38:13PM -0600, Jeff Law wrote: >> On 07/17/2017 01:23 AM, Yuri Gribov wrote: >>> I've rebased the previous patch to trunk per Andrew's suggestion. >>> Original patch description/motivation/questions are in >>>

Re: [PATCH] Fix target attribute handling (PR c++/81355).

On 08/02/2017 01:04 PM, Jeff Law wrote: On 07/28/2017 05:13 AM, Martin Liška wrote: Hello. Following patch skips empty strings in 'target' attribute. Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. Ready to be installed? Martin gcc/ChangeLog: 2017-07-13 Martin

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

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:38:13PM -0600, Jeff Law wrote: On 07/17/2017 01:23 AM, Yuri

[RFC] [PATCH] Patch candidate for PR81657

Hi. I've just sketched a patch for the PR. Well I'm not fully happy about complexity of emit_block_move_hints function and I need to add another logic. Any ideas how to make it more transparent? Maybe split it into analysis function that will return decision and second that will just do the

Re: [PATCH] Make mempcpy more optimal (PR middle-end/70140).

On Wed, Aug 2, 2017 at 8:26 AM, Martin Liška wrote: > On 08/02/2017 09:16 AM, Jakub Jelinek wrote: >> On Wed, Aug 02, 2017 at 09:13:40AM +0200, Martin Liška wrote: >>> On 08/01/2017 09:50 PM, Jakub Jelinek wrote: On Thu, Jul 20, 2017 at 08:59:29AM +0200, Martin Liška wrote:

Re: [PATCHv2][PING^2][PR 56727] Bypass PLT for recursive calls

Hi, > On Mon, Jul 17, 2017 at 10:27 AM, Jan Hubicka wrote: > >> Hi all, > >> > >> This is a new version of previous patch > >> (https://gcc.gnu.org/ml/gcc-patches/2017-07/msg00020.html), fixed > >> after Rainer's remarks. > > Hi, > > the patch looks OK, but I wonder why you

Re: [PATCH][PR 59521] Respect probabilities when expanding switch statement

Hello, sorry for not responding for a while. Martin Liska has patch to move switch expansion to gimple level that will likely simplify the code combinatoin. > > combine_predictions_for_bb calculates final probability for edges of > if-else or switch statements. > > For if-elses this is done by

[PATCH] Tweak tree-ssa/pr81588.c testcase (PR tree-optimization/81655)

Hi! I have an improvement that handles tree-ssa/pr81588.c optimization even on branch cost 1, but as it is effectively an extension to the range var_bound optimization, I'm not sure it should be backported. Therefore, this patch treats this testcase similarly to other testcases that rely on

Re: [PATCH] Add macro DISABLE_COPY_AND_ASSIGN

On Wed, Jul 26, 2017 at 9:55 AM, Yao Qi wrote: > On 17-07-19 10:30:45, Yao Qi wrote: >> We have many classes that copy cotr and assignment operator are deleted >> in different projects, gcc, gdb and gold. So this patch adds a macro >> to do this, and replace these existing

Re: [PATCH] Make mempcpy more optimal (PR middle-end/70140).

On 08/02/2017 11:45 AM, Bin.Cheng wrote: > Hi Martin, > With r250771, GCC failed to build glibc for arm/aarch64 linux cross toolchain: Hi. Sorry for the breakage, I accidentally installed wrong version of patch. Should be fixed in r250789. M.

Re: [PATCH][PR 59521] Respect probabilities when expanding switch statement

On 08/02/2017 11:53 AM, Jan Hubicka wrote: > Hello, > sorry for not responding for a while. Martin Liska has patch to move switch > expansion to gimple level that will likely simplify the code combinatoin. Hello. Yep, will land today to gcc-patches mailing list. > >> >>

Re: [PATCH][PR 59521] Respect probabilities when expanding switch statement

On 08/02/2017 12:52 PM, Yury Gribov wrote: > On Wed, Aug 2, 2017 at 11:42 AM, Martin Liška > wrote: > > On 08/02/2017 11:53 AM, Jan Hubicka wrote: > > Hello, > > sorry for not responding for a while. Martin Liska has patch to move > switch >

Re: [GCC]][AArch64][PATCH] Fix big endian float immediate moves.

On Tue, Aug 01, 2017 at 04:05:13PM +0100, Tamar Christina wrote: > Hi All, > > real_to_target seems to return the order of the elements in the array > differently depending on the endiannes. This undoes the endianness when > combining the values back to a HOST_WIDE_INT. > > Regtested on

C PATCH to fix a crash on invalid (PR c/81289)

The compiler could crash in c_parser_unary_expression in case CPP_MULT: 7050 exp_loc = c_parser_peek_token (parser)->location; 7051 op = c_parser_cast_expression (parser, NULL); 7052 finish = op.get_finish (); ... 7054 location_t combined_loc = make_location

GCC 7.2 Status Report (2017-08-02)

Status == The GCC 7 branch is now frozen for the upcoming release candidate and release. All changes require release manager approval.

Re: [PATCH] Tweak tree-ssa/pr81588.c testcase (PR tree-optimization/81655)

On Wed, Aug 2, 2017 at 12:11 PM, Jakub Jelinek wrote: > Hi! > > I have an improvement that handles tree-ssa/pr81588.c optimization > even on branch cost 1, but as it is effectively an extension to the > range var_bound optimization, I'm not sure it should be backported. > >

Re: [PATCH] toplev: avoid recursive emergency_dump_function

Hello, On Thu, 20 Jul 2017, Alexander Monakov wrote: > Segher pointed out on IRC that ICE reporting with dumps enabled got worse: > if emergency_dump_function itself leads to an ICE (e.g. by segfaulting), > nested ICE reporting will invoke emergency_dump_function in exactly the > same context,

Re: [PATCH][2/2] early LTO debug, main part

On Wed, 2 Aug 2017, Jason Merrill wrote: > On 05/19/2017 06:42 AM, Richard Biener wrote: > > + /* ??? In some cases the C++ FE (at least) fails to > > +set DECL_CONTEXT properly. Simply globalize stuff > > +in this case. For example > > +__dso_handle created via

Re: [PATCH] Make mempcpy more optimal (PR middle-end/70140).

On Wed, Aug 2, 2017 at 10:54 AM, Martin Liška wrote: > On 08/02/2017 11:45 AM, Bin.Cheng wrote: >> Hi Martin, >> With r250771, GCC failed to build glibc for arm/aarch64 linux cross >> toolchain: > > Hi. > > Sorry for the breakage, I accidentally installed wrong version of patch.

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

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

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

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

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

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:

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

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

Re: [PATCH v2][RFC] Canonize names of attributes.

On Thu, 13 Jul 2017, Martin Liška wrote: > +/* For a given IDENTIFIER_NODE, strip leading and trailing '_' characters > + so that we have a canonical form of attribute names. */ > + > +static inline tree > +canonicalize_attr_name (tree attr_name) > +{ > + const size_t l = IDENTIFIER_LENGTH

Re: C PATCH to further improve diagnostic for -Wsign-compare (PR c/81417)

On Tue, Aug 01, 2017 at 04:48:01PM -0400, David Malcolm wrote: > I'm wondering if the messages could use a slight rewording, to give a > clue to the user about the reason *why* the expression has changed > signedness. The old message "signed and unsigned type in conditional > expression" gave the

Re: [PATCH] Fix PR bootstrap/81638

To fix the false-positive uninitialized warning, this patch initializes the "incl" variable. * xcoff.c (xcoff_process_linenos): Initialize incl to NULL. Index: xcoff.c === --- xcoff.c (revision 250806) +++ xcoff.c (working

[testsuite, committed] Use relative line number in gcc.dg/Walloca-14.c

[ was: Re: Avoid generating useless range info ] On 06/28/2017 09:56 AM, Aldy Hernandez wrote: void *q = __builtin_alloca (p); /* { dg-warning "passing argument 1" } */ + /* { dg-warning "unbounded use of 'alloca'" "unbounded" { target *-*-* } 11 } */ Hi, Please use relative line

PATCH for base_pool_allocator (PR other/81667)

This PR points out that m_elt_size was left uninitialized in the member initializer list for base_pool_allocator. I think it makes sense to initialize it like this. Bootstrapped/regtested on x86_64-linux, ok for trunk? 2017-08-02 Marek Polacek PR other/81667

Re: [PATCH] Fix PR81354 (rewrite gimple_split_edge)

On Aug 2, 2017, at 3:09 AM, Richard Biener wrote: > > On Tue, Aug 1, 2017 at 5:56 PM, Bill Schmidt > wrote: >> On Aug 1, 2017, at 8:50 AM, Bill Schmidt wrote: >>> >>> On Aug 1, 2017, at 7:44 AM, Bill Schmidt

Re: PATCH for base_pool_allocator (PR other/81667)

On Wed, Aug 2, 2017 at 2:45 PM, Marek Polacek wrote: > This PR points out that m_elt_size was left uninitialized in the member > initializer list for base_pool_allocator. I think it makes sense to > initialize > it like this. > > Bootstrapped/regtested on x86_64-linux, ok

[patch, fortran] Fix PR 60355, missing error for BIND(C) outside module scope

Hello world, the attached patch is a bit smaller than it looks, because most of it is due to reformatting a large comment. It is rather simple - checking for an incorrectly placed BIND(C) variable was sometimes not done because the test was mixed in with other tests where implicitly typed

[nvptx, committed] Add missing probabilities in nvptx_lock{less,full}_update

[ was: Re: [PATCH] update edge profile info in nvptx.c ] On 07/13/2017 06:53 PM, Cesar Philippidis wrote: The recent basic block profiling changes broke a couple of libgomp OpenACC execution tests involving reductions with nvptx offloading. For gang and worker reductions, the nvptx BE updates

[PATCH] Fix PR bootstrap/81638

To fix the false-positive uninitialized warning, this patch initializes the "incl" variable.

Re: c-family PATCH to improve and simplify -Wmultistatement-macros (PR c/81448, c/81306)

On Thu, 27 Jul 2017, Marek Polacek wrote: > Bootstrapped/regtested on x86_64-linux and ppc64le-linux, ok for trunk? > > 2017-07-27 Marek Polacek > > PR c/81448 > PR c/81306 > * c-warn.c (warn_for_multistatement_macros): Prevent bogus > warnings.

Re: C PATCH to fix a crash on invalid (PR c/81289)

On Wed, 2 Aug 2017, Marek Polacek wrote: > 2017-08-02 Marek Polacek > > PR c/81289 > * c-parser.c (c_parser_unary_expression): Use set_error. > > * gcc.dg/noncompile/pr81289.c: New test. OK. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH][RFC] Make expansion of balanced binary trees of switches on tree level.

On 08/02/2017 01:51 PM, Richard Biener wrote: > On Wed, Aug 2, 2017 at 1:20 PM, Martin Liška wrote: >> Hello. >> >> After some discussions with Honza, I've decided to convert current code in >> stmt.c that >> is responsible for switch expansion. More precisely, I would like to

Re: [PATCH][RFC] Make expansion of balanced binary trees of switches on tree level.

On Wed, 2017-08-02 at 13:20 +0200, Martin Liška wrote: > Hello. > > After some discussions with Honza, I've decided to convert current > code in stmt.c that > is responsible for switch expansion. More precisely, I would like to > convert the code > to expand gswitch statements on tree level.

[PATCH, i386]: Fix PR81644, ICE in rtl_verify_bICE in rtl_verify_bb_insn, BBRO pass duplicates BB that ends with flow control insnb_insn, BBRO pass duplicates BB that ends with flow control insn

It turned out we can't simply emit trap insn during epilogue expansion, since epilogue is expanded on FALLTHRU edge to EXIT BB. Later passes rightfully choke on control flow insn in BB with FALTHRU exit edge. So, instead of using TRAP_IF in the prologue, use UNSPEC_VOLATILE that directly emits

Re: [PATCH], PR target/81593, Optimize PowerPC vector sets coming from a vector extracts

On Fri, Jul 28, 2017 at 04:08:50PM -0500, Segher Boessenkool wrote: > > "emit" is not a good name for this: that is generally used for something > that does emit_insn, i.e. put an insn in the instruction stream. This > function returns a string a define_insn can return. For the rl* insns > I

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

This patch adds support to GCC for the misaligned load/store instructions introduced in the Oracle SPARC Architecture 2017 and implemented by the SPARC M8 processor. A new command line option -mmisaligned is added, that activates the usage of the new instructions. The

Re: [PATCH] Improve extraction of changed file in contrib/mklog

On 07/27/2017 08:33 PM, Yuri Gribov wrote: > On Wed, Jul 26, 2017 at 6:11 PM, Jeff Law wrote: >> On 07/09/2017 01:03 PM, Yuri Gribov wrote: >>> Hi, >>> >>> Currently mklog will fail to analyze lines like this in patches: >>> diff -rupN gcc/gcc/testsuite/lib/profopt.exp >>>

Re: [PATCH 0/2] Python testcases to check DWARF output

On 07/26/2017 10:00 AM, Pierre-Marie de Rodat wrote: > Hello, > > At the last GNU Cauldron, Richard Biener and I talked about DWARF output > testing. Except for guality tests, which are disabled on several > targets, the only way tests check the DWARF is scanning the annotated > assembly (-dA),

Re: Add include-guard to tree-vrp.h

On 08/02/2017 08:16 AM, Prathamesh Kulkarni wrote: > Add include-guard to tree-vrp.h. > OK to commit ? OK. jeff

[PATCH, rs6000] Clean up capitalized diagnostic messages

Hi, Jakub pointed out that we've had several error messages slip into the POWER back end that incorrectly start with a capital letter. This patch hunts those down and cleans them up. Bootstrapped and tested on powerpc64le-linux-gnu with no regressions. Is this ok for trunk? I don't

Re: [PATCH 0/2] Python testcases to check DWARF output

On 07/27/2017 01:52 AM, Richard Biener wrote: > On Wed, Jul 26, 2017 at 11:25 PM, Mike Stump wrote: >> On Jul 26, 2017, at 9:00 AM, Pierre-Marie de Rodat >> wrote: >>> At the last GNU Cauldron, Richard Biener and I talked about DWARF output >>>

Add include-guard to tree-vrp.h

Add include-guard to tree-vrp.h. OK to commit ? Thanks, Prathamesh 2017-08-02 Prathamesh Kulkarni * tree-vrp.h: Add include guard. diff --git a/gcc/tree-vrp.h b/gcc/tree-vrp.h index ef2c68a752b..f84403a0f83 100644 --- a/gcc/tree-vrp.h +++

[PATCH][RFC] Make expansion of balanced binary trees of switches on tree level.

Hello. After some discussions with Honza, I've decided to convert current code in stmt.c that is responsible for switch expansion. More precisely, I would like to convert the code to expand gswitch statements on tree level. Currently the newly created pass is executed at the end of tree

Re: [PATCH] Add -nolibc option

On Thu, 27 Jul 2017, Tristan Gingold wrote: > Index: gcc/common.opt > === > --- gcc/common.opt(revision 250563) > +++ gcc/common.opt(working copy) > @@ -2956,6 +2956,10 @@ > nostdlib > Driver > > +nolibc > +Driver > +Do

Re: [PATCH][RFC] Make expansion of balanced binary trees of switches on tree level.

On Wed, Aug 2, 2017 at 1:20 PM, Martin Liška wrote: > Hello. > > After some discussions with Honza, I've decided to convert current code in > stmt.c that > is responsible for switch expansion. More precisely, I would like to convert > the code > to expand gswitch statements on

Re: c-family PATCH to improve and simplify -Wmultistatement-macros (PR c/81448, c/81306)

On Wed, Aug 02, 2017 at 11:37:27AM +, Joseph Myers wrote: > On Thu, 27 Jul 2017, Marek Polacek wrote: > > > Bootstrapped/regtested on x86_64-linux and ppc64le-linux, ok for trunk? > > > > 2017-07-27 Marek Polacek > > > > PR c/81448 > > PR c/81306 > > *

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

From: qinzhao Date: Wed, 2 Aug 2017 10:27:51 -0500 > This patch adds support to GCC for the misaligned load/store > instructions introduced in the Oracle SPARC Architecture 2017 and > implemented by the SPARC M8 processor. > > A new command line option

Re: ping [PATCH] [MSP430] Fix PR78849: ICE on initialization of global struct containing __int20 array

On Wed, 2 Aug 2017, Jeff Law wrote: > I think Joseph's suggestion for looking at partial float handling may be > useful, though ia64's RFmode may be more interesting as it's not a > multiple of 8 in bitsize. IF/KF modes on the ppc port have similar > properties. The key issue those

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

On 08/02/2017 09:29 AM, Bill Schmidt wrote: Hi, Jakub pointed out that we've had several error messages slip into the POWER back end that incorrectly start with a capital letter. This patch hunts those down and cleans them up. Others have already commented on some aspects of the patch but

Re: [PATCH] enhance overflow and truncation detection in strncpy and strncat (PR 81117)

On 07/31/2017 01:42 PM, Martin Sebor wrote: >> So I *think* TYPE_SIZE_UNIT isn't necessarily guaranteed to be a >> INTEGER_CST, it could be a non-constant expression for the size. Are >> the callers of compute_objsize prepared to handle that? Just to be >> clear, I'd prefer to return

Re: [PING #2] [PATCH] enhance -Wrestrict to handle string built-ins (PR 78918)

On 08/01/2017 03:25 AM, Richard Biener wrote: > On Tue, Aug 1, 2017 at 11:23 AM, Richard Biener > wrote: >> On Tue, Aug 1, 2017 at 4:27 AM, Martin Sebor wrote: >>> Richard, >>> >>> in discussing this work Jeff mentioned that your comments on >>> the

Re: [PATCH 6/6] qsort comparator consistency checking

On 07/31/2017 12:27 PM, Alexander Monakov wrote: > On Mon, 31 Jul 2017, Jeff Law wrote: >> I must have missed something. Can't you just define >> >> qsort (BASE, NMEMB, SIZE, COMPARE) into >> >> qsort_chk (BASE, NMEMB, SIZE, COMPARE) >> >> That shouldn't affect the qsort from vec? Right? Or am

Re: c-family PATCH to improve and simplify -Wmultistatement-macros (PR c/81448, c/81306)

On Wed, 2017-08-02 at 11:37 +, Joseph Myers wrote: > On Thu, 27 Jul 2017, Marek Polacek wrote: > > > > > Bootstrapped/regtested on x86_64-linux and ppc64le-linux, ok for > > trunk? > > > > 2017-07-27  Marek Polacek   > > > > PR c/81448 > > PR c/81306 > > *

Re: [PATCH 6/6] qsort comparator consistency checking

On 07/15/2017 02:47 PM, Alexander Monakov wrote: > This is the updated qsort comparator verifier. > > Since we have vec::qsort(cmp), the patch uses the macro argument counting > trick to redirect only the four-argument invocations of qsort to qsort_chk. > I realize that won't win much sympathies,

Re: c-family PATCH to improve and simplify -Wmultistatement-macros (PR c/81448, c/81306)

On Wed, Aug 02, 2017 at 10:16:54AM -0700, Steve Ellcey wrote: > On Wed, 2017-08-02 at 11:37 +, Joseph Myers wrote: > > On Thu, 27 Jul 2017, Marek Polacek wrote: > > > > > > > > Bootstrapped/regtested on x86_64-linux and ppc64le-linux, ok for > > > trunk? > > > > > > 2017-07-27  Marek

Re: [PATCH] Add missing edge probability in simd_clone_adjust

On Wed, Aug 02, 2017 at 06:15:31PM +0200, Tom de Vries wrote: > 2017-08-02 Tom de Vries > > * omp-simd-clone.c (simd_clone_adjust): Add missing edge probability. I think that is not the right probability, when using masked simd, the point is that it is called from

Re: ping [PATCH] [MSP430] Fix PR78849: ICE on initialization of global struct containing __int20 array

On 08/01/2017 10:26 AM, Jozef Lawrynowicz wrote: > On 01/08/2017 00:08, Joseph Myers wrote: >> On Wed, 26 Jul 2017, Jeff Law wrote: >> >>> TYPE_SIZE, according to my understanding, should be a tree for the size >>> of the expression in bits. >>> >>> The problem is for msp430 that size varies

Re: [PATCH 1/5] Rename existing insn_cost to insn_sched_cost

On 07/31/2017 05:25 PM, Segher Boessenkool wrote: > haifa-sched exports an insn_cost function, but it is only used in a > few places and specialised to scheduling. This patch renames it to > insn_sched_cost. > > --- > gcc/haifa-sched.c | 14 +++--- > gcc/sched-int.h| 2 +- >

Re: [PATCH 2/5] Replace insn_rtx_cost with insn_cost and pattern_cost

On 07/31/2017 05:25 PM, Segher Boessenkool wrote: > This renames insn_rtx_cost to pattern cost, and adds a new function > insn_cost that takes an rtx_insn * instead of an instruction pattern > as input. It uses the latter function anywhere an instruction is > readily available (instead of just an

Re: [PATCH 4/5] Add targetm.insn_cost hook

On 07/31/2017 05:25 PM, Segher Boessenkool wrote: > This introduces a hook to implement insn_cost. If a target does not > implement the hook, the old function (i.e. pattern_cost) is used. > > --- > gcc/doc/tm.texi| 12 > gcc/doc/tm.texi.in | 2 ++ > gcc/rtlanal.c | 3 +++

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

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 look at it). But it's good enough to test > things with. So what

  1   2   >