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

2017-11-25 Thread Martin Sebor
On 11/22/2017 04:50 PM, Jeff Law wrote: On 11/16/2017 02:29 PM, Martin Sebor wrote: Ping. I've fixed the outstanding false positive exposed by the Linux kernel. The kernel builds with four instances of the warning, all of them valid (perfect overlap in memcpy). I also built Glibc. It shows o

Re: [C++ PATCH] Avoid -Wreturn-type warnings if a switch has default label, no breaks inside of it, but is followed by a break (PR sanitizer/81275)

2017-11-25 Thread Jakub Jelinek
On Sat, Nov 25, 2017 at 10:01:22AM +0100, Jakub Jelinek wrote: > Actually, thinking about it some more, maybe it would be more efficient > to gather this information during construction of the SWITCH_STMT in some > new flag on the tree, so cxx_block_may_fallthru would just: Here it is implemented,

Re: [RFTesting] New POINTER_DIFF_EXPR

2017-11-25 Thread Gerald Pfeifer
On Mon, 20 Nov 2017, Marc Glisse wrote: new version, regtested on powerpc64le-unknown-linux-gnu. The front-end parts are up for review. 2017-10-28 Marc Glisse gcc/c/ * c-fold.c (c_fully_fold_internal): Handle POINTER_DIFF_EXPR. * c-typeck.c (pointer_diff): Use POINTER_DIFF_E

Re: RFA (hash-map): PATCH to support GTY((cache)) with hash_map

2017-11-25 Thread Markus Trippelsdorf
On 2017.11.14 at 13:32 +0100, Richard Biener wrote: > On Fri, Sep 15, 2017 at 11:45 PM, Jason Merrill wrote: > > The hash_map interface is a lot more convenient than that of > > hash_table for cases where it makes sense, but there hasn't been a way > > to get the ggc_cache_remove behavior with a h

Re: [PATCH] Fix combine's simplify_if_then_else (PR rtl-optimization/81553)

2017-11-25 Thread Segher Boessenkool
On Sat, Nov 25, 2017 at 09:39:54AM +0100, Jakub Jelinek wrote: > > Okay, thanks! Is this better than bailing out though, do you have > > an example? > > I don't, but I haven't bootstrapped/regtested with a patch to gather > statistics on whether it would ever be successful (how? Should I note in

[wwwdocs] Shorten bugs/index.html a little

2017-11-25 Thread Gerald Pfeifer
Just having pointed someone to our bug reporting instructions, I had a look myself and noticed that they are a little unwieldy . This is a first step of shortening the text a little. Help and contributions very welcome! Gerald Index: index.html ==

Fix -Wreturn-type fallout

2017-11-25 Thread Andreas Schwab
Installed as obvious. Andreas. * g++.dg/abi/structret1.C (FrameworkObject::action): Return a value. Index: g++.dg/abi/structret1.C === --- g++.dg/abi/structret1.C (revision 255148) +++ g++.dg/abi/structret1.C

[wwwdocs] Adjust a comment in bin/preprocess

2017-11-25 Thread Gerald Pfeifer
I've had this in a local tree since February; better to flush local changes. This just adjust a comment to match the code wrt. ordering of steps. Applied, and I updated the instance on gcc.gnu.org as well. Gerald Index: bin/preprocess

Re: [fortran] Add support for #pragma GCC unroll v3

2017-11-25 Thread Steve Kargl
On Sat, Nov 25, 2017 at 11:21:49AM +0100, Eric Botcazou wrote: > > this is the (hopefully) final implementation of the support for the unrolling > pragma in the Fortran front-end. However the documentation is still missing > because I don't really know where and under which form to put it. > >

Re: [PATCH] Implement std::to_address for C++2a

2017-11-25 Thread Glen Fernandes
(Just a minor update to the last patch to use is_function_v instead of is_function::value) Implement std::to_address for C++2a 2017-11-25 Glen Joseph Fernandes * include/bits/ptr_traits.h (to_address): Implement to_address. * testsuite/20_util/to_address/1.cc: New tests. Test

[fortran] Add support for #pragma GCC unroll v3

2017-11-25 Thread Eric Botcazou
Hi, this is the (hopefully) final implementation of the support for the unrolling pragma in the Fortran front-end. However the documentation is still missing because I don't really know where and under which form to put it. Tested on x86_64-suse-linux, OK for the mainline? 2017-11-25 Bernha

[C/C++] Add support for #pragma GCC unroll v3

2017-11-25 Thread Eric Botcazou
Hi, this is the (hopefully) final implementation of the support for the unrolling pragma in the C and C++ front-ends. It contains a couple of fixes for the C++ front-ends to make it correctly handle unroll and ivdep for the same loop. Tested on x86_64-suse-linux, OK for the mainline? 2017-11

[i386] Mask generation in avx2intrin.h

2017-11-25 Thread Marc Glisse
Hello, the way full masks are generated currently in avx2intrin.h is questionable: opaque for the inline functions, weird/wrong for the macros. It is possible we may want to add code so the constant mask with all ones may be generated with vxorpd+vcmpeqpd instead of loading it from memory, b

Re: [C++ PATCH] Avoid -Wreturn-type warnings if a switch has default label, no breaks inside of it, but is followed by a break (PR sanitizer/81275)

2017-11-25 Thread Jakub Jelinek
On Fri, Nov 24, 2017 at 10:59:53PM +0100, Jakub Jelinek wrote: > The testcase below has a useless break; that causes a bogus -Wreturn-type > warning. The C++ FE already has code to avoid adding a BREAK_STMT > after a return or similar sequence that is known not to return. > The following patch ext

Re: [PATCH][PR target/81535] Fix tests on Power

2017-11-25 Thread Yury Gribov
On Tue, Aug 8, 2017 at 12:32 AM, Segher Boessenkool wrote: > Hi Yuri, > > Sorry I missed this. Please cc: me to prevent that from happening. Segher, Sorry, somehow I missed your reply! > On Fri, Jul 28, 2017 at 05:42:00AM +0100, Yury Gribov wrote: >> This patch fixes issues reported in >> http

Re: [PATCH] Fix combine's simplify_if_then_else (PR rtl-optimization/81553)

2017-11-25 Thread Jakub Jelinek
On Fri, Nov 24, 2017 at 07:19:29PM -0600, Segher Boessenkool wrote: > Hi, > > On Fri, Nov 24, 2017 at 10:38:13PM +0100, Jakub Jelinek wrote: > > The following testcase ICEs in wide-int*, but the reason is a mode mismatch > > (we build a SImode MULT with one QImode argument and one VOIDmode argumen