Re: [PATCH] Fix SLP ICE (PR tree-optimization/69083)

2016-01-05 Thread Jeff Law
On 01/04/2016 01:17 PM, Jakub Jelinek wrote: Hi! The vec-cmp SLP patch added + if (VECTOR_BOOLEAN_TYPE_P (vector_type)) + { + /* Can't use VIEW_CONVERT_EXPR for booleans because +of possibly different sizes of scalar v

[Patch] Fix libstdc++ build when using uclibc instead of glibc

2016-01-05 Thread Steve Ellcey
While trying to build GCC with uclibc instead of glibc I ran into a build failure in libstdc++. uclibc doesn't seem to provide the isfinite function like glibc does so that means that libstdc++ doesn't have std::isfinite. This in turn caused include/ext/random.tcc to not compile because it uses s

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

2016-01-05 Thread Sergei Trofimovich
On Tue, 5 Jan 2016 10:41:14 -0700 Jeff Law wrote: > On 01/05/2016 02:39 AM, Eric Botcazou wrote: > >> 'make bootstrap' works fine on ia64 but I've failed to run 'make check'. > >> Testsuite's LD_LIBRARY_PATHs are not correct and can run tests. > > > > You need to run 'make -k check' from the top

Re: [PATCH] Fix -Wmisleading indentation false-positive for do-while statement

2016-01-05 Thread Bernd Schmidt
On 01/05/2016 10:58 PM, David Malcolm wrote: In general I don't think we can get any useful indentation warning out of a do-while statement, so this patch makes it so that we just skip them. Is this OK to commit after testing? It looks like this is PR c++/69029. FWIW, I agree with your ratio

Re: [PATCH 2/4 v2][AArch64] Add support for FCCMP

2016-01-05 Thread Andrew Pinski
On Tue, Jan 5, 2016 at 2:00 PM, Evandro Menezes wrote: > Hi, Wilco. > > On 12/15/2015 04:32 AM, Wilco Dijkstra wrote: >>> >>> -Original Message- >>> From: Wilco Dijkstra [mailto:wilco.dijks...@arm.com] >>> Sent: 17 November 2015 18:36 >>> To: gcc-patches@gcc.gnu.org >>> Subject: [PATCH 2/4

Re: [PATCH 2/4 v2][AArch64] Add support for FCCMP

2016-01-05 Thread Evandro Menezes
Hi, Wilco. On 12/15/2015 04:32 AM, Wilco Dijkstra wrote: -Original Message- From: Wilco Dijkstra [mailto:wilco.dijks...@arm.com] Sent: 17 November 2015 18:36 To: gcc-patches@gcc.gnu.org Subject: [PATCH 2/4 v2][AArch64] Add support for FCCMP (v2 version removes 4 enums) This patch adds

Re: [PATCH] Fix -Wmisleading indentation false-positive for do-while statement

2016-01-05 Thread David Malcolm
On Tue, 2015-12-22 at 12:33 -0500, Patrick Palka wrote: > We are emitting a bogus warning for the code > > do foo (0); while (flagA); > ^--- NEXT > ^ BODY > ^--- GUARD > > In general I don't think we can get any useful indentation warning out > of a

Re: [doc, committed] document RL78 saddr attribute

2016-01-05 Thread DJ Delorie
> I've checked in this patch to add some minimal documentation for the > RL78 "saddr" variable attribute. That's pretty much all there is to say about the saddr attribute, too.

Re: [PATCH] PR target/68991: Add vector_memory_operand and "Bm" constraint

2016-01-05 Thread Richard Sandiford
Jakub Jelinek writes: > On Mon, Jan 04, 2016 at 03:25:48PM -0800, H.J. Lu wrote: >> LRA is fine. I should use >> >> (define_memory_constraint "Bm" >> "@internal Vector memory operand." >> (match_operand 0 "vector_memory_operand")) >> >> instead of >> >> (define_constraint "Bm" >> "@inter

Re: [patch] ARM FreeBSD fix bootstrap

2016-01-05 Thread Andreas Tobler
On 05.01.16 11:32, Richard Earnshaw (lists) wrote: On 23/12/15 19:28, Andreas Tobler wrote: 2015-12-23 Andreas Tobler * config/arm/freebsd.h: Rename SUBTARGET_OVERRIDE_OPTIONS to SUBTARGET_OVERRIDE_INTERNAL_OPTIONS. Adjust to check unaligned_access on the gcc_options set.

Re: [PATCH] libiberty: support demangling of rvalue reference typenames

2016-01-05 Thread Ian Lance Taylor
On Tue, Jan 5, 2016 at 12:38 PM, Artemiy Volkov wrote: > On Tue, Jan 05, 2016 at 07:26:36AM -0800, Ian Lance Taylor wrote: > >> If gdb is using the code in cplus-dem.c in any way for a program >> compiled in the last 15 years, then gdb is doing something horribly >> wrong. What function is gdb ca

[doc, committed] document RL78 saddr attribute

2016-01-05 Thread Sandra Loosemore
I've checked in this patch to add some minimal documentation for the RL78 "saddr" variable attribute. -Sandra 2016-01-05 Sandra Loosemore PR 1078 gcc/ * doc/extend.texi (RL78 Variable Attributes): New section. Index: gcc/doc/extend.texi

Re: [PATCH] Fix pr69012 ICE on building libgfortran for mips

2016-01-05 Thread Richard Sandiford
Matthew Fortune writes: > Bernd Edlinger writes: >> On 30.12.2015 15:31, Richard Sandiford wrote: >> > I think the problem is deeper than that though. The instructions that >> > are triggering the ICE are only generated by the prologue, so this >> > means that we're trying to lay out the frame ag

Re: [PATCH] libiberty: support demangling of rvalue reference typenames

2016-01-05 Thread Artemiy Volkov
On Tue, Jan 05, 2016 at 07:26:36AM -0800, Ian Lance Taylor wrote: > On Tue, Jan 5, 2016 at 5:59 AM, Artemiy Volkov wrote: > > On Tue, Jan 05, 2016 at 05:26:28AM -0800, Ian Lance Taylor wrote: > >> Artemiy Volkov writes: > >> > >> > On Mon, Jan 04, 2016 at 10:06:44PM -0800, Ian Lance Taylor wrote:

Re: [PATCH] libiberty: {count,dup,write}argv: constify argv input slightly

2016-01-05 Thread Mike Frysinger
On 05 Jan 2016 07:32, Ian Lance Taylor wrote: > On Sat, Jan 2, 2016 at 10:39 PM, Mike Frysinger wrote: > > Would be more useful if we could use "const char * const *", but there's > > a long standing bug where gcc warns about incompatible pointers when you > > try to pass in "char **". > > That's

Re: [doc, committed] document MicroBlaze interrupt_handler and fast_interrupt attributes

2016-01-05 Thread Michael Eager
On 01/05/2016 10:08 AM, Sandra Loosemore wrote: I've checked in this patch to add some minimal documentation for the MicroBlaze interrupt handler attributes. I was able to figure this out by inspection of the code and reading the processor documentation, which has a more extensive discussion o

Re: [C++ PATCH] Fix #pragma implementation diagnostics (PR c++/69145)

2016-01-05 Thread David Malcolm
On Tue, 2016-01-05 at 19:23 +0100, Jakub Jelinek wrote: > Hi! > > Now that input_location can be adhoc location (if it represents a location > range rather than a single loc and it is long enough), we need to avoid > passing it to cpp_included_before which compares locations as numbers. > This can

Re: [PATHCH] Disable inline asm for in-tree mpfr (PR69134)

2016-01-05 Thread Bernd Edlinger
On 05.01.2016 20:23 Jeff Law wrote: > On 01/05/2016 11:52 AM, Bernd Edlinger wrote: > >> Maybe, that can of course improve things but... >> >> I tried to update to gmp 6.0.0 a while ago, and I noticed that it won't >> build unless we also update to latest mpfr and mpc at the same time. > I just che

[doc, committed] document V850 function and variable attributes

2016-01-05 Thread Sandra Loosemore
In reviewing the state of target-specific attribute documentation for PR1078, I saw that there was no documentation for any of the V850 attributes. I was able to fill in the information from reading the code and referring to the existing docs for the -msda/-mtda/-mzda options. -Sandra 2016-0

Re: [CilkPlus] fix issues when cilk_spawn is used with nontrivial expressions

2016-01-05 Thread Jeff Law
On 12/25/2015 09:08 AM, Ryan Burn wrote: This patch fixes issues with cilk_spawn where bad diagnostics are emitted for expressions invoking conversion operators or constructor calls (PR69024, PR68997). It also fixes an issue with a missing CLEANUP_POINT_EXPR that causes an ICE when gimplifying c

Re: [PATCH] PR target/68991: Add vector_memory_operand and "Bm" constraint

2016-01-05 Thread Uros Bizjak
On Tue, Jan 5, 2016 at 8:20 PM, H.J. Lu wrote: > On Tue, Jan 5, 2016 at 11:14 AM, Uros Bizjak wrote: >> On Tue, Jan 5, 2016 at 7:58 PM, H.J. Lu wrote: >>> On Tue, Jan 5, 2016 at 4:32 AM, H.J. Lu wrote: On Tue, Jan 5, 2016 at 12:11 AM, Jakub Jelinek wrote: > On Mon, Jan 04, 2016 at 03:

Re: [PATHCH] Disable inline asm for in-tree mpfr (PR69134)

2016-01-05 Thread Jeff Law
On 01/05/2016 11:52 AM, Bernd Edlinger wrote: Maybe, that can of course improve things but... I tried to update to gmp 6.0.0 a while ago, and I noticed that it won't build unless we also update to latest mpfr and mpc at the same time. I just checked -- mpfr-3.1.3 only requires gmp-4.1 or newer

Re: RFA: PR 68913: Provide weak version of __fread_chk for PR61886 test

2016-01-05 Thread Jeff Law
On 12/22/2015 02:54 AM, Nick Clifton wrote: Hi Jeff, PR 68913 notes that the test gcc.dg/lto/pr61886_0.c test fails on targets whose C library does not provide a __fread_chk function. Since the purpose of the test is to show that GCC will correctly discard the invocation of __fread

Re: [PATCH] PR target/68991: Add vector_memory_operand and "Bm" constraint

2016-01-05 Thread H.J. Lu
On Tue, Jan 5, 2016 at 11:14 AM, Uros Bizjak wrote: > On Tue, Jan 5, 2016 at 7:58 PM, H.J. Lu wrote: >> On Tue, Jan 5, 2016 at 4:32 AM, H.J. Lu wrote: >>> On Tue, Jan 5, 2016 at 12:11 AM, Jakub Jelinek wrote: On Mon, Jan 04, 2016 at 03:25:48PM -0800, H.J. Lu wrote: > LRA is fine. I sh

[gomp4] Non-unity default partition size

2016-01-05 Thread Nathan Sidwell
This patch implements support for providing non-unity default values for unspecified compute dimensions. For instance, given: #pragma acc parallel #pragma acc loop gang ... The user has not specified num_gangs, it would be good to choose a value greater than 1. This patch achieves this via

Re: [PATCH] PR target/68991: Add vector_memory_operand and "Bm" constraint

2016-01-05 Thread Uros Bizjak
On Tue, Jan 5, 2016 at 7:58 PM, H.J. Lu wrote: > On Tue, Jan 5, 2016 at 4:32 AM, H.J. Lu wrote: >> On Tue, Jan 5, 2016 at 12:11 AM, Jakub Jelinek wrote: >>> On Mon, Jan 04, 2016 at 03:25:48PM -0800, H.J. Lu wrote: LRA is fine. I should use (define_memory_constraint "Bm" "@

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

2016-01-05 Thread Jeff Law
On 01/05/2016 04:03 AM, Nick Clifton wrote: Hi Jeff, You probably know the capabilities of COFF/PE better than I, so the patch itself is fine. Thanks - committed. Is there any way that test can be shoved into our dejagnu testing harness? I think we've got support somewhere for tests which

Re: [PATCH] PR target/68991: Add vector_memory_operand and "Bm" constraint

2016-01-05 Thread H.J. Lu
On Tue, Jan 5, 2016 at 4:32 AM, H.J. Lu wrote: > On Tue, Jan 5, 2016 at 12:11 AM, Jakub Jelinek wrote: >> On Mon, Jan 04, 2016 at 03:25:48PM -0800, H.J. Lu wrote: >>> LRA is fine. I should use >>> >>> (define_memory_constraint "Bm" >>> "@internal Vector memory operand." >>> (match_operand 0

Re: [PATHCH] Disable inline asm for in-tree mpfr (PR69134)

2016-01-05 Thread Bernd Edlinger
Hi Jeff, On 05.01.2016 19:21, Jeff Law wrote: > On 01/05/2016 06:20 AM, Bernd Edlinger wrote: >> Hi, >> >> On 05.01.2016 13:58, Bernd Schmidt wrote: >>> On 01/05/2016 09:44 AM, Bernd Edlinger wrote: an in-tree mpfr build enables inline asm code, which makes the mips-bootstrap fail,

Re: [PATCH] Don't leak C_MAYBE_CONST_EXPRs into gimplifier from float-cast instrumentation (PR sanitizer/69099)

2016-01-05 Thread Jeff Law
On 01/05/2016 06:03 AM, Marek Polacek wrote: The problem in this PR is that we were leaking C_MAYBE_CONST_EXPRs into the gimplifier. It was created in the float-cast instrumentation in convert(). Fixed by using the fully-folded EXPR only, thus without C_MAYBE_CONST_EXPR. Which means that we can

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

2016-01-05 Thread Martin Sebor
I agree that the rules are sometimes tedious to follow (and rebasing patches just to fix some formatting issues isn't exactly fun). I don't use git to commit either. My "secret" is to enable highlighting of trailing whitespaces in vim ("let c_space_errors=1"), but that's of no use to you I guess

Re: [PATHCH] Disable inline asm for in-tree mpfr (PR69134)

2016-01-05 Thread Richard Biener
On January 5, 2016 2:20:42 PM GMT+01:00, Bernd Edlinger wrote: >Hi, > >On 05.01.2016 13:58, Bernd Schmidt wrote: >> On 01/05/2016 09:44 AM, Bernd Edlinger wrote: >>> an in-tree mpfr build enables inline asm code, which makes the >>> mips-bootstrap fail, >>> because at least mpfr 2.4.2 uses the "

Re: [PATCH] Preserve user alignment on decls (PR middle-end/68960)

2016-01-05 Thread Jeff Law
On 01/05/2016 11:25 AM, Jakub Jelinek wrote: Hi! When a VAR_DECL is copied (e.g. for OpenMP/OpenACC privatization, or when moving SESE region to another function), copy_var_decl only preserves user alignment on types, but not on decls. Fixed thusly, bootstrapped/regtested on x86_64-linux and i6

Re: [PATCH] -Winvalid-memory-model warning not given for stdatomic.h macros (PR c/69104)

2016-01-05 Thread Jeff Law
On 01/05/2016 08:00 AM, Marek Polacek wrote: At present, we fail to warn about wrong memory orders for certain atomic functions. This happens for macros in stdatomic.h. The fix is to use the expansion point location rather than the input location. While at it, I made a trivial fix to the wordi

[PATCH] Preserve user alignment on decls (PR middle-end/68960)

2016-01-05 Thread Jakub Jelinek
Hi! When a VAR_DECL is copied (e.g. for OpenMP/OpenACC privatization, or when moving SESE region to another function), copy_var_decl only preserves user alignment on types, but not on decls. Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2016-01-05 Jakub Jeli

[C++ PATCH] Fix #pragma implementation diagnostics (PR c++/69145)

2016-01-05 Thread Jakub Jelinek
Hi! Now that input_location can be adhoc location (if it represents a location range rather than a single loc and it is long enough), we need to avoid passing it to cpp_included_before which compares locations as numbers. This can't be done on the libcpp side, because cpp_included_before isn't cal

Re: [PATCH] Fix -fdump-tree-fre ICE (PR tree-optimization/69141)

2016-01-05 Thread Jeff Law
On 01/05/2016 11:20 AM, Jakub Jelinek wrote: Hi! On the following testcase we ICE, because fn is __builtin_unreachable that doesn't have to have a cgraph node created. The name method on cgraph nodes calls decl_printable_name, so the following patch does that even if the cgraph node is NULL. Bo

Re: [PATHCH] Disable inline asm for in-tree mpfr (PR69134)

2016-01-05 Thread Jeff Law
On 01/05/2016 06:20 AM, Bernd Edlinger wrote: Hi, On 05.01.2016 13:58, Bernd Schmidt wrote: On 01/05/2016 09:44 AM, Bernd Edlinger wrote: an in-tree mpfr build enables inline asm code, which makes the mips-bootstrap fail, because at least mpfr 2.4.2 uses the "=h" constraint but in config/mips/

[PATCH] Fix -fdump-tree-fre ICE (PR tree-optimization/69141)

2016-01-05 Thread Jakub Jelinek
Hi! On the following testcase we ICE, because fn is __builtin_unreachable that doesn't have to have a cgraph node created. The name method on cgraph nodes calls decl_printable_name, so the following patch does that even if the cgraph node is NULL. Bootstrapped/regtested on x86_64-linux and i686-l

Re: [PATCH][RTL-ifcvt] PR rtl-optimization/68841: Make sure one basic block doesn't clobber CC reg usage of the other

2016-01-05 Thread Kyrill Tkachov
On 05/01/16 17:06, Kyrill Tkachov wrote: On 05/01/16 16:34, Bernd Schmidt wrote: On 01/05/2016 03:22 PM, Kyrill Tkachov wrote: This works around the issue but we don't want to do perform the check for pairs of simple basic blocks because then we'll end up rejecting code that does things like

[doc, committed] document MicroBlaze interrupt_handler and fast_interrupt attributes

2016-01-05 Thread Sandra Loosemore
I've checked in this patch to add some minimal documentation for the MicroBlaze interrupt handler attributes. I was able to figure this out by inspection of the code and reading the processor documentation, which has a more extensive discussion of how to use these features. -Sandra 2016-01-0

Re: [PATCH, PR67627][RFC] broken libatomic multilib parallel build

2016-01-05 Thread Jeff Law
On 12/17/2015 02:51 AM, Szabolcs Nagy wrote: On 16/12/15 17:06, Jeff Law wrote: On 12/04/2015 05:39 AM, Szabolcs Nagy wrote: As described in pr other/67627, the all-multi target can be built in parallel with the %_.lo targets which generate make dependencies that are parsed during the build of

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

2016-01-05 Thread Jeff Law
On 01/02/2016 04:50 AM, Sergei Trofimovich wrote: I did ran tests only on amd64 host thus the execution failure tests. Attached updated patch and test runs on a crosscompiler before and after the change. Now I'm building on real ia64 hardware as make bootstrap make -k check for bot

Re: [PATCH 2/2] Ada: add XVE/XVU parallel types to lexical scopes

2016-01-05 Thread Eric Botcazou
> This change adds TYPE_DECL nodes for XVE/XVU parallel types to lexical > scopes so that these types are properly scoped in the debug information. OK, thanks. -- Eric Botcazou

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

2016-01-05 Thread Jeff Law
On 01/05/2016 02:39 AM, Eric Botcazou wrote: 'make bootstrap' works fine on ia64 but I've failed to run 'make check'. Testsuite's LD_LIBRARY_PATHs are not correct and can run tests. You need to run 'make -k check' from the top level build directory. No worries. I had an ia64 box provisioned wh

Re: [Ping^2][PATCH][GCC][ARM] testcase memset-inline-10.c uses -mfloat-abi=hard but does not check whether target supports it

2016-01-05 Thread Andre Vieira
On 27/11/15 14:28, Andre Vieira wrote: On 12/11/15 15:16, Andre Vieira wrote: On 12/11/15 15:08, Andre Vieira wrote: Hi, This patch changes the memset-inline-10.c testcase to make sure that it is only compiled for ARM targets that support -mfloat-abi=hard using the fact that all non-thumb1

Re: [PATCH, rs6000] Fix PR target/68872, make -mcpu=powerpc64le pass correct assembler option

2016-01-05 Thread Peter Bergner
On Thu, 2015-12-24 at 10:15 -0800, Mike Stump wrote: > On Dec 22, 2015, at 9:13 AM, Peter Bergner > > gcc/testsuite/ > >PR target/68772 > > Likewise. Please retroactively update the number in the changelog. Wow, thanks for catching this! Fixed everywhere. Peter

Re: [PATCH][RTL-ifcvt] PR rtl-optimization/68841: Make sure one basic block doesn't clobber CC reg usage of the other

2016-01-05 Thread Kyrill Tkachov
On 05/01/16 16:34, Bernd Schmidt wrote: On 01/05/2016 03:22 PM, Kyrill Tkachov wrote: This works around the issue but we don't want to do perform the check for pairs of simple basic blocks because then we'll end up rejecting code that does things like: x = cond ? x + 1 : x - 1 i.e. source of t

RE: [PATCH][MIPS] Reorder function types

2016-01-05 Thread Moore, Catherine
> -Original Message- > From: Matthew Fortune [mailto:matthew.fort...@imgtec.com] > Sent: Tuesday, January 05, 2016 11:45 AM > To: Robert Suchanek; Moore, Catherine > Cc: gcc-patches@gcc.gnu.org > Subject: RE: [PATCH][MIPS] Reorder function types > > Robert Suchanek writes: > > gcc/ > >

RE: [PATCH][MIPS] Reorder function types

2016-01-05 Thread Matthew Fortune
Robert Suchanek writes: > gcc/ > * config/mips/mips-ftypes.def: Sort to lexicographical order. The patch is fine. I don't know what we can/should commit at this stage. Catherine: Any idea what is acceptable? I'd think this kind of small change to be OK and make it easier to maintain MSA out

Re: adjust fallback_frame_state for 32bits AIX 7.1

2016-01-05 Thread Olivier Hainque
> On Jan 5, 2016, at 17:33 , David Edelsohn wrote: > There are no 32 bit kernels for AIX 7.1. This is the signal handler > path for 32 bit environment, but there are no 32 bit kernels. OK, thanks for correcting. >> The patch is very short and in line with what is already there, and we >> tho

Re: [PATCH 2/4] Equate MEM_REFs and ARRAY_REFs in tree-ssa-scopedtables.c

2016-01-05 Thread Jeff Law
On 01/05/2016 09:29 AM, Alan Lawrence wrote: Without looking at the patch, ARRAY_REFs can have non-constant indices which get_ref_base_and_extend handles conservative. You should make sure to not regress here. Thanks for the warning - my understanding is that in such a case, get_ref_base_and_e

Re: [PATCH][RTL-ifcvt] PR rtl-optimization/68841: Make sure one basic block doesn't clobber CC reg usage of the other

2016-01-05 Thread Bernd Schmidt
On 01/05/2016 03:22 PM, Kyrill Tkachov wrote: This works around the issue but we don't want to do perform the check for pairs of simple basic blocks because then we'll end up rejecting code that does things like: x = cond ? x + 1 : x - 1 i.e. source of the set in both blocks reads and writes the

Re: adjust fallback_frame_state for 32bits AIX 7.1

2016-01-05 Thread David Edelsohn
On Tue, Jan 5, 2016 at 6:15 AM, Olivier Hainque wrote: > Hello, > > This is a tiny change we have been using successfully for at least a couple > of years now, improving exception propagation through signal handlers on > 32bits > AIX 7.1. > > While this isn't a complete generalization to all poss

Re: [PATCH 2/4] Equate MEM_REFs and ARRAY_REFs in tree-ssa-scopedtables.c

2016-01-05 Thread Alan Lawrence
On 05/01/16 07:29, Richard Biener wrote: On January 4, 2016 8:08:17 PM GMT+01:00, Jeff Law wrote: On 12/21/2015 06:13 AM, Alan Lawrence wrote: This is a respin of patches https://gcc.gnu.org/ml/gcc-patches/2015-10/msg03266.html and https://gcc.gnu.org/ml/gcc-patches/2015-10/msg03267.html, whic

RE: [PATCH 3/4] Add support to run auto-vectorization tests for multiple effective targets

2016-01-05 Thread Robert Suchanek
Ping. > -Original Message- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On > Behalf Of Robert Suchanek > Sent: 10 August 2015 13:15 > To: catherine_mo...@mentor.com; Matthew Fortune > Cc: gcc-patches@gcc.gnu.org > Subject: [PATCH 3/4] Add support to run auto

RE: [PATCH 1/4] [MIPS] Add support for MIPS SIMD Architecture (MSA)

2016-01-05 Thread Robert Suchanek
Hi, Here is the updated patch for MSA. The patch requires updated MSA tests and preparatory patch that reorders function types. Tested on mips-img-linux-gnu and mips-mti-linux-gnu. Regards, Robert gcc/ChangeLog: * config.gcc: Add MSA header file for mips*-*-* target. * config/

RE: [PATCH 1/4] [MIPS] Add support for MIPS SIMD Architecture (MSA)

2016-01-05 Thread Robert Suchanek
Hi, Comments inlined. > >+;; The attribute gives half modes for vector modes. > >+(define_mode_attr VHMODE > >+ [(V8HI "V16QI") > >+ (V4SI "V8HI") > >+ (V2DI "V4SI") > >+ (V2DF "V4SF")]) > >+ > >+;; The attribute gives double modes for vector modes. > >+(define_mode_attr VDMODE > >+ [(V4S

RE: [PATCH 1/4] [MIPS] Add support for MIPS SIMD Architecture (MSA)

2016-01-05 Thread Robert Suchanek
Hi, Comments inlined. The updated patch will be sent in another email as this message is already long. > Hi Robert, > > Next batch of comments. This set covers the rest of mips-msa.md. > > >+++ b/gcc/config/mips/mips-msa.md > >+(define_expand "vec_perm" > >+ [(match_operand:MSA 0 "register_ope

RE: [PATCH 1/4] [MIPS] Add support for MIPS SIMD Architecture (MSA)

2016-01-05 Thread Robert Suchanek
Hi, Comments inlined. > >diff --git a/gcc/config/mips/constraints.md b/gcc/config/mips/constraints.md > >index 7d1a8ba..cde0196 100644 > >--- a/gcc/config/mips/constraints.md > >+++ b/gcc/config/mips/constraints.md > >@@ -308,6 +308,53 @@ (define_constraint "Yx" > >"@internal" > >(match_o

[PATCH][MIPS] Reorder function types

2016-01-05 Thread Robert Suchanek
Hi, The following patch reorders some of the function types to follow lexicographical order. This patch should go in before the MSA patch. Regards, Robert gcc/ * config/mips/mips-ftypes.def: Sort to lexicographical order. --- gcc/config/mips/mips-ftypes.def | 12 ++-- 1 file

PING^2: [PATCH] Add TYPE_EMPTY_RECORD for C++ empty class

2016-01-05 Thread H.J. Lu
On Mon, Dec 14, 2015 at 2:08 PM, H.J. Lu wrote: > On Mon, Dec 14, 2015 at 12:43 PM, Jason Merrill wrote: >> On 12/14/2015 03:39 PM, H.J. Lu wrote: >>> >>> On Mon, Dec 14, 2015 at 12:16 PM, Jason Merrill wrote: On 12/12/2015 01:42 PM, Marc Glisse wrote: > > > On Sat, 12 Dec

Re: [PATCH] Fix Fortran deviceptr clause.

2016-01-05 Thread James Norris
Ping! Thanks, Jim On 12/06/2015 02:10 PM, James Norris wrote: Hi, Attached is a patch that fixes some runtime issues dealing with the deviceptr clause in Fortran. There were some corner cases that were not being dealt with correctly, e.g., specifying the deviceptr clause and not using the va

Re: [PATCH] Remove use of 'struct map' from plugin (nvptx)

2016-01-05 Thread James Norris
Ping! Thanks, Jim On 12/16/2015 02:35 PM, James Norris wrote: Hi, The attached patch removes the use of the map structure (struct map) from the NVPTX plugin. Regtested on x86_64-pc-linux-gnu Ok for trunk? Thanks! Jim ChangeLog = 2015-12-XX James Norris libgomp/

Re: [PATCH] OpenACC documentation for libgomp

2016-01-05 Thread James Norris
Hi! I've updated the original patch after some very helpful comments from Sandra (thank you, thank you). OK to commit to trunk? Thanks! Jim diff --git a/libgomp/ChangeLog b/libgomp/ChangeLog index 87ec337..fc7b9fe 100644 --- a/libgomp/ChangeLog +++ b/libgomp/ChangeLog @@ -1,3 +1,8 @@ +2016-01-

Re: [PATCH] libiberty: {count,dup,write}argv: constify argv input slightly

2016-01-05 Thread Ian Lance Taylor
On Sat, Jan 2, 2016 at 10:39 PM, Mike Frysinger wrote: > Would be more useful if we could use "const char * const *", but there's > a long standing bug where gcc warns about incompatible pointers when you > try to pass in "char **". That's not a bug. It's how C works. http://c-faq.com/ansi/cons

Re: [PATCH] libiberty: dupargv: rewrite to use xstrdup

2016-01-05 Thread Ian Lance Taylor
On Sat, Jan 2, 2016 at 10:36 PM, Mike Frysinger wrote: > This func is basically open coding the xstrdup function, so gut it > and use it directly. > > 2016-01-03 Mike Frysinger > > * argv.c (dupargv): Replace strlen/xmalloc/strcpy with xstrdup. This is OK. Thanks. Ian

Re: [PATCH] libiberty: support demangling of rvalue reference typenames

2016-01-05 Thread Ian Lance Taylor
On Tue, Jan 5, 2016 at 5:59 AM, Artemiy Volkov wrote: > On Tue, Jan 05, 2016 at 05:26:28AM -0800, Ian Lance Taylor wrote: >> Artemiy Volkov writes: >> >> > On Mon, Jan 04, 2016 at 10:06:44PM -0800, Ian Lance Taylor wrote: >> >> Artemiy Volkov writes: >> >> >> >> > This patch adds handling of 'O'

[PING][PATCH] Mark symbols in offload tables with force_output in read_offload_tables

2016-01-05 Thread Tom de Vries
[ was: Re: [PATCH] Handle BUILT_IN_GOACC_PARALLEL in ipa-pta ] On 16/12/15 17:02, Tom de Vries wrote: On 10/12/15 14:14, Tom de Vries wrote: [ copy-pasting-with-quote from https://gcc.gnu.org/ml/gcc-patches/2015-12/msg00420.html , for some reason I didn't get this email ] On Thu, 3 Dec 2015,

[PATCH] -Winvalid-memory-model warning not given for stdatomic.h macros (PR c/69104)

2016-01-05 Thread Marek Polacek
At present, we fail to warn about wrong memory orders for certain atomic functions. This happens for macros in stdatomic.h. The fix is to use the expansion point location rather than the input location. While at it, I made a trivial fix to the wording of one of the warnings: don't start with a c

Re: [Bug c++/58583] NSDMI parsing

2016-01-05 Thread Nathan Sidwell
On 12/31/15 16:49, Jason Merrill wrote: Maybe disable the fold if parsing_nsdmi()? thanks. I've applied the attached. Richard, I've augmented the comment about the folding to emphasize the checkingness. nathan 2016-01-05 Nathan Sidwell gcc/cp/ PR c++/58583 * pt.c (build_non_dependen

Re: [RFC][PATCH, ARM 1/8] Add support for ARMv8-M's Security Extensions flag and intrinsics

2016-01-05 Thread Andre Vieira
On 31/12/15 20:54, Joseph Myers wrote: On Sat, 26 Dec 2015, Thomas Preud'homme wrote: +#define CMSE_TT_ASM(flags) \ +{ \ + cmse_address_info_t result; \ + __asm__ ("tt" # flags " %0,%1" \ + : "=r"(result) \ + : "r"(p) \ + : "memory"); \ + return result; \ Are th

[PATCH 2/2] Ada: add XVE/XVU parallel types to lexical scopes

2016-01-05 Thread Pierre-Marie de Rodat
Hello, This change adds TYPE_DECL nodes for XVE/XVU parallel types to lexical scopes so that these types are properly scoped in the debug information. Ok to commit? Thank you in advance! gcc/ada/ChangeLog: * gcc-interface/utils.c (rest_of_record_type_compilation): Add XVE/XVU pa

[PATCH 1/2] DWARF: process all TYPE_DECL nodes when iterating on scopes

2016-01-05 Thread Pierre-Marie de Rodat
Hello, In Ada, it is possible to have nested subprograms in the following configuration: procedure Parent is type T; [...] procedure Child (Value : T) is begin [...] end Child; begin [...] end Parent; If we generate debugging infor

[gomp4] merge ptx changes

2016-01-05 Thread Nathan Sidwell
This patch merges my most recent sequence of ptx backend changes to the gomp4 branch. nathan 2016-01-05 Nathan Sidwell Merge from mainline: 2015-12-30 Nathan Sidwell * config/nvptx/nvptx.c (nvptx_assemble_undefined_decl): Check it's not a constant pool object. 2015-12-28 Nathan Si

Re: [PATCH][RTL-ifcvt] PR rtl-optimization/68841: Make sure one basic block doesn't clobber CC reg usage of the other

2016-01-05 Thread Kyrill Tkachov
On 05/01/16 13:42, Bernd Schmidt wrote: On 12/21/2015 10:23 AM, Kyrill Tkachov wrote: Here is a version integrating the new checks into bbs_ok_for_cmove_arith. We might as well do it there since it already iterates over all the instructions in the basic blocks. The patch looks somewhat compl

Re: [PATCH 1/3, libgomp] Resolve libgomp plugin deadlock on exit, libgomp proper parts

2016-01-05 Thread Chung-Lin Tang
Patch has been updated to accommodate the gomp_fini_device() removal changes. And ping. On 2015/12/14 11:47 PM, Chung-Lin Tang wrote: > [sorry, forgot to C gcc-patches in last send] > > Hi Jakub, > these patches are a revision of > https://gcc.gnu.org/ml/gcc-patches/2015-08/msg01701.html > since

Re: [patch] ARM FreeBSD fix bootstrap

2016-01-05 Thread Nathan Sidwell
On 01/04/16 16:45, Andreas Tobler wrote: Ping :) I noticed this morning that VxWorks might be affected. To be on the safe side I'd like to propose the attached version since it makes clear where the override belongs to and I don't think hijacking SUBTARGET_OVERRIDE_OPTIONS is a good idea here.

Re: [PATCH] libiberty: support demangling of rvalue reference typenames

2016-01-05 Thread Artemiy Volkov
On Tue, Jan 05, 2016 at 05:26:28AM -0800, Ian Lance Taylor wrote: > Artemiy Volkov writes: > > > On Mon, Jan 04, 2016 at 10:06:44PM -0800, Ian Lance Taylor wrote: > >> Artemiy Volkov writes: > >> > >> > This patch adds handling of 'O' (rvalue ref) type codes in the C++ > >> > demangling > >> >

FW: [RFA] Compact EH Patch [Ping * 2]

2016-01-05 Thread Moore, Catherine
Ping, Ping. -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On Behalf Of Moore, Catherine Sent: Monday, December 21, 2015 9:22 AM To: Richard Henderson; gcc-patches@gcc.gnu.org Cc: ja...@redhat.com; Matthew Fortune Subject: RE: [RFA] Compact E

Re: [PATCH][RTL-ifcvt] PR rtl-optimization/68841: Make sure one basic block doesn't clobber CC reg usage of the other

2016-01-05 Thread Bernd Schmidt
On 12/21/2015 10:23 AM, Kyrill Tkachov wrote: Here is a version integrating the new checks into bbs_ok_for_cmove_arith. We might as well do it there since it already iterates over all the instructions in the basic blocks. The patch looks somewhat complicated and asymmetrical to me. I tried to

Re: [PATCH] libiberty: support demangling of rvalue reference typenames

2016-01-05 Thread Ian Lance Taylor
Artemiy Volkov writes: > On Mon, Jan 04, 2016 at 10:06:44PM -0800, Ian Lance Taylor wrote: >> Artemiy Volkov writes: >> >> > This patch adds handling of 'O' (rvalue ref) type codes in the C++ >> > demangling >> > code which is done similarly to the 'R' (regular references) >> > case. It also ad

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

2016-01-05 Thread Nick Clifton
Hi Jeff, PR target/68770 * ira-costs.c (copy_cost): Initialise the t_code field of the sri structure. Can you please reduce the testcase from the BZ so that it can be included in the testsuite? I realize it might take a long time with multidelta because of the need to run the com

Re: [PATHCH] Disable inline asm for in-tree mpfr (PR69134)

2016-01-05 Thread Bernd Edlinger
Hi, On 05.01.2016 13:58, Bernd Schmidt wrote: > On 01/05/2016 09:44 AM, Bernd Edlinger wrote: >> an in-tree mpfr build enables inline asm code, which makes the >> mips-bootstrap fail, >> because at least mpfr 2.4.2 uses the "=h" constraint but in >> config/mips/constraints.md >> we find: "Former

[PATCH] OpenACC use_device clause ICE fix

2016-01-05 Thread Chung-Lin Tang
Hi, we've been encountering an ICE for OpenACC host_data sections, which has a use_device() clause similar to OpenMP use_device_ptr. The ICE happens in make_decl_rtl() for scan-created variables, which IIUC, should not be entered at all for automatic variables. I believe the problem is, unlike ot

[PATCH] Don't leak C_MAYBE_CONST_EXPRs into gimplifier from float-cast instrumentation (PR sanitizer/69099)

2016-01-05 Thread Marek Polacek
The problem in this PR is that we were leaking C_MAYBE_CONST_EXPRs into the gimplifier. It was created in the float-cast instrumentation in convert(). Fixed by using the fully-folded EXPR only, thus without C_MAYBE_CONST_EXPR. Which means that we can drop a parameter of ubsan_instrument_float_cas

Re: [PATHCH] Disable inline asm for in-tree mpfr (PR69134)

2016-01-05 Thread Bernd Schmidt
On 01/05/2016 09:44 AM, Bernd Edlinger wrote: an in-tree mpfr build enables inline asm code, which makes the mips-bootstrap fail, because at least mpfr 2.4.2 uses the "=h" constraint but in config/mips/constraints.md we find: "Formerly the @code{hi} register. This constraint is no longer supp

Re: [PATCH] PR target/68991: Add vector_memory_operand and "Bm" constraint

2016-01-05 Thread H.J. Lu
On Tue, Jan 5, 2016 at 12:11 AM, Jakub Jelinek wrote: > On Mon, Jan 04, 2016 at 03:25:48PM -0800, H.J. Lu wrote: >> LRA is fine. I should use >> >> (define_memory_constraint "Bm" >> "@internal Vector memory operand." >> (match_operand 0 "vector_memory_operand")) >> >> instead of >> >> (define

Re: [PATCH] Fix PR c/69122 (-Wmisleading-indentation false positive with empty macros)

2016-01-05 Thread Bernd Schmidt
On 01/05/2016 09:43 AM, David Malcolm wrote: gcc/c-family/ChangeLog: PR c/69122 * c-indentation.c (get_visual_column): Remove default argument. (should_warn_for_misleading_indentation): For the multiline case, update call to get_visual_column for next_stmt_exploc s

Re: [PATCH][RTL-ifcvt] PR rtl-optimization/68841: Make sure one basic block doesn't clobber CC reg usage of the other

2016-01-05 Thread Kyrill Tkachov
Ping. https://gcc.gnu.org/ml/gcc-patches/2015-12/msg01966.html Thanks, Kyrill On 21/12/15 09:23, Kyrill Tkachov wrote: On 18/12/15 13:16, Bernd Schmidt wrote: On 12/18/2015 02:07 PM, Kyrill Tkachov wrote: In this PR we have a THEN and an ELSE block where one uses the condition reg from the p

adjust fallback_frame_state for 32bits AIX 7.1

2016-01-05 Thread Olivier Hainque
Hello, This is a tiny change we have been using successfully for at least a couple of years now, improving exception propagation through signal handlers on 32bits AIX 7.1. While this isn't a complete generalization to all possible configurations (haven't had the time to converge on 64bits kernels

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

2016-01-05 Thread Nick Clifton
Hi Jeff, You probably know the capabilities of COFF/PE better than I, so the patch itself is fine. Thanks - committed. Is there any way that test can be shoved into our dejagnu testing harness? I think we've got support somewhere for tests which require multiple input files. dg-additional

Re: [PATCH, testsuite] Fix g++.dg/pr67989.C test failure when running with -march or -mcpu

2016-01-05 Thread Kyrill Tkachov
On 05/01/16 10:52, Richard Earnshaw (lists) wrote: On 05/01/16 10:47, Kyrill Tkachov wrote: Hi Thomas, On 05/01/16 07:37, Thomas Preud'homme wrote: Hi, g++.dg/pr67989.C passes -march=armv4t to gcc when compiling which fails if RUNTESTFLAGS passes -mcpu or -march with a different value. This

Re: [PATCH, testsuite] Fix g++.dg/pr67989.C test failure when running with -march or -mcpu

2016-01-05 Thread Richard Earnshaw (lists)
On 05/01/16 10:47, Kyrill Tkachov wrote: > Hi Thomas, > > On 05/01/16 07:37, Thomas Preud'homme wrote: >> Hi, >> >> g++.dg/pr67989.C passes -march=armv4t to gcc when compiling which >> fails if >> RUNTESTFLAGS passes -mcpu or -march with a different value. This patch >> adds a >> dg-skip-if direct

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

2016-01-05 Thread Marek Polacek
On Mon, Jan 04, 2016 at 06:18:32PM -0700, Martin Sebor wrote: > I've fixed the problems you pointed out in the attached patch. > The typos are my bad. As for the whitespace, I have to confess > I'm finding all the rules tedious to follow without some sort > of automation. Jason suggested some opt

Re: [PATCH, testsuite] Fix g++.dg/pr67989.C test failure when running with -march or -mcpu

2016-01-05 Thread Kyrill Tkachov
Hi Thomas, On 05/01/16 07:37, Thomas Preud'homme wrote: Hi, g++.dg/pr67989.C passes -march=armv4t to gcc when compiling which fails if RUNTESTFLAGS passes -mcpu or -march with a different value. This patch adds a dg-skip-if directive to skip the test when such a thing happens. ChangeLog entry

Re: [patch] ARM FreeBSD fix bootstrap

2016-01-05 Thread Richard Earnshaw (lists)
On 23/12/15 19:28, Andreas Tobler wrote: > On 23.12.15 11:22, Richard Earnshaw (lists) wrote: >> On 22/12/15 19:53, Andreas Tobler wrote: >>> Hi all, >>> >>> the commit for PR68617 broke boostrap on armv6*-*-freebsd*. >>> >>> We still have unaligned_access = 0 on armv6 here on FreeBSD. >>> >>> The

Re: [PATHCH] Disable inline asm for in-tree mpfr (PR69134)

2016-01-05 Thread Marc Glisse
On Tue, 5 Jan 2016, Marc Glisse wrote: On Tue, 5 Jan 2016, Bernd Edlinger wrote: an in-tree mpfr build enables inline asm code, which makes the mips-bootstrap fail, because at least mpfr 2.4.2 uses the "=h" constraint but in config/mips/constraints.md we find: "Formerly the @code{hi} register

Re: [PATHCH] Disable inline asm for in-tree mpfr (PR69134)

2016-01-05 Thread Marc Glisse
On Tue, 5 Jan 2016, Bernd Edlinger wrote: an in-tree mpfr build enables inline asm code, which makes the mips-bootstrap fail, because at least mpfr 2.4.2 uses the "=h" constraint but in config/mips/constraints.md we find: "Formerly the @code{hi} register. This constraint is no longer support

Re: [PATCH][combine][v2] Canonicalise (r + r) to (r << 1) to aid recognition

2016-01-05 Thread Kyrill Tkachov
Hi all, On 18/12/15 14:39, Bernd Schmidt wrote: On 12/18/2015 03:29 PM, Kyrill Tkachov wrote: Bootstrapped and tested on arm, aarch64, x86_64. As before, there were no codegen differences for SPEC2006 on x86_64. aarch64 SPEC2006 sees the effects described above. I think this is OK. There may

  1   2   >