Re: [PATCH 00/19] cleanup of memory stats prototypes

2017-07-28 Thread Trevor Saunders
On Thu, Jul 27, 2017 at 10:43:09AM +0200, Richard Biener wrote: > On Thu, Jul 27, 2017 at 10:30 AM, wrote: > > From: Trevor Saunders > > > > The preC++ way of passing information about the call site of a function was > > to > > use a macro that passed __file__, __LINE__, and __FUNCTION__ to a f

Re: [PATCH 00/19] cleanup of memory stats prototypes

2017-07-28 Thread Trevor Saunders
On Fri, Jul 28, 2017 at 06:31:05PM +0200, Bernhard Reutner-Fischer wrote: > On 27 July 2017 10:43:09 CEST, Richard Biener > wrote: > >On Thu, Jul 27, 2017 at 10:30 AM, wrote: > >> From: Trevor Saunders > >> > >> The preC++ way of passing information about the call site of a > >function was to

Re: [PATCH][1/2] Early LTO debug, simple-object part

2017-07-28 Thread Jason Merrill
On 07/28/2017 05:55 PM, Jason Merrill wrote: On Fri, Jul 28, 2017 at 8:54 AM, Richard Biener wrote: On Tue, 4 Jul 2017, Richard Biener wrote: On Tue, 20 Jun 2017, Richard Biener wrote: On Wed, 7 Jun 2017, Richard Biener wrote: On Fri, 19 May 2017, Richard Biener wrote: This is a repost

Re: [PATCH][1/2] Early LTO debug, simple-object part

