Re: [PATCH] Fix PR61441 introduced regression (PR tree-optimization/69070)

2016-01-01 Thread Jeff Law
On 12/31/2015 02:29 AM, Jakub Jelinek wrote: Hi! The PR61441 change added REAL_VALUE_ISSIGNALING_NAN checks for both operands, but only arg1 is guaranteed to be a REAL_CST at this point. If arg0 is not a REAL_CST, I think we can allow all the opts the function does, as all transformations replac

Re: [PATCH] c/68966 - atomic_fetch_* on atomic_bool not diagnosed

2016-01-01 Thread Jeff Law
On 12/22/2015 09:46 PM, Martin Sebor wrote: The attached patch rejects invocations of atomic fetch_op intrinsics on objects of _Bool type as discussed in the context of PR c/68908. Tested on x86_64. Martin gcc-68966.patch gcc/testsuite/ChangeLog: 2015-12-22 Martin Sebor PR c/68966

Re: [Patch,tree-optimization]: Add new path Splitting pass on tree ssa representation

2016-01-01 Thread Jeff Law
On 12/25/2015 01:40 AM, Ajit Kumar Agarwal wrote: Hello Jeff: I am out on vacation till 3rd Jan 2016. Is it okay If I respond on the below once I am back in office. Yes. I'm on vacation until then as well. Jeff

Re: [PATCH] ia64: don't use dynamic relocations for local symbols

2016-01-01 Thread Jeff Law
On 12/28/2015 02:54 PM, Sergei Trofimovich wrote: From: Sergei Trofimovich Tested on the following example: void * a[77] __attribute((visibility("hidden"))); void f(long o, void * v) { a[0x6eff - o + 66] = v; } Before the patch generated code uses .GOT entry: addl r14

Re: RFA: PR 68770: Fix use of uninitialised value in secondary_reload

2016-01-01 Thread Jeff Law
On 12/22/2015 03:04 AM, Nick Clifton wrote: Hi Guys, The patch below fixes PR 68770 - a warning from valgrind about an uninitialised value being used in the default_secondary_reload. The problem turns out to the in copy_costs which creates its own secondary reload information struct

Re: RFA: PR 66655: Use COFF/PE weak symbols

2016-01-01 Thread Jeff Law
On 12/22/2015 03:17 AM, Nick Clifton wrote: Hi Guys, The patch below is a proposed fix for PR 66655. The issue I believe is not that the ming32 definition of bind_local_p is wrong, but rather that G++ thinks that it cannot make the decl weak even though bind_local_p says that it sho

Re: [PATCH 1/3] Fix logic bug in Cilk Plus array expansion

2016-01-01 Thread Jeff Law
On 12/31/2015 08:40 AM, Patrick Palka wrote: The Cilk Plus code may sometimes turn a COND_EXPR into an error_mark_node for no good reason. This can be seen by compiling the test case c-c++-common/cilk-plus/CK/pr60469.c with both gcc and g++ and observing the differences of the -fdump-tree-origin

[doc, committed] document no_stack_limit attribute

2016-01-01 Thread Sandra Loosemore
I've checked in this patch to address one of the long-standing documentation bugs reported in PR 1078. -Sandra 2016-01-01 Sandra Loosemore PR 1078 gcc/ * extend.texi (Common Function Attributes) : New. * invoke.texi (Code Gen Options) <-fno-stack-limit>: Add pointer to corresponding a

[patch, c++] delete "com_interface" attribute

2016-01-01 Thread Sandra Loosemore
PR 1078 (yes, that is a very old issue) notes that there is no documentation for the C++ "com_interface" attribute. In fact, this attribute has done nothing but issue a warning saying it is obsolete for almost 15 years (r42465, to be exact). It is hard to imagine that anyone is using it any m

Re: [PATCH] PR/68089: C++-11: Ingore "alignas(0)".

2016-01-01 Thread Martin Sebor
On 12/31/2015 04:50 AM, Dominik Vogt wrote: The attached patch fixes C++-11 handling of "alignas(0)" which should be ignored but currently generates an error message. A test case is included; the patch has been tested on S390x. Since it's a language issue it should be independent of the backend

