C++ PATCH for c++/66999 - 'this' captured by reference

2019-06-08 Thread Marek Polacek
This patch improves a diagnostic when parsing a lambda introducer; in particular, we should handle '&this' better than we currently do. clang/icc specifically point out that 'this' cannot be captured by reference, let's do the same. Bootstrapped/regtested on x86_64-linux, ok for trunk? 2019-06-

[PATCH v2] [MIPS] Inhibit trailing .insn if pool is not followed by code

2019-06-08 Thread Faraz Shahbazker
The __pool and __pend symbols are used to mark the beginning and end of inline constant pools in MIPS16 code regions. However if the pool occurs at the boundary of a code region and is not followed by further code, presence of the __pend symbol can confuse the dissassembler in to treating subseque

fold_real_zero_addition_p for vectors

2019-06-08 Thread Marc Glisse
Hello, this small tweak of fold_real_zero_addition_p lets it handle vectors as well. Bootstrap and regtest on x86_64-pc-linux-gnu. 2019-06-10 Marc Glisse PR tree-optimization/62041 * fold-const.c (fold_real_zero_addition_p): Handle vectors. -- Marc GlisseIndex: gcc/fold-c

Re: Test for C++20 p0858 - ConstexprIterator requirements.

2019-06-08 Thread Jonathan Wakely
On 08/06/19 12:05 -0400, Ed Smith-Rowland wrote: On 6/7/19 11:42 AM, Jonathan Wakely wrote: On 01/06/19 15:40 -0400, Ed Smith-Rowland via libstdc++ wrote: On 6/1/19 2:42 PM, Ville Voutilainen wrote: On Sat, 1 Jun 2019 at 21:09, Ed Smith-Rowland <3dw...@verizon.net> wrote: On 5/31/19 6:29 PM,

Re: [Patch, fortran] PR90786 - [7/8/9/10 Regression] ICE on procedure pointer assignment to function with class pointer result

2019-06-08 Thread Andrew Benson
Thanks Paul for the quick fix! On Saturday, June 8, 2019 4:56:46 PM PDT Paul Richard Thomas wrote: > Committed as obvious in revision 272084. > > The problem was that the lhs symbol itself was not being checked as a > proc_pointer - just the expression component. > > I will get on with backporti

Re: Test for C++20 p0858 - ConstexprIterator requirements.

2019-06-08 Thread Ed Smith-Rowland via gcc-patches
On 6/7/19 11:42 AM, Jonathan Wakely wrote: On 01/06/19 15:40 -0400, Ed Smith-Rowland via libstdc++ wrote: On 6/1/19 2:42 PM, Ville Voutilainen wrote: On Sat, 1 Jun 2019 at 21:09, Ed Smith-Rowland <3dw...@verizon.net> wrote: On 5/31/19 6:29 PM, Ville Voutilainen wrote: On Sat, 1 Jun 2019 at 01

[Patch, fortran] PR90786 - [7/8/9/10 Regression] ICE on procedure pointer assignment to function with class pointer result

2019-06-08 Thread Paul Richard Thomas
Committed as obvious in revision 272084. The problem was that the lhs symbol itself was not being checked as a proc_pointer - just the expression component. I will get on with backporting tomorrow. Cheers Paul 2019-06-08 Paul Thomas PR fortran/90786 * trans-expr.c (pointer_assignme

[C++ PATCH] Add test for c++/52269

2019-06-08 Thread Marek Polacek
Continuing scouring the Bugzilla. This one compiles fine since r224008 which deferred instantiating constexpr functions until we actually need the definition. Tested other compilers too; all of them accept the test. Bootstrapped/regtested on x86_64-linux, applying to trunk. 2019-06-08 Marek Po

[patch, committed] PR 90744

2019-06-08 Thread Thomas Koenig
Hello world, I have just committed the attached patch to trunk as obvious an simple. The problem, which led to wrong code, was that copying the "deferred" attribute from the dummy to the formal argument a) makes no sense b) led to wrong code. The analysis and the first working version of the patc

[PATCH] Use consistent spelling of PCLMUL instruction

2019-06-08 Thread Jonathan Wakely
* doc/invoke.texi (C Dialect Options): Minor grammatical change. (x86 Options): Replace all uses of "PCL_MUL" with "PCLMUL" Committed to trunk. commit 94f631371312bcc6bf1582b867b98e9e8dee43fe Author: redi Date: Sat Jun 8 13:40:25 2019 + Use consistent spelling of PCL

[C++ PATCH] Add test for c++/77548

2019-06-08 Thread Marek Polacek
Continuing scouring older C++ PRs. This one ICEd but was fixed by r240098; let's make sure we don't re-introduce that bug. Tested x86_64-linux, applying to trunk. 2019-06-08 Marek Polacek PR c++/77548 * g++.dg/other/pr77548.C: New test. diff --git gcc/testsuite/g++.dg/other/

Re: [C++ PATCH] Add test for c++/72845

2019-06-08 Thread Jakub Jelinek
On Sat, Jun 08, 2019 at 08:48:43AM -0400, Marek Polacek wrote: > This was crashing because our handling of friend decls with > noexcept-specifiers > was broken. Fixed by fairly recent r270005. > > Tested x86_64-linux, applying to trunk. > > 2019-06-08 Marek Polacek > > PR c++/72845.pa

Re: [C++ PATCH] Add test for c++/72845

2019-06-08 Thread Marek Polacek
On Sat, Jun 08, 2019 at 08:48:43AM -0400, Marek Polacek wrote: > This was crashing because our handling of friend decls with > noexcept-specifiers > was broken. Fixed by fairly recent r270005. > > Tested x86_64-linux, applying to trunk. > > 2019-06-08 Marek Polacek > > PR c++/72845.pa

[C++ PATCH] Add test for c++/72845

2019-06-08 Thread Marek Polacek
This was crashing because our handling of friend decls with noexcept-specifiers was broken. Fixed by fairly recent r270005. Tested x86_64-linux, applying to trunk. 2019-06-08 Marek Polacek PR c++/72845.patch * g++.dg/cpp0x/noexcept41.C: New test. diff --git gcc/testsuite/g++

Re: [AArch64] [SVE] PR88837 - Poor vector construction code in VL-specific mode

2019-06-08 Thread Prathamesh Kulkarni
On Fri, 7 Jun 2019 at 22:47, Richard Sandiford wrote: > > Prathamesh Kulkarni writes: > > 2019-06-07 Prathamesh Kulkarni > > > > * gcc.target/aarch64/sve/init_1.c: Remove options > > -O2 -fno-schedule-insns and instead pass -O. > > Update assembly in comments. > > * gcc

Re: V2: [PATCH] Update preferred_stack_boundary only when expanding function call

2019-06-08 Thread Richard Sandiford
"H.J. Lu" writes: > On Fri, Jun 7, 2019 at 1:22 AM Richard Biener wrote: >> >> On Fri, 7 Jun 2019, Richard Sandiford wrote: >> >> > "H.J. Lu" writes: >> > > locate_and_pad_parm is called when expanding function call from >> > > initialize_argument_information and when generating function body >>