Re: [Patch, Fortran, F03] PR 85395: private clause contained in derived type acquires spurious scope

2018-09-09 Thread Paul Richard Thomas
Hi Janus, That's OK for trunk and, I would suggest 8-branch. Thanks for the patch. Paul On 9 September 2018 at 17:31, Janus Weil wrote: > Hi all, > > the attached patch fixes a problem with the accessibility of procedure > pointer components (private/public attribute). It is rather > straight

Re: [Patch, Fortran] PR 86830: [8/9 Regression] Contiguous array pointer function result not recognized as contiguous

2018-09-09 Thread Paul Richard Thomas
Hi Janus, That's fine - OK for both branches. Thanks Paul On 9 September 2018 at 21:34, Janus Weil wrote: > Hi all, > > the attached patch fixes a rejects-valid regression, where a > type-bound procedure call was not correctly detected to have a > contiguous result. The patch is functionally

[PATCH, OpenACC] C++ reference mapping (PR middle-end/86336)

2018-09-09 Thread Julian Brown
This patch (by Cesar) changes the way C++ references are mapped in OpenACC regions, fixing an ICE in the non-scalar-data.C testcase. Post-patch, references are mapped like this (from the omplower dump): map(force_present:*x [len: 4]) map(firstprivate ref:x [pointer assign, bias: 0]) Tested with

C++ PATCH to tidy up build_vtbl_ref

2018-09-09 Thread Marek Polacek
The wrapper for build_vtbl_ref_1 doesn't seem to do anything useful. Bootstrapped/regtested on x86_64-linux, ok for trunk? 2018-09-09 Marek Polacek * class.c (build_vtbl_ref): Remove. (build_vtbl_ref_1): Rename to build_vtbl_ref. (build_vfn_ref): Call build_vtbl_ref in

Re: [C++ PATCH] PR c++/87093

