Re: [PATCH][GCC][AArch64] Restrict lrint inlining on ILP32.

2017-09-09 Thread Tamar Christina
Hi Andrew, Yes I noticed the changelog after I sent it. But I have a new version of the patch on internal review Which is why I didn't ping this one. I'll send it out after the Cauldron. Thanks, Tamar From: Andrew Pinski Sent: Saturday, September 9, 2017

Re: [C++] Fix PR bootstrap/81926

2017-09-09 Thread Jason Merrill
On Mon, Sep 4, 2017 at 10:08 AM, Eric Botcazou wrote: >> A solution would be to put them into a global GCed pointer-map or vector, >> freeing that at free-lang-data time. > > Here's a version that implements a bona-fide type->offset_type map. > > PR bootstrap/81926 > * cp-objcp-com

Re: [PATCH] PR c++/81852 define feature-test macro for -fthreadsafe-statics

2017-09-09 Thread Jason Merrill
On Fri, Sep 8, 2017 at 10:03 PM, Jonathan Wakely wrote: > Define the __cpp_lib_threadsafe_static_init feature-test macro as per > recent SD-6 drafts. > > Tested powerpc64le-linux and x86_64-linux. > > OK for trunk? > > The branches too? Yes. Jason

Re: [PATCH, rs6000] Fix PR target/80210: ICE in extract_insn

2017-09-09 Thread Andreas Schwab
On Sep 08 2017, Peter Bergner wrote: > The following patch fixes the problem I saw on Linux and bootstraps and > regtests > with no regressions on LE and BE (running testsuite in both 32-bit and 64-bit > modes). I was waiting to submit this until David had a chance to verify this > fixes the pr

Re: [PATCH] Improve alloca alignment

2017-09-09 Thread Eric Botcazou
> No, the stack never gets misaligned - my patch doesn't change that at all. Yes, it does. Dynamic allocation works like this: the amount to be allocated is added to VIRTUAL_STACK_DYNAMIC_REGNUM and the result is then dynamically aligned. Your patch assumes that VIRTUAL_STACK_DYNAMIC_REGNUM is

Re: [PATCH] Add macro DISABLE_COPY_AND_ASSIGN

2017-09-09 Thread Yao Qi
On Fri, Aug 11, 2017 at 3:14 PM, Pedro Alves > Yeah, this is a macro that lots of projects out there reinvent, > can't imagine it being very controversial. > > I could have used this today in another spot in gdb. > > The patch as is touches areas with different maintainers, it > may have fallen vic

[Ada] Small tweak to conversion to padding types

2017-09-09 Thread Eric Botcazou
While investigating another issue, I came across a case where a simple assignment gives rise to the creation of two CONSTRUCTORS in a row when a padding type is involved. I think we can get away with only one. Tested on x86_64-suse-linux, applied on the mainline. 2017-09-09 Eric Botcazou

Re: [Patch, fortran] Parameterized Derived Types

2017-09-09 Thread Paul Richard Thomas
Dear All, The patch has been committed as revision 251925. I look forward to "feedback" (aka PRs) in the coming weeks. I will return to Parameterized Derived Types at the end of this month to clear up some of the known deficiencies (see the notes attached to the patch submission) and any PRs that

[Ada] Internal error on volatile array with VFA component

2017-09-09 Thread Eric Botcazou
It's caused by a mode mismatch between variants of the same type. Tested on x86_64-suse-linux, applied on the mainline and 7 branch. 2017-09-09 Eric Botcazou * gcc-interface/decl.c (gnat_to_gnu_entity): Only set theTYPE_ALIGN_OK and TYPE_BY_REFERENCE_P flags on types after va

[Ada] Improve handling of aliased constant of UNC array type

2017-09-09 Thread Eric Botcazou
Tested on x86_64-suse-linux, applied on the mainline and 7 branch. 2017-09-09 Eric Botcazou * gcc-interface/decl.c (gnat_to_gnu_entity) : Apply the promotion to static memory earlier in the processing. -- Eric BotcazouIndex: gcc-interface/decl.c =

Re: [PATCH] Add macro DISABLE_COPY_AND_ASSIGN

2017-09-09 Thread Ian Lance Taylor
On Sat, Sep 9, 2017 at 2:45 AM, Yao Qi wrote: > On Fri, Aug 11, 2017 at 3:14 PM, Pedro Alves >> Yeah, this is a macro that lots of projects out there reinvent, >> can't imagine it being very controversial. >> >> I could have used this today in another spot in gdb. >> >> The patch as is touches are

[Ada] Pragma Atomic wrongly rejected on composite component