2017-07-28 Thread Jason Merrill
On Fri, Jul 28, 2017 at 8:54 AM, Richard Biener wrote: > On Tue, 4 Jul 2017, Richard Biener wrote: > >> On Tue, 20 Jun 2017, Richard Biener wrote: >> >> > On Wed, 7 Jun 2017, Richard Biener wrote: >> > >> > > On Fri, 19 May 2017, Richard Biener wrote: >> > > >> > > > >> > > > This is a repost (unc

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

2017-07-28 Thread Segher Boessenkool
Hi! On Thu, Jul 27, 2017 at 07:21:14PM -0400, Michael Meissner wrote: > This patches optimizes the PowerPC vector set operation for 64-bit doubles and > longs where the elements in the vector set may have been extracted from > another > vector (PR target/81593): > * config/rs6000/rs6000.c

Re: [PATCH] PR libstdc++/79162 ambiguity in string assignment due to string_view overload (LWG 2946)

2017-07-28 Thread Daniel Krügler
2017-07-28 22:29 GMT+02:00 Daniel Krügler : > 2017-07-28 22:25 GMT+02:00 Tim Song : >> On Fri, Jul 28, 2017 at 4:10 PM, Daniel Krügler >> wrote: >>> + // Performs an implicit conversion from _Tp to __sv_type. >>> + template >>> +static __sv_type _S_to_string_view(const _Tp& __svt

Re: [PATCH] PR libstdc++/79162 ambiguity in string assignment due to string_view overload (LWG 2946)

2017-07-28 Thread Daniel Krügler
2017-07-28 22:25 GMT+02:00 Tim Song : > On Fri, Jul 28, 2017 at 4:10 PM, Daniel Krügler > wrote: >> + // Performs an implicit conversion from _Tp to __sv_type. >> + template >> +static __sv_type _S_to_string_view(const _Tp& __svt) >> +{ >> + return __svt; >> +

Re: [PATCH] PR libstdc++/79162 ambiguity in string assignment due to string_view overload (LWG 2946)

2017-07-28 Thread Tim Song
On Fri, Jul 28, 2017 at 4:10 PM, Daniel Krügler wrote: > + // Performs an implicit conversion from _Tp to __sv_type. > + template > +static __sv_type _S_to_string_view(const _Tp& __svt) > +{ > + return __svt; > +} I might have gone for +static _

[PATCH] PR libstdc++/79162 ambiguity in string assignment due to string_view overload (LWG 2946)

2017-07-28 Thread Daniel Krügler
This patch attempts to solve an issue in basic_string after adding string_view to several functions that came in during incomplete patch of LWG 2758/LWG 2946, in short: Functions that take a string_view parameter value are now replaced by constrained templates taking const T& that is convertible to

Go frontend: Add another backend type conversion

2017-07-28 Thread Ian Lance Taylor
This patch to the Go frontend by Than McIntosh tweaks Interface_field_reference_expression::do_get_backend to apply an additional backend type conversion to the returned result. This is needed due to the fact that the top level type of the expression is a function descriptor, however the value bei

Re: [PATCH] Emit DWARF5 DW_AT_export_symbols for namespaces

2017-07-28 Thread Jason Merrill
On Fri, Jul 21, 2017 at 4:33 PM, Jakub Jelinek wrote: > DWARF5 introduced DW_AT_export_symbols that may be preset on > DW_TAG_namespace or DW_TAG_{structure,union,class}_type to signalize > inline or anonymous namespaces or anonymous structures/unions/classes. > > What we were emitting instead is

Re: [patch, libfortran] Fix PR 80365, undefined memcpy

2017-07-28 Thread Janne Blomqvist
On Mon, Jul 24, 2017 at 2:12 PM, Thomas Koenig wrote: > Hi Janne, > >> On Sun, Jul 23, 2017 at 10:23 PM, Thomas Koenig >> wrote: >>> >>> Hello world, >>> >>> the attached test cases fixes a problematic invocation of >>> memcpy where src=NULL and n=0 by simply doing nothing >>> in the surrounding

Re: [PATCH,AIX] Enable libffi for AIX

2017-07-28 Thread David Edelsohn
On Wed, Jul 26, 2017 at 11:17 AM, REIX, Tony wrote: > Description: > * This patch enables libffi on AIX. > > Tests: > * Fedora25/x86_64 + GCC trunk : Configure/Build: SUCCESS >- build made by means of gmake. > > ChangeLog: > * configure.ac, configure: Enable libffi for AIX I committed this

Re: [PATCH,AIX] Enable Go for AIX

2017-07-28 Thread David Edelsohn
On Wed, Jul 26, 2017 at 8:48 AM, REIX, Tony wrote: > Description: > * This patch enables Go on AIX. > > Tests: > * Fedora25/x86_64 + GCC trunk : Configure/Build: SUCCESS >- build made by means of gmake. > > ChangeLog: > * configure.ac, configure: Enable Go for AIX > * contrib/config-list.m

Re: [PATCH,AIX] Don't leak a file descriptor if an archive is malformed.

2017-07-28 Thread Ian Lance Taylor
On Thu, Jul 27, 2017 at 8:31 AM, REIX, Tony wrote: > Better with the patch file... > Sorry. The Resend did not add the joint file I added with first message (in > HTML format, refused). > Hope it's OK now. Thanks. Committed to mainline. Ian

[patch, fortran] Generate C prototypes from Fortran code

2017-07-28 Thread Thomas Koenig
Hello world, the attached patch generates C prototypes from all things BIND(C) that it can find and dumps them to standard output, under control of the appropriate flag. Enums are not yet supported (we translate them to parameters almost immediately, so this will need more work). I have added a

Re: [PATCH 00/17] RFC: New source-location representation; Language Server Protocol

2017-07-28 Thread Martin Sebor
Thanks for the explanation (snipped). * C++: add a fix-it hint to -Wsuggest-final-methods To answer your call for other ideas below: There are other suggestions that GCC could offer to help improve code, including * to add the const qualifier to function pointer and reference argument

Re: [Patch] Testsuite fixes for failures caused by patch for PR 80925 - loop peeling and alignment

2017-07-28 Thread Steve Ellcey
On Fri, 2017-07-28 at 09:47 +0200, Richard Biener wrote: > On Fri, Jul 28, 2017 at 12:16 AM, Steve Ellcey wrote: > >  > > Any comments from the power and/or vectorizer folks? > On one side I'm inclined to simplify the testsuite by adding > --param vect-max-peeling-for-alignment=0 in addition to >

Re: [patch 0/2] PR49847: Add hook to place read-only lookup-tables in named address-space

2017-07-28 Thread Georg-Johann Lay
Richard Biener schrieb: On Fri, Jul 28, 2017 at 12:18 PM, Georg-Johann Lay wrote: Richard Biener schrieb: On Thu, Jul 27, 2017 at 3:32 PM, Georg-Johann Lay wrote: On 27.07.2017 14:34, Richard Biener wrote: On Thu, Jul 27, 2017 at 2:29 PM, Georg-Johann Lay wrote: For some targets, the bes

Go patch commited: Track placeholder pointer types for conversion

2017-07-28 Thread Ian Lance Taylor
The Go frontend recently started walking through the hash table of pointer types to finalize them. Unfortunately it is possible to create a new pointer type while finalizing an existing one (test case: test/fixedbugs/issue5291) and that breaks the iteration. So, instead, keep a list of placeholde

Re: [PATCH/AARCH64] Decimal floating point support for AARCH64

2017-07-28 Thread Joseph Myers
On Thu, 13 Jul 2017, Andrew Pinski wrote: > Hi, > This patch adds Decimal floating point support to aarch64. It is > the base support in that since there is no hardware support for DFP, > it just defines the ABI. The ABI I chose is that _Decimal32 is > treated like float, _Decimal64 is treated

Go patch committed: Use a single temporary for a call with multiple results

2017-07-28 Thread Ian Lance Taylor
In the Go frontend, for calls that return multiple results, we used to create a temporary of struct type to hold the results, and also create a separate temporary for each result. Then the call expression would copy each result out of the struct to the temporary, and Call_result_expression would r

Re: [PATCH] [PowerPC/RTEMS] Add 64-bit support using ELFv2 ABI

2017-07-28 Thread Segher Boessenkool
On Fri, Jul 28, 2017 at 10:29:21AM +0200, Sebastian Huber wrote: > Would you mind having a look at: > > https://gcc.gnu.org/ml/gcc/2017-07/msg00196.html > > Is the small data area no longer supported by GCC for 64-bit PowerPC? Yes, I'm looking at it, it takes a bit longer. Currently we do not s

Patch ping

2017-07-28 Thread Jakub Jelinek
Hi! I'd like to ping the following patch: - DWARF - emit DW_AT_export_symbols for inline and anon namespaces https://gcc.gnu.org/ml/gcc-patches/2017-07/msg01349.html Thanks Jakub

Re: std::vector default default and move constructors

2017-07-28 Thread François Dumont
Hi There was a little issue in this patch, here is the correct version. François On 23/07/2017 19:41, François Dumont wrote: Hi Is it time now to consider this patch ? * include/bits/stl_vector.h (_Vector_impl_data): New. (_Vector_impl): Inherit from latter. (_Vertor

Re: std::list optimizations

2017-07-28 Thread François Dumont
Hi Completing execution of the testsuite revealed a bug. So here is the correct version of this patch. François On 21/07/2017 19:14, François Dumont wrote: Hi Here is a proposal for 2 optimizations in the std::list implementation. Optimization on the move constructor taking a

Re: [PATCH 00/19] cleanup of memory stats prototypes

2017-07-28 Thread Bernhard Reutner-Fischer
On 27 July 2017 10:43:09 CEST, Richard Biener wrote: >On Thu, Jul 27, 2017 at 10:30 AM, wrote: >> From: Trevor Saunders >> >> The preC++ way of passing information about the call site of a >function was to >> use a macro that passed __file__, __LINE__, and __FUNCTION__ to a >function with >> t

Re: [PATCH] New C++ warning option '-Wduplicated-access-specifiers'

2017-07-28 Thread Martin Sebor
On 07/27/2017 01:13 PM, Richard Sandiford wrote: Martin Sebor writes: On 07/23/2017 02:42 PM, Volker Reichelt wrote: On 21 Jul, Martin Sebor wrote: On 07/20/2017 10:35 AM, Volker Reichelt wrote: Hi, the following patch introduces a new C++ warning option -Wduplicated-access-specifiers that

Re: [PATCH 15/17] Language Server Protocol: add lsp::server abstract base class

2017-07-28 Thread David Malcolm
On Thu, 2017-07-27 at 09:55 +0200, Richard Biener wrote: > On Mon, Jul 24, 2017 at 10:05 PM, David Malcolm > wrote: > > This patch adds an lsp::server abstract base class for implementing > > servers for the Language Server Protocol: > > https://github.com/Microsoft/language-server-protocol > >

Re: [PATCH GCC][4/4]Better handle store-stores chain if eliminated stores only store loop invariant

2017-07-28 Thread Bin.Cheng
On Tue, Jul 25, 2017 at 12:48 PM, Richard Biener wrote: > On Mon, Jul 10, 2017 at 10:24 AM, Bin.Cheng wrote: >> On Tue, Jun 27, 2017 at 11:49 AM, Bin Cheng wrote: >>> Hi, >>> This is a followup patch better handling below case: >>> for (i = 0; i < n; i++) >>>{ >>> a[i] = 1;

Re: [PATCH GCC][5/6]Record initialization statements and only insert it for valid chains

2017-07-28 Thread Bin.Cheng
On Mon, Jun 26, 2017 at 10:57 AM, Richard Biener wrote: > On Mon, Jun 26, 2017 at 11:47 AM, Bin.Cheng wrote: >> On Fri, May 12, 2017 at 12:28 PM, Bin Cheng wrote: >>> Hi, >>> This patch caches initialization statements and only inserts it for valid >>> chains. >>> Looks like current code even i

Re: [PATCH GCC][4/6]Simple patch skips single element component

2017-07-28 Thread Bin.Cheng
On Fri, Jun 23, 2017 at 5:56 PM, Jeff Law wrote: > On 05/12/2017 05:28 AM, Bin Cheng wrote: >> Hi, >> This is a simple patch discarding simple element components earlier in >> predcom. >> Bootstrap and test on x86_64 and AArch64, is it OK? >> >> Thanks, >> bin >> 2017-05-10 Bin Cheng >> >>

Re: [PATCH][GCC][AArch64] optimize float immediate moves (1 /4) - infrastructure.

2017-07-28 Thread Tamar Christina
Hi James, I have made a tiny change in the patch to allow 0 to always pass through, since in all cases we can always do a move of 0. I will assume your OK to still stand under the obvious rule and will commit with this change. Thanks, Tamar From: James

Re: [PATCH 15/17] Language Server Protocol: add lsp::server abstract base class

2017-07-28 Thread David Malcolm
On Thu, 2017-07-27 at 00:14 -0400, Trevor Saunders wrote: > On Mon, Jul 24, 2017 at 04:05:12PM -0400, David Malcolm wrote: > > This patch adds an lsp::server abstract base class for implementing > > servers for the Language Server Protocol: > > https://github.com/Microsoft/language-server-protoco

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

2017-07-28 Thread Michael Meissner
On Fri, Jul 28, 2017 at 09:51:30AM +0200, Richard Biener wrote: > On Fri, Jul 28, 2017 at 1:21 AM, Michael Meissner > wrote: > > This patches optimizes the PowerPC vector set operation for 64-bit doubles > > and > > longs where the elements in the vector set may have been extracted from > > anot

Re: [PING] [PATCH v4 0/12] [i386] Improve 64-bit Microsoft to System V ABI pro/epilogues

2017-07-28 Thread H.J. Lu
On Fri, Jul 28, 2017 at 6:57 AM, Daniel Santos wrote: > On 07/26/2017 02:03 PM, H.J. Lu wrote: >> >> This patch caused: >> >> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81563 > > > Hello. I've rebased my patch set and I'm now retesting. I'm afraid that > your changes are wrong because my my sp

Re: [PATCH 00/17] RFC: New source-location representation; Language Server Protocol

2017-07-28 Thread David Malcolm
On Thu, 2017-07-27 at 13:51 -0600, Martin Sebor wrote: > I think this is great. My overall question is: is the new BLT > representation available in the middle-end? Given that, err, I'm not freeing it anywhere yet (ahem), the answer is yes. Although it lives in the "gcc" dir, the generated tre

Re: [PATCH PR81228]Fixes ICE by adding LTGT in vec_cmp.

2017-07-28 Thread Bin.Cheng
On Fri, Jul 28, 2017 at 3:15 PM, Richard Sandiford wrote: > "Bin.Cheng" writes: >> On Fri, Jul 28, 2017 at 12:55 PM, Richard Sandiford >> wrote: >>> Bin Cheng writes: Hi, This simple patch fixes the ICE by adding LTGT in vec_cmp pattern. I also modified the original test cas

Re: [PATCH PR81228]Fixes ICE by adding LTGT in vec_cmp.

2017-07-28 Thread Richard Sandiford
"Bin.Cheng" writes: > On Fri, Jul 28, 2017 at 12:55 PM, Richard Sandiford > wrote: >> Bin Cheng writes: >>> Hi, >>> This simple patch fixes the ICE by adding LTGT in >>> vec_cmp pattern. >>> I also modified the original test case into a compilation one since >>> -fno-wrapping-math >>> should not

Re: [PING] [PATCH v4 0/12] [i386] Improve 64-bit Microsoft to System V ABI pro/epilogues

2017-07-28 Thread Daniel Santos
On 07/26/2017 02:03 PM, H.J. Lu wrote: This patch caused: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81563 Hello. I've rebased my patch set and I'm now retesting. I'm afraid that your changes are wrong because my my sp_valid_at and fp_valid_at functions are wrong -- these are supposed to

[PATCH] Support cfi pseudo for debugging on UI_DWARF2 target

2017-07-28 Thread Tristan Gingold
Hi, this patch fixes a thinko in dwarf2out_do_cfi_asm: some targets (x86_64-windows) so support cfi psuedo for debug info, but not dwarf2 exception cfi. No regressions on x86_64-linux-gnu. Ok to commit ? 2017-07-28 gingold * dwarf2out.c (dwarf2out_do_cfi_startproc): Return early

Re: [PATCH,AIX] Changes for linking gotools on AIX.

2017-07-28 Thread David Edelsohn
On Thu, Jul 27, 2017 at 11:51 AM, REIX, Tony wrote: > Hi Ian, David, > > On AIX, that is more complicated... > > We have to use -static-libgo when building the libgo tests. Because AIX does > not work like Linux does and because the Go libgo tests are done by > duplicating several .go files of l

Re: [PATCH][1/2] Early LTO debug, simple-object part

2017-07-28 Thread Richard Biener
On Tue, 4 Jul 2017, Richard Biener wrote: > On Tue, 20 Jun 2017, Richard Biener wrote: > > > On Wed, 7 Jun 2017, Richard Biener wrote: > > > > > On Fri, 19 May 2017, Richard Biener wrote: > > > > > > > > > > > This is a repost (unchanged) of the simple-object ELF support for > > > > early LTO

[PR 81477] Set versionable regardless of optimization level

2017-07-28 Thread Martin Jambor
Hi, in revision 250048 the setting of cgraph's node versionable flag was made conditional on optimization level, which is undesirable as there may be other reasons to make a copy of a function, not just optimization for example expansion to HSA which is what the PR 81477 is about. Fixed thusly, t

[PATCH] match.pd and pattern ordering

2017-07-28 Thread Richard Biener
So I did an item from my TODO list, implemented a warning whenever pattern ordering in match.pd prevents genmatch.c from merging nodes in the decision tree (to avoid that it matches a pattern following another that would eventually also match). I noticed we're doing this a bit too conservatively,

Re: [PATCH PR81228]Fixes ICE by adding LTGT in vec_cmp.

2017-07-28 Thread Bin.Cheng
On Fri, Jul 28, 2017 at 12:55 PM, Richard Sandiford wrote: > Bin Cheng writes: >> Hi, >> This simple patch fixes the ICE by adding LTGT in >> vec_cmp pattern. >> I also modified the original test case into a compilation one since >> -fno-wrapping-math >> should not be used in general. >> Bootstra

Re: [PATCH PR81228]Fixes ICE by adding LTGT in vec_cmp.

2017-07-28 Thread Richard Sandiford
Bin Cheng writes: > Hi, > This simple patch fixes the ICE by adding LTGT in > vec_cmp pattern. > I also modified the original test case into a compilation one since > -fno-wrapping-math > should not be used in general. > Bootstrap and test on AArch64, test result check for x86_64. Is it OK? > I w

Re: [PATCH] Set VALUE_EXPR after a debug stmt is generated (PR sanitizer/81340).

2017-07-28 Thread Jakub Jelinek
On Fri, Jul 28, 2017 at 01:36:54PM +0200, Martin Liška wrote: > > I think target_for_debug_bind () gets confused. > > Exactly: > > tree > target_for_debug_bind (tree var) > { > ... > > if (DECL_HAS_VALUE_EXPR_P (var)) > return target_for_debug_bind (DECL_VALUE_EXPR (var)); > ... > > There

[PATCH PR81228]Fixes ICE by adding LTGT in vec_cmp.

2017-07-28 Thread Bin Cheng
Hi, This simple patch fixes the ICE by adding LTGT in vec_cmp pattern. I also modified the original test case into a compilation one since -fno-wrapping-math should not be used in general. Bootstrap and test on AArch64, test result check for x86_64. Is it OK? I would also need to backport it t

Re: [PATCH] Set VALUE_EXPR after a debug stmt is generated (PR sanitizer/81340).

2017-07-28 Thread Martin Liška
On 07/28/2017 01:32 PM, Richard Biener wrote: > On Fri, Jul 28, 2017 at 1:19 PM, Jakub Jelinek wrote: >> On Fri, Jul 28, 2017 at 01:11:51PM +0200, Martin Liška wrote: >>> Hello. >>> >>> Following patch fixed race where we first need to properly build >>> all gimple_build_debug_bind (arg, var, NULL

Re: [PATCH] Set VALUE_EXPR after a debug stmt is generated (PR sanitizer/81340).

2017-07-28 Thread Richard Biener
On Fri, Jul 28, 2017 at 1:19 PM, Jakub Jelinek wrote: > On Fri, Jul 28, 2017 at 01:11:51PM +0200, Martin Liška wrote: >> Hello. >> >> Following patch fixed race where we first need to properly build >> all gimple_build_debug_bind (arg, var, NULL) and then DECL_HAS_VALUE_EXPR_P >> can be registered

Re: [PATCH][2/2] Fix PR81502

2017-07-28 Thread Richard Biener
On Thu, 27 Jul 2017, Andrew Pinski wrote: > On Thu, Jul 27, 2017 at 6:50 AM, Richard Biener wrote: > > > > I am testing the following additional pattern for match.pd to fix > > PR81502 resulting in the desired optimization to > > > > bar: > > .LFB526: > > .cfi_startproc > > movl

Re: [PATCH] Dump BB number when dumping a BB with label.

2017-07-28 Thread Richard Biener
On Fri, Jul 28, 2017 at 12:52 PM, Martin Liška wrote: > On 07/28/2017 09:58 AM, Richard Biener wrote: >> On Fri, Jul 28, 2017 at 9:50 AM, Martin Liška wrote: >>> On 07/28/2017 09:21 AM, Richard Biener wrote: On Thu, Jul 27, 2017 at 4:24 PM, Martin Liška wrote: > Hi. > > Followin

[PATCH] Do UBSAN sanitization just when current_function_decl != NULL_TREE (PR sanitize/81530).

2017-07-28 Thread Martin Liška
Hi. In r249158 I added new sanitize_flags_p function. However I removed various calls of do_ubsan_in_current_function: /* True if we want to play UBSan games in the current function. */ bool do_ubsan_in_current_function () { return (current_function_decl != NULL_TREE && !lookup_attr

Re: [PATCH] Set VALUE_EXPR after a debug stmt is generated (PR sanitizer/81340).

2017-07-28 Thread Jakub Jelinek
On Fri, Jul 28, 2017 at 01:11:51PM +0200, Martin Liška wrote: > Hello. > > Following patch fixed race where we first need to properly build > all gimple_build_debug_bind (arg, var, NULL) and then DECL_HAS_VALUE_EXPR_P > can be registered. > > Patch can bootstrap on ppc64le-redhat-linux and surviv

Re: [patch 0/2] PR49847: Add hook to place read-only lookup-tables in named address-space

2017-07-28 Thread Richard Biener
On Fri, Jul 28, 2017 at 12:18 PM, Georg-Johann Lay wrote: > Richard Biener schrieb: > >> On Thu, Jul 27, 2017 at 3:32 PM, Georg-Johann Lay wrote: >>> >>> On 27.07.2017 14:34, Richard Biener wrote: On Thu, Jul 27, 2017 at 2:29 PM, Georg-Johann Lay wrote: > > For some targets, th

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

2017-07-28 Thread Martin Liška
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 PR c++/81355 * attribs.c (sorted_attr_string): Skip empty strings

[PATCH] Set VALUE_EXPR after a debug stmt is generated (PR sanitizer/81340).

2017-07-28 Thread Martin Liška
Hello. Following patch fixed race where we first need to properly build all gimple_build_debug_bind (arg, var, NULL) and then DECL_HAS_VALUE_EXPR_P can be registered. Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. Ready to be installed? Martin gcc/ChangeLog: 2017-07

Re: [PATCH] Dump BB number when dumping a BB with label.

2017-07-28 Thread Martin Liška
On 07/28/2017 09:58 AM, Richard Biener wrote: > On Fri, Jul 28, 2017 at 9:50 AM, Martin Liška wrote: >> On 07/28/2017 09:21 AM, Richard Biener wrote: >>> On Thu, Jul 27, 2017 at 4:24 PM, Martin Liška wrote: Hi. Following simple patch adds support for dumping of BBs when it's a BB >

Re: [patch 0/2] PR49847: Add hook to place read-only lookup-tables in named address-space

2017-07-28 Thread Georg-Johann Lay
Richard Biener schrieb: On Thu, Jul 27, 2017 at 3:32 PM, Georg-Johann Lay wrote: On 27.07.2017 14:34, Richard Biener wrote: On Thu, Jul 27, 2017 at 2:29 PM, Georg-Johann Lay wrote: For some targets, the best place to put read-only lookup tables as generated by -ftree-switch-conversion is not

Re: [PATCH] Do not handle VLA in sanitization (PR sanitizer/81460).

2017-07-28 Thread Jakub Jelinek
On Fri, Jul 28, 2017 at 10:44:36AM +0200, Martin Liška wrote: > Simple patch that bails out when having a variable-length parameter of a > function. > Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. > > Ready to be installed? > Martin > > gcc/ChangeLog: > > 2017-07-17

[PATCH] Do not handle VLA in sanitization (PR sanitizer/81460).

2017-07-28 Thread Martin Liška
Hi. Simple patch that bails out when having a variable-length parameter of a function. Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. Ready to be installed? Martin gcc/ChangeLog: 2017-07-17 Martin Liska PR sanitizer/81460 * sanopt.c (sanitize_rew

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

2017-07-28 Thread Marc Glisse
On Fri, 28 Jul 2017, Andrew Pinski wrote: For the vector case, can't we write it as: _1 = BIT_FIELD_REF ; _2 = BIT_FIELD_REF ; res_8 = {_1, _2}; And then have some match.pd patterns (which might get complex), to rewrite that into VEC_PERM_EXPR? For this last part, we have simplify_vector_cons

Re: [PATCH] [PowerPC/RTEMS] Add 64-bit support using ELFv2 ABI

2017-07-28 Thread Sebastian Huber
On 27/07/17 21:57, Segher Boessenkool wrote: On Thu, Jul 27, 2017 at 07:28:30AM +0200, Sebastian Huber wrote: This deletes eabi.h and I don't see you add all its definitions to rtems.h directly (NAME__MAIN etc.) Is this on purpose? Yes, I always wondered why GCC added the __eabi() call to mai

Re: [7/7] Pool alignment information for common bases

2017-07-28 Thread Richard Biener
On Thu, Jul 27, 2017 at 2:50 PM, Richard Sandiford wrote: > Richard Biener writes: >> On Tue, Jul 4, 2017 at 2:01 PM, Richard Sandiford >> wrote: >>> Richard Biener writes: On Mon, Jul 3, 2017 at 9:49 AM, Richard Sandiford wrote: > @@ -2070,8 +2143,7 @@ vect_find_same_alignment_d

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

2017-07-28 Thread Richard Biener
On Fri, Jul 28, 2017 at 10:02 AM, Andrew Pinski wrote: > On Fri, Jul 28, 2017 at 12:51 AM, Richard Biener > wrote: >> On Fri, Jul 28, 2017 at 1:21 AM, Michael Meissner >> wrote: >>> This patches optimizes the PowerPC vector set operation for 64-bit doubles >>> and >>> longs where the elements i

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

2017-07-28 Thread Andrew Pinski
On Fri, Jul 28, 2017 at 12:51 AM, Richard Biener wrote: > On Fri, Jul 28, 2017 at 1:21 AM, Michael Meissner > wrote: >> This patches optimizes the PowerPC vector set operation for 64-bit doubles >> and >> longs where the elements in the vector set may have been extracted from >> another >> vect

Re: Patch ping

2017-07-28 Thread Richard Biener
On Thu, 27 Jul 2017, Jakub Jelinek wrote: > On Thu, Jul 27, 2017 at 09:19:34AM +0200, Richard Biener wrote: > > > I'm building both addresses and subtracting them to get the offset. > > > I guess the other option is to compute just the address of the base > > > (i.e. base_addr), and add offset (if

Re: [PATCH] Dump BB number when dumping a BB with label.

2017-07-28 Thread Richard Biener
On Fri, Jul 28, 2017 at 9:50 AM, Martin Liška wrote: > On 07/28/2017 09:21 AM, Richard Biener wrote: >> On Thu, Jul 27, 2017 at 4:24 PM, Martin Liška wrote: >>> Hi. >>> >>> Following simple patch adds support for dumping of BBs when it's a BB >>> that contains a label. That makes it easier for de

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

2017-07-28 Thread Richard Biener
On Fri, Jul 28, 2017 at 1:21 AM, Michael Meissner wrote: > This patches optimizes the PowerPC vector set operation for 64-bit doubles and > longs where the elements in the vector set may have been extracted from > another > vector (PR target/81593): > > Here an an example: > > vector doub

Re: [PATCH] Dump BB number when dumping a BB with label.

2017-07-28 Thread Martin Liška
On 07/28/2017 09:21 AM, Richard Biener wrote: > On Thu, Jul 27, 2017 at 4:24 PM, Martin Liška wrote: >> Hi. >> >> Following simple patch adds support for dumping of BBs when it's a BB >> that contains a label. That makes it easier for debugging as one can >> find destination for an edge in dump fi

Re: [Patch] Testsuite fixes for failures caused by patch for PR 80925 - loop peeling and alignment

2017-07-28 Thread Richard Biener
On Fri, Jul 28, 2017 at 12:16 AM, Steve Ellcey wrote: > I was looking at the latest aarch64 failures and noticed PR 80925. There > seems to be a consensus to change the tests to reflect the current loop > peeling behaviour so I have created a patch to do that. There are three > issues with this

Re: Fix thinko in gimple_assign_set_rhs_with_ops

2017-07-28 Thread Eric Botcazou
> I think we should get rid of that update-EH-info flag, always assuming > false. This is IMHO a too low-level interface to do this (and we lack a > gsi_replace_without_update) This looks doable (there is only one call to gsi_replace that tests the return value) but the current interface is prob

Re: [patch 0/2] PR49847: Add hook to place read-only lookup-tables in named address-space

2017-07-28 Thread Richard Biener
On Thu, Jul 27, 2017 at 3:32 PM, Georg-Johann Lay wrote: > On 27.07.2017 14:34, Richard Biener wrote: >> >> On Thu, Jul 27, 2017 at 2:29 PM, Georg-Johann Lay wrote: >>> >>> For some targets, the best place to put read-only lookup tables as >>> generated by -ftree-switch-conversion is not the gene

Re: [PATCH] Dump BB number when dumping a BB with label.

2017-07-28 Thread Richard Biener
On Thu, Jul 27, 2017 at 4:24 PM, Martin Liška wrote: > Hi. > > Following simple patch adds support for dumping of BBs when it's a BB > that contains a label. That makes it easier for debugging as one can > find destination for an edge in dump file. > > Sample, before: > > foo (int a) > { > int D

Re: C++-ify vec_info structures

2017-07-28 Thread Richard Biener
On Thu, Jul 27, 2017 at 2:39 PM, Richard Sandiford wrote: > [ Needed to unblock https://gcc.gnu.org/ml/gcc-patches/2017-07/msg00198.html ] > > This patch uses new, delete, constructors and desctructors to manage > vec_info. This includes making ~vec_info free all the data shared > by bb_vec_info

[PATCH] Fix PR81573 testcase

2017-07-28 Thread Richard Biener
Tested on x86_64-unknown-linux-gnu, applied. Richard. 2017-07-28 Richard Biener PR tree-optimization/81573 * gcc.dg/torture/pr81573.c: Explicitely use signed char. Index: gcc/testsuite/gcc.dg/torture/pr81573.c =