2018-09-09 Thread Jason Merrill
OK for trunk and branches. On Sun, Sep 9, 2018 at 11:02 PM, Ville Voutilainen wrote: > Tested manually on Linux-x64, running full suite on Linux-PPC64. This > seems half-obvious. OK for trunk? Backports? > > 2018-09-10 Ville Voutilainen > > gcc/cp > > PR c++/87093 > * method.c (co

[C++ PATCH] PR c++/87093

2018-09-09 Thread Ville Voutilainen
Tested manually on Linux-x64, running full suite on Linux-PPC64. This seems half-obvious. OK for trunk? Backports? 2018-09-10 Ville Voutilainen gcc/cp PR c++/87093 * method.c (constructible_expr): We're in an unevaluated context in all cases, not just for class targets.

[wwwdocs] Add a few missing s to egcs-1.1/regress.html

2018-09-09 Thread Gerald Pfeifer
Not strictly speaking an HTML 5 compliance issue, but the validator issued warnings, so I had to look into it and went ahead and addressed it. Applied. Gerald Index: egcs-1.1/regress.html === RCS file: /cvs/gcc/wwwdocs/htdocs/egcs-1

[wwwdocs] Avoid empty headings in gcc-5/changes.html

2018-09-09 Thread Gerald Pfeifer
Not sure why those were added (probably out of unawareness of the option to put an id on a list), but it doesn't make sense; fixed thusly. Gerald Index: gcc-5/changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-5/changes.html,v re

[wwwdocs] Completely switch navigation bar to CSS

2018-09-09 Thread Gerald Pfeifer
Introduce a new CSS class "middle". Use that for the icons in the navigation bar instead of align="middle" (except one case where it is not necessary at all). Applied, and I also regenerated the pages on gcc.gnu.org. As a side note, this actually looks better now! Gerald Index: gcc.css ===

Re: Prune TYPE_NEXT_VARIANT lists in free_lang_data

2018-09-09 Thread Jan Hubicka
> On September 8, 2018 10:49:02 AM GMT+01:00, Jan Hubicka > wrote: > >Hi > >while working on path to replace type variant by first compatible one I > >run > >into issue that the first vairant was not seen by free_lang_data. I > >think this > >is a bug becuase nothing prevents middle-end from look

[wwwdocs] testing/index.html -- move to full use of CSS

2018-09-09 Thread Gerald Pfeifer
Committed. (Also removes an unnecessary and makes source formatting a bit clearer.) Committed. Gerald Index: testing/index.html === RCS file: /cvs/gcc/wwwdocs/htdocs/testing/index.html,v retrieving revision 1.41 diff -u -r1.41 ind

[Patch, Fortran] PR 86830: [8/9 Regression] Contiguous array pointer function result not recognized as contiguous

2018-09-09 Thread Janus Weil
Hi all, the attached patch fixes a rejects-valid regression, where a type-bound procedure call was not correctly detected to have a contiguous result. The patch is functionally identical with comment #2 in the PR, with a little bit of cleanup on top of it. It regtests cleanly on x86_64-linux-gnu.

Re: [wwwdocs] projects/prefetch.html -- replace direct formatting of tables with CSS

2018-09-09 Thread Gerald Pfeifer
On Sun, 2 Sep 2018, Gerald Pfeifer wrote: > Committed. Buiding on my changes last Sunday, this uses the new class "border" I introduced earlier today, thus avoiding validation warnings. Committed. Gerald Index: projects/prefetch.html =

[wwwdocs] Mirror the recent gcc-4.3/porting_to.html change to gcc-4.4

2018-09-09 Thread Gerald Pfeifer
Applied. Gerald Index: gcc-4.4/porting_to.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.4/porting_to.html,v retrieving revision 1.12 diff -u -r1.12 porting_to.html --- gcc-4.4/porting_to.html 1 Sep 2018 23:42:04 - 1.

[wwwdocs] Continue CSSification, introducing table.border and updating gcc-4.3/porting.html

2018-09-09 Thread Gerald Pfeifer
Introduce a new style table.border that approaches the classic , just with a bit less use of borders. Use this in gcc-4.3/porting.html. Applied. Gerald Index: gcc.css === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc.css,v retrieving revis

Re: [wwwdocs] gcc-3.3/criteria.html - simplify

2018-09-09 Thread Gerald Pfeifer
On Sun, 2 Sep 2018, Gerald Pfeifer wrote: > Similar changes as for gcc-3.4/criteria.html, now HTML 5. And similar to what I did with gcc-3.1/criteria.html earlier today, now again with centered tables (and avoiding the border="1" attribute). Committed. Gerald Index: gcc-3.3/criteria.html ==

Re: Prune TYPE_NEXT_VARIANT lists in free_lang_data

2018-09-09 Thread Richard Biener
On September 8, 2018 10:49:02 AM GMT+01:00, Jan Hubicka wrote: >Hi >while working on path to replace type variant by first compatible one I >run >into issue that the first vairant was not seen by free_lang_data. I >think this >is a bug becuase nothing prevents middle-end from looking up a variant

Committed, PR target/86794, MMIX TARGET_HAVE_SPECULATION_SAFE_VALUE: Not needed.

2018-09-09 Thread Hans-Peter Nilsson
The mythical MMIX hardware engineers have wisely understood the need to automatically disable speculation when a speculated execution path transitions to kernel mode. There might be mythical talk about using a bit in a configuration register to enable even that, but that's just speculation and som

Committed, PR target/85666 2/2, MMIX: Handle emitting data bytes as non-literals

2018-09-09 Thread Hans-Peter Nilsson
Until location views (in gcc-8), there apparently was no need to emit single bytes of data as anything but bare CONST_INTs, neither actual data nor dwarf2 debug. With location views, there's a field within dwarf2 records for inlined subroutines that as assembly code looks as follows in context (cu

Committed, PR target/85666 1/2, MMIX: Don't call leaf_function_p

2018-09-09 Thread Hans-Peter Nilsson
It's IMO never a good idea to call leaf_function_p in port-specific code. Sooner or later, you'll need that information in a context where calling leaf_function_p is either a bad idea (it does a linear walk over all emitted insns in a function) or invalid (called when global context is within a se

[Patch, Fortran, F03] PR 85395: private clause contained in derived type acquires spurious scope

2018-09-09 Thread Janus Weil
Hi all, the attached patch fixes a problem with the accessibility of procedure pointer components (private/public attribute). It is rather straightforward and regtest cleanly on x86_64-linux-gnu (for further details see the PR). Ok for trunk? Cheers, Janus diff --git a/gcc/fortran/ChangeLog b/gcc

Go patch committed: put type descriptor initializers in .rodata

2018-09-09 Thread Ian Lance Taylor
This patch to the Go frontend by Than McIntosh puts type descriptor initializers in the .rodata section. When creating type descriptor variables, the frontend (specifically Type::make_type_descriptor_var) invokes the immutable_struct() and immutable_struct_set_init() back end methods, so as to ins

Re: [wwwdocs] gcc-3.0/criteria.html -- fix up to HTML 5

2018-09-09 Thread Gerald Pfeifer
On Sun, 2 Sep 2018, Gerald Pfeifer wrote: > ...by stripping use of some obsolete features and adding missing > table cells (). Thinking about it again, this centers the tables as I just did with gcc-3.1/criteria.html. Applied. Gerald Index: gcc-3.0/criteria.html

[wwwdocs] Use CSS to format gcc-3.1/criteria.html

2018-09-09 Thread Gerald Pfeifer
This was the last regular page (outside our main page, where I also nearly completed the conversion) that wasn't HTML 5 but used deprecated elements. Committed. Gerald Generalize the "center" class to also center non-text elements such as tables. Use that for gcc-3.1/criteria.html. Index: gcc

[wwwdocs] Avoid widths for tables used for design of our main page

2018-09-09 Thread Gerald Pfeifer
This does not actually make a real difference (visually); tested with a couple of browsers to make sure. Committed. Gerald Index: style.mhtml === RCS file: /cvs/gcc/wwwdocs/htdocs/style.mhtml,v retrieving revision 1.153 diff -u -r1

Re: [PATCH 2/6] detect unterminated const arrays in strlen calls (PR 86552)

2018-09-09 Thread Bernd Edlinger
On 09/09/18 01:47, Jeff Law wrote: > On 9/8/18 3:47 PM, Bernd Edlinger wrote: >> Hi, >> >> >>> -fold_builtin_strlen (location_t loc, tree type, tree arg) >>> +fold_builtin_strlen (location_t loc, tree fndecl, tree type, tree arg) >>> { >>> if (!validate_arg (arg, POINTER_TYPE)) >>> retu

Re: [ARM/FDPIC v2 02/21] [ARM] FDPIC: Handle arm*-*-uclinuxfdpiceabi in configure scripts

2018-09-09 Thread Christophe Lyon
On Fri, 31 Aug 2018 at 16:13, Christophe Lyon wrote: > > On Wed, 29 Aug 2018 at 12:46, Kyrill Tkachov > wrote: > > > > > > On 13/07/18 17:10, christophe.l...@st.com wrote: > > > From: Christophe Lyon > > > > > > The new arm-uclinuxfdpiceabi target behaves pretty much like > > > arm-linux-gnueabi

[PATCH] internalize a driver fn

2018-09-09 Thread Nathan Sidwell
I discovered pfatal_with_name is only used inside gcc.c. Making it static as obvious. nathan -- Nathan Sidwell 2018-09-09 Nathan Sidwell * gcc.h (pfatal_with_name): Don't declare here. * gcc.c (pfatal_with_name): Make static. Index: gcc.c =