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

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" > "@internal Vector memory operand." >

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

Successfully bootstrapped on x86_64-pc-linux-gnu. OK for trunk? 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

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

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

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

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}

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

Hi, 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 supported.". Using asm code is generally not

Re: [PATCHES, PING*5] Enhance standard DWARF for Ada

On 01/03/2016 12:45 AM, Eric Botcazou wrote: On powerpc64: FAIL: gnat.dg/specs/debug1.ads scan-assembler-times DW_AT_artificial 18 $ grep -c DW_AT_artificial debug1.s 17 Test removed. Thank you both! -- Pierre-Marie de Rodat

Re: [ping] pending patches

> It doesn't look to me like DW_AT_endianity is applicable to array types > or members in DWARF 3/4; instead, it should be applied to the underlying > base type. OK, the attached patch does that so is more invasive as expected. Tested on x86_64-suse-linux, OK for the mainline? 2016-01-05 Eric

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

On Sat, 2 Jan 2016 11:50:56 + Sergei Trofimovich wrote: > 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 both clean and patched trees.

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

Bernd Edlinger writes: > 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

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

Bernd Edlinger writes: > Hi, > > 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

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

> '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. -- Eric Botcazou

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

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

Re: [patch] ARM FreeBSD fix bootstrap

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: [PATCH] libiberty: dupargv: rewrite to use xstrdup

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.

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

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. *

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

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

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

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

Re: adjust fallback_frame_state for 32bits AIX 7.1

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

Re: adjust fallback_frame_state for 32bits AIX 7.1

> 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

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

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

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

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

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

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

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

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

[PATCH][MIPS] Reorder function types

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

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

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" > >

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

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,

RE: [PATCH][MIPS] Reorder function types

> -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

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

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

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

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 > >+

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

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

RE: [PATCH][MIPS] Reorder function types

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

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

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: [Bug c++/58583] NSDMI parsing

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

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

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.

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

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

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

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

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

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

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

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

Re: [patch] ARM FreeBSD fix bootstrap

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

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

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

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

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

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

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

Re: [PATCH] Fix Fortran deviceptr clause.

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

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

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 >

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

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

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

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

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

[ 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,

[gomp4] merge ptx changes

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

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

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

Re: [PATCH] OpenACC documentation for libgomp

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 @@

[doc, committed] document MicroBlaze interrupt_handler and fast_interrupt attributes

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

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

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

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

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

[gomp4] Non-unity default partition size

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

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

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

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

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

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

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

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

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

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

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

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

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

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:

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

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

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

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

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

> 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

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

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

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

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

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

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

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

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

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

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

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

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

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] Fix -fdump-tree-fre ICE (PR tree-optimization/69141)

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.

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

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

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

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

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

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

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

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

[doc, committed] document RL78 saddr attribute

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: [doc, committed] document MicroBlaze interrupt_handler and fast_interrupt attributes

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

[doc, committed] document V850 function and variable attributes

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

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

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

adjust fallback_frame_state for 32bits AIX 7.1

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

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

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] c/68966 - atomic_fetch_* on atomic_bool not diagnosed

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

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

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: RFA: PR 66655: Use COFF/PE weak symbols

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.

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

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 >> >>

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

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: [doc, committed] document RL78 saddr attribute

> 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] Fix -Wmisleading indentation false-positive for do-while statement

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: [patch] ARM FreeBSD fix bootstrap

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

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

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

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

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

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

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

  1   2   >