Re: [C++ PATCH] hash-table for extern-c fns.

2017-10-09 Thread Nathan Sidwell
On 10/09/2017 11:57 AM, Jason Merrill wrote: On Fri, Oct 6, 2017 at 10:18 AM, Nathan Sidwell wrote: This patch converts the extern "C" function map to use a hash table, in the same way as I've just changed namespace bindings. Hmm, why do we only check extern "C" conflicts for functions? I

Re: [PATCH] Enable ifunc attribute by default for ARM GNU/Linux

2017-10-09 Thread Joseph Myers
On Mon, 9 Oct 2017, Adhemerval Zanella wrote: > *-*-linux*) > case ${target} in > - aarch64*-* | i[34567]86-* | powerpc*-* | s390*-* | sparc*-* | x86_64-*) > + aarch64*-* | i[34567]86-* | powerpc*-* | s390*-* | sparc*-* | x86_64-* > | arm*-*) I think the cases here are meant to be

Re: [PATCH][mingw] Enable colorized diagnostics

2017-10-09 Thread Manuel López-Ibáñez
On 08/10/17 12:39, Liu Hao wrote: On 2017/9/28 4:09, Joseph Myers wrote: On Thu, 28 Sep 2017, Liu Hao wrote: Colorized diagnostics used to be disabled for MinGW targets (on which the macro `_WIN32` is defined), and this patch enables it. I'd hope this is all to do with MinGW host, and nothin

Re: [PATCH] Add a warning for invalid function casts

2017-10-09 Thread Bernd Edlinger
On 10/09/17 20:34, Martin Sebor wrote: > On 10/09/2017 11:50 AM, Bernd Edlinger wrote: >> On 10/09/17 18:44, Martin Sebor wrote: >>> On 10/07/2017 10:48 AM, Bernd Edlinger wrote: Hi! I think I have now something useful, it has a few more heuristics added, to reduce the number of

Re: Restore functional DONT_USE_BUILTIN_SETJMP support

2017-10-09 Thread Eric Botcazou
> You're right, not clear why I didn't spot it on Aarch64. Hunk reverted. The attached patch replaces it with an ad-hoc definition of setjmp, like the ones used for the coverage routines. Tested on Aarch64/Linux with and without --enable-sjlj-exceptions and on IA-64/Linux without --enable-sjlj

[PATCH] rs6000: Move processor_costs to header

2017-10-09 Thread Segher Boessenkool
This moves the processor_costs structure declaration to the rs6000.h header file, so that things in other files can use it (not just things in rs6000.c). Tested as usual; committing. 2010-10-09 Segher Boessenkool * config/rs6000/rs6000.c (processor_costs): Move to ... * confi

Re: [PATCH, rs6000] fix-up int128 fold vector multiply tests

2017-10-09 Thread Segher Boessenkool
Hi Will, On Mon, Oct 09, 2017 at 04:27:54PM -0500, Will Schmidt wrote: > Fix up a few issues with the tests. > - add -O2 to int128-p8 test, and firm up the -mcpu options > - update the set of expected instructions for the int128-p9 test. > - replace the float128_hw requirement with

Re: [PATCH][mingw] Enable colorized diagnostics

2017-10-09 Thread Manuel López-Ibáñez
On 09/10/17 23:25, Manuel López-Ibáñez wrote: Even if the host-specific part is not done, I honestly think it is a good idea to match grep's code as much as possible since we may want to merge bugfixes between the two and eventually this code may end up in gnulib. Moreover, if somebody else imp

Re: [PATCH][mingw] Enable colorized diagnostics

2017-10-09 Thread Liu Hao
On 2017/10/10 6:25, Manuel López-Ibáñez wrote: For what is worth, the color output of GCC comes originally from grep, and grep does have code for colorizing in Windows: http://git.savannah.gnu.org/cgit/grep.git/tree/lib and there are significant differences with this patch. For once,   /* $T

Re: [PATCH] Fix ICEs with ss{e,e2,se3} builtins with -mno-mmx (PR target/82483)

2017-10-09 Thread H.J. Lu
On 10/9/17, Jakub Jelinek wrote: > On Mon, Oct 09, 2017 at 09:28:58PM +0200, Uros Bizjak wrote: >> On Mon, Oct 9, 2017 at 9:16 PM, Jakub Jelinek wrote: >> > Hi! >> > >> > The following builtins have {SSE,SSE2,SSSE3} CPUID, yet they have >> > MMX register operands (either input or output or both).

Re: [PATCH] libstdc++: istreambuf_iterator proxy

2017-10-09 Thread Petr Ovtchenkov
On Sun, 8 Oct 2017 16:59:39 +0200 François Dumont wrote: > ... > >>> We have three issues with istreambuf_iterator: > >>> - debug-dependent behaviour > >> Fixed. > > + __glibcxx_requires_cond(_M_sbuf, > > _M_message(__gnu_debug::__msg_inc_istreambuf) > >

Re: [patch, libfortran] Fix EXECUTE_COMMAND_LINE error return

2017-10-09 Thread Janne Blomqvist
On Tue, Oct 10, 2017 at 12:34 AM, Thomas Koenig wrote: > Hello world, > > currently, EXECUTE_COMMAND_LINE aborts the program if the called program > is not found, if cmdstat is present, but cmdmsg isn't. This is a 6/7/8 > regression. > > This patch fixes the issue by simply removing the erronous c

[committed] Fix return type on stack clash protection hook

2017-10-09 Thread Jeff Law
Pointed out in IRC. The intent as well as the docs and prototype had the return type as a bool, but the default implementation used HOST_WIDE_INT. Opps. This fixes the default implementation to match the intent, docs & prototype. Bootstrapped and regression tested. Installing on the trunk as

<    1   2