2017-09-09 Thread Eric Botcazou
The compiler wrongly rejects a pragma Atomic on a component of a record whose type is composite, but it accepts the pragma on a variable of the same type. Tested on x86_64-suse-linux, applied on the mainline and 7 branch. 2017-09-09 Eric Botcazou * gcc-interface/decl.c (promote_obje

[Ada] Fix wrong derivation of record type into unchecked union

2017-09-09 Thread Eric Botcazou
It's a fallout of the new implementation of layout for derived record types. Tested on x86_64-suse-linux, applied on the mainline. 2017-09-09 Eric Botcazou * gcc-interface/decl.c (gnat_to_gnu_entity) : Copy the layout of the record from the parent type only if both are or are

[Ada] Override inlining heuristics for expression functions

2017-09-09 Thread Eric Botcazou
Expression functions are supposed to be very small so it makes sense to inline them in almost all cases. Tested on x86_64-suse-linux, applied on the mainline. 2017-09-09 Eric Botcazou * gcc-interface/trans.c (Subprogram_Body_to_gnu): Disregard inlining limits for expression

[Ada] Take into account alignment gap for component reordering

2017-09-09 Thread Eric Botcazou
This disables the recently implemented component reordering in one more case. Tested on x86_64-suse-linux, applied on the mainline. 2017-09-09 Eric Botcazou * gcc-interface/decl.c (components_to_record): Do not reorder in non- packed record types if pragma Optimize_Alignment

[Ada] Fix fallout of component reordering with -fgnat-encodings=minimal

2017-09-09 Thread Eric Botcazou
Tested on x86_64-suse-linux, applied on the mainline. 2017-09-09 Pierre-Marie de Rodat * gcc-interface/decl.c (gnat_to_gnu_entity) : Don't generate debug info for inner record types if -fgnat-encodings=minimal (gnat_to_gnu_entity) : Use the ultimate base record

Re: [PATCH] Improve -Ofast vectorization of std::sin etc. (PR libstdc++/81706)

2017-09-09 Thread Jason Merrill
On Fri, Sep 1, 2017 at 1:12 PM, Jakub Jelinek wrote: > + tree s = lookup_attribute ("omp declare simd", > +DECL_ATTRIBUTES (newdecl)); > + if (s) > + { > + tree b > +

Re: [wwwdocs] Update reference to MMIX

2017-09-09 Thread Tim Song
On Sat, Sep 9, 2017 at 2:34 AM, Gerald Pfeifer wrote: > + href="http://www-cs-faculty.stanford.edu/~knuth/mmix.htmll";>MMIX This one has one too many l's.

Re: C/C++ PATCH to add __remove_qualifiers (PR c/65455, c/39985)

2017-09-09 Thread Jason Merrill
On Fri, Sep 1, 2017 at 2:46 PM, Marek Polacek wrote: > @@ -16960,6 +16961,24 @@ cp_parser_simple_type_specifier (cp_parser* parser, > >return type; > > +case RID_REMOVE_QUALS: > + /* Consume the `__remove_qualifiers' token. */ > + cp_lexer_consume_token (parser->lexer); > +

Re: [wwwdocs] Update reference to MMIX

2017-09-09 Thread Gerald Pfeifer
On Sat, 9 Sep 2017, Tim Song wrote: >> + href="http://www-cs-faculty.stanford.edu/~knuth/mmix.htmll";>MMIX > This one has one too many l's. You've got sharp eyes, Tim! Thanks for catching this; fixed thusly. Gerald Index: readings.html =

Re: [PATCH] Early LTO debug TODO

2017-09-09 Thread Jason Merrill
On Thu, Aug 31, 2017 at 4:03 PM, Richard Biener wrote: > On Thu, 31 Aug 2017, Richard Biener wrote: > >> >> As suspected during review the DECL_ABSTRACT_P handling in >> gen_formal_parameter_die is no longer necessary so the following >> patch removes it. >> >> [LTO] bootstrapped on x86_64-unknown

Re: Announcing ARM and AArch64 port maintainers.

2017-09-09 Thread James Greenhalgh
On Sat, Sep 09, 2017 at 12:44:14PM +0100, Ramana Radhakrishnan wrote: > I'm pleased to announce that the steering committee has appointed > > - James Greenhalgh as a full maintainer for the AArch64 port > > and > > - Kyrylo Tkachov as a full maintainer for the ARM port. > > James & Kyrylo, if yo

Re: [PATCH, DWARF] Add DW_CFA_AARCH64_negate_ra_state to dwarf2.def/h and dwarfnames.c

2017-09-09 Thread Jason Merrill
On Thu, Aug 10, 2017 at 6:39 PM, Jiong Wang wrote: > Hi, > > A new vendor CFA DW_CFA_AARCH64_negate_ra_state was introduced for > ARMv8.3-A > return address signing, it is multiplexing DW_CFA_GNU_window_save in CFA > vendor > extension space. > > This patch adds necessary code to make it avail

Re: [PATCH] streambuf_iterator: avoid debug-dependent behaviour

2017-09-09 Thread François Dumont
Hi Completing the execution of tests revealed a lot about the current implementation. The main point of current implementation is to delay as much as possible the capture of the current streambuf position. So my original proposal capturing state on instantiation was wrong. This

Re: [PATCH] Early LTO debug TODO

2017-09-09 Thread Jason Merrill
On Sat, Sep 9, 2017 at 4:00 PM, Jason Merrill wrote: > On Thu, Aug 31, 2017 at 4:03 PM, Richard Biener wrote: >> On Thu, 31 Aug 2017, Richard Biener wrote: >> >>> >>> As suspected during review the DECL_ABSTRACT_P handling in >>> gen_formal_parameter_die is no longer necessary so the following >>