[doc, committed] tidy function attributes documentation

2016-01-01 Thread Sandra Loosemore
I was starting to work on PR1078 and noticed that some recent-ish entries in the function attributes section were mis-alphabetized in the table and in need of copy-editing. I've checked in this patch to fix that, and the related bits in options.texi. BTW, I also think that the -fstack-protect

Re: [PATCH] * doc/invoke.texi: fix typos of -finite-math-only

2016-01-01 Thread Sandra Loosemore
On 01/01/2016 12:31 PM, Vladimír Čunát wrote: Hello, I noticed tiny typos in the docs. (CC me with replies, please, as I'm not reading the list.) This patch is fine with an appropriate ChangeLog entry. Generally, fixing typos in documentation or comments falls under the "obvious patch" rule,

[PATCH] Fix tilegx libgcc with multilib

2016-01-01 Thread Bernd Edlinger
Hi Walter, while playing with the tilegx cross compiler I noticed another defect. Currently building a tilegx cross compiler fails in libgcc multilib configuration, because of the following static assert in _FP_FROM_INT: _FP_STATIC_ASSERT ((rsize) <= 2 * _FP_W_TYPE_SIZE,\

[PATCH] * doc/invoke.texi: fix typos of -finite-math-only

2016-01-01 Thread Vladimír Čunát
Hello, I noticed tiny typos in the docs. (CC me with replies, please, as I'm not reading the list.) Vladimir From 707de44fdb74ecfbc7120e2d6b5b9fdb4d281855 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Vladim=C3=ADr=20=C4=8Cun=C3=A1t?= Date: Fri, 1 Jan 2016 20:10:56 +0100 Subject: [PATCH] * doc/invok

cilkplus fails without pthreads for me

2016-01-01 Thread Mike Stump
cilkplus fails without pthreads for me: xg++: error: unrecognized command line option '-pthread' compiler exited with status 1 output is: xg++: error: unrecognized command line option '-pthread' FAIL: c-c++-common/attr-simd-3.c -std=gnu++14 PR68158 (test for errors, line 5) I suspect pthreads i

Re: RFD: annotate iterator patterns with expanded forms

2016-01-01 Thread Hans-Peter Nilsson
On Tue, 1 Dec 2015, Bernd Schmidt wrote: > On 12/01/2015 04:31 PM, Bernd Schmidt wrote: > > On 12/01/2015 04:23 PM, Jakub Jelinek wrote: > > > With the comments in the *.md file I'd worry about them getting out of > > > date, > > > or people feeling they have to edit them manually (rather than bein

[Bug fortran/68864] [6 Regression] ICE: in gfc_get_descriptor_dimension, at fortran/trans-array.c:268

2016-01-01 Thread Paul Richard Thomas
Dear All, Happy New Year! I have committed the fix for the above as 'obvious' and entirely harmless - revision 232026. 2016-01-01 Paul Thomas PR fortran/68864 * trans-array.c (evaluate_bound): If deferred, test that 'desc' is an array descriptor before using gfc_conv_descriptor_x

Re: [PATCH] Fix ubsan ICE with non-C/C++ FEs (PR sanitizer/69055)

2016-01-01 Thread Richard Biener
On December 31, 2015 10:17:34 AM GMT+01:00, Jakub Jelinek wrote: >Hi! > >ubsan_instrument_float_cast is called from convert.c, therefore for all >FEs >that uses convert.c. For C/C++, the ubsan builtins are already >initialized >through their inclusion from builtins.def, but for other builtins th

Re: [PATCH] Fix PR61441 introduced regression (PR tree-optimization/69070)

2016-01-01 Thread Richard Biener
On December 31, 2015 10:29:19 AM GMT+01:00, Jakub Jelinek wrote: >Hi! > >The PR61441 change added REAL_VALUE_ISSIGNALING_NAN checks for both >operands, but only arg1 is guaranteed to be a REAL_CST at this point. >If arg0 is not a REAL_CST, I think we can allow all the opts the >function >does, as