Re: [PATCH][MIPS] Enable load-load/store-store bonding

Hi Mike, Thanks for your comments. Please find my comments inlined. - Thanks and regards, Sameera D. On Monday 11 May 2015 10:09 PM, Mike Stump wrote: On May 11, 2015, at 4:05 AM, sameera wrote: +(define_insn "*join2_loadhi" + [(set (match_operand:SI 0 "register_operand" "=r") + (an

Re: [PATCH][PR66010] Don't take address of ap unless necessary

On 05/12/2015 01:45 PM, Tom de Vries wrote: > On 12-05-15 12:04, Tom de Vries wrote: >> Committed with comments below added. > > Hmm, this causes an ice for s390 in gcc.dg/tree-ssa/stdarg-2.c: > ... > gimplification failed: > &ap type type 0x7f132f46b888 __va_list_tag> >

RE: [PATCH, ping 1] Move insns without introducing new temporaries in loop2_invariant

> From: Jeff Law [mailto:l...@redhat.com] > Sent: Wednesday, May 13, 2015 4:05 AM > OK for the trunk. > > Thanks for your patience, Thanks. Committed with the added "PR rtl-optimization/64616" to both ChangeLog entries. Best regards, Thomas

Partial fix for PR66047

Hi, this patch works around quite nasty issue where we compile function with SSE calling convention because it is local, but it is called from function that does nothave SSE (by target attribute or LTO merging). We used to produce silent wrong code here for years, but I added error to block this.

Re: PATCHes to help with C++11 bootstrap

On 05/11/2015 07:30 AM, Markus Trippelsdorf wrote: On 2015.05.08 at 23:30 -0500, Jason Merrill wrote: One C++11 compatibility issue that turns up a lot in the GCC sources is that in C++98, #define BAR "bar" const char *p = "foo"BAR; There was a missing fix for gcc/config/rs6000/option-default

Re: [PATCH, FT32] initial support

On Tue, May 12, 2015 at 10:17:01PM +, James Bowman wrote: > It seems that with whenever a function's frame is bigger than 512 bytes, > LRA loops. Likely this causes a problem because there is no actual > instruction for subtracting constants more than 512. However, there is a > "link" pattern

Re: [PATCH, FT32] initial support

On May 12, 2015, at 3:36 PM, Jeff Law wrote: > > It really depends on the complexity of getting LRA working for the target and > given what I saw when looking at the port, I don't believe it should be much > work. LRA should default to on? Only preexisting ports about ask for, and get non-LR

Re: [PATCH, FT32] initial support

On 05/12/2015 04:17 PM, James Bowman wrote: On 05/08/2015 06:04 PM, James Bowman wrote: Are you using LRA or reload? The former is definitely preferred and for a simple target like this, I'd expect the transition to be very easy. I'm using reload. Attempting to naively switch on LRA resulte

RE: [PATCH, FT32] initial support

> On 05/08/2015 06:04 PM, James Bowman wrote: > > > >> Are you using LRA or reload? The former is definitely preferred and for > >> a simple target like this, I'd expect the transition to be very easy. > > > > I'm using reload. Attempting to naively switch on LRA resulted in > >internal compil

gccgo patch committed: If unary & does not escape, variable does not escape

This patch to the Go frontend notes that if a unary & of a variable was marked as not escaping, then we don't need to set it to be escaping based on the variable. Bootstrapped and ran ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r d50f4d14cfec go/expressions.cc

Work around PR65873

Hi, this patch works around PR where we refuse to inline always_inline memcpy into function with explicit Ofast optimization attribute. This is because we think we can not promote -fno-fast-math code to -fast-math code. This is of course safe for memcpy because it contains to fast-math code, but

Re: [PATCH][expr.c] PR 65358 Avoid clobbering partial argument during sibcall

On 05/11/2015 03:28 AM, Kyrill Tkachov wrote: The more I think about this, the more I think it's an ugly can of worms and maybe we should just disable sibcalls for partial arguments. I doubt it's a big performance issue in general. We already have quite a bit of code in calls.c to detect cases

Re: [Patch, fortran, pr65548, 2nd take, v3] [5/6 Regression] gfc_conv_procedure_call

Hello, Le 30/04/2015 15:07, Andre Vehreschild a écrit : > Hi all, > > this is just a service release. I encountered that the new testcase in the > previous release included the testcase of the initial patch, that is > already on trunk. I therefore replaced the testcase allocate_with_source_5.f90

RE: [RFC]: Remove Mem/address type assumption in combiner

> -Original Message- > From: Steve Ellcey [mailto:sell...@imgtec.com] > Sent: Tuesday, May 12, 2015 12:51 PM > To: Kumar, Venkataramanan > Cc: Segher Boessenkool; Jeff Law (l...@redhat.com); gcc- > patc...@gcc.gnu.org; maxim.kuvyr...@linaro.org; Matthew Fortune; > Moore, Catherine > Subje

Re: [PATCH 1/3] Implement -Wmisleading-indentation (v4)

On Tue, 2015-05-12 at 17:21 -0400, David Malcolm wrote: [...] > Thanks; I've removed the new warning from -Wall, making the appropriate > trivial doc changes, and committed it to trunk (r223098; attached). ...having bootstrapped®rtested it on x86_64-unknown-linux-gnu (Fedora 20).

Re: [PATCH 1/3] Implement -Wmisleading-indentation (v4)

On Mon, 2015-05-11 at 15:23 -0600, Jeff Law wrote: > On 04/28/2015 06:02 PM, David Malcolm wrote: > > This is an updated implementation of the proposed > > -Wmisleading-indentation warning. > > > > Changes since last patch: > > > > * I've rewritten it to use Manuel's approach from > > https://

Re: [PATCH][PR66010] Don't take address of ap unless necessary

On 05/12/2015 06:45 AM, Tom de Vries wrote: Hmm, this causes an ice for s390 in gcc.dg/tree-ssa/stdarg-2.c: ... gimplification failed: &ap type 0x7f132f46b888 __va_list_tag> sizes-gimplified BLK size unit size align 32 symtab 0 alias set

Re: [patch, fortran] Fix PR 66111

Hello, Le 12/05/2015 22:07, Thomas Koenig a écrit : > Hello world, > > this patch fixes a regression from the inline matmul patch by > not inlining a case that is not yet handled, namely vector > subscripts. > > Regression-tested. OK for trunk? > OK with... > > Index: frontend-passes.c >

[Patch ARM-AArch64/testsuite Neon intrinsics 13/13] Add vqshrun_n tests.

diff --git a/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqshrun_n.c b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqshrun_n.c new file mode 100644 index 000..ce1a3ff --- /dev/null +++ b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqshrun_n.c @@ -0,0 +1,133 @@ +#includ

[Patch ARM-AArch64/testsuite Neon intrinsics 09/13] Add vqshl tests.

diff --git a/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqshl.c b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqshl.c new file mode 100644 index 000..f5a3084 --- /dev/null +++ b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqshl.c @@ -0,0 +1,829 @@ +#include +#include

[Patch ARM-AArch64/testsuite Neon intrinsics 12/13] Add vqshrn_n tests.

diff --git a/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqshrn_n.c b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqshrn_n.c new file mode 100644 index 000..b3556f4 --- /dev/null +++ b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqshrn_n.c @@ -0,0 +1,177 @@ +#include

[Patch ARM-AArch64/testsuite Neon intrinsics 06/13] Add vqrshl tests.

diff --git a/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqrshl.c b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqrshl.c new file mode 100644 index 000..3f0cb37 --- /dev/null +++ b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqrshl.c @@ -0,0 +1,1090 @@ +#include +#inc

[Patch ARM-AArch64/testsuite Neon intrinsics 11/13] Add vqshlu_n tests.

diff --git a/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqshlu_n.c b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqshlu_n.c new file mode 100644 index 000..a6710ef --- /dev/null +++ b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqshlu_n.c @@ -0,0 +1,263 @@ +#include

[Patch ARM-AArch64/testsuite Neon intrinsics 08/13] Add vqrshrun_n tests.

diff --git a/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqrshrun_n.c b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqrshrun_n.c new file mode 100644 index 000..f5e431e --- /dev/null +++ b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqrshrun_n.c @@ -0,0 +1,189 @@ +#inc

[Patch ARM-AArch64/testsuite Neon intrinsics 10/13] Add vqshl_n tests.

diff --git a/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqshl_n.c b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqshl_n.c new file mode 100644 index 000..92a5e0d --- /dev/null +++ b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqshl_n.c @@ -0,0 +1,234 @@ +#include +#i

[Patch ARM-AArch64/testsuite Neon intrinsics 04/13] Add vqrdmulh_lane tests.

diff --git a/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqrdmulh_lane.c b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqrdmulh_lane.c new file mode 100644 index 000..2235e74 --- /dev/null +++ b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqrdmulh_lane.c @@ -0,0 +1,16

[Patch ARM-AArch64/testsuite Neon intrinsics 07/13] Add vqrshrn_n tests.

diff --git a/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqrshrn_n.c b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqrshrn_n.c new file mode 100644 index 000..5a20d98 --- /dev/null +++ b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqrshrn_n.c @@ -0,0 +1,174 @@ +#includ

[Patch ARM-AArch64/testsuite Neon intrinsics 05/13] Add vqrdmulh_n tests.

diff --git a/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqrdmulh_n.c b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqrdmulh_n.c new file mode 100644 index 000..fdc739c --- /dev/null +++ b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqrdmulh_n.c @@ -0,0 +1,155 @@ +#inc

[Patch ARM-AArch64/testsuite Neon intrinsics 02/13] Add vqmovun tests.

diff --git a/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqmovun.c b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqmovun.c new file mode 100644 index 000..d744765 --- /dev/null +++ b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqmovun.c @@ -0,0 +1,93 @@ +#include +#in

[Patch ARM-AArch64/testsuite Neon intrinsics 03/13] Add vqrdmulh tests.

diff --git a/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqrdmulh.c b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqrdmulh.c new file mode 100644 index 000..8875c53 --- /dev/null +++ b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqrdmulh.c @@ -0,0 +1,161 @@ +#include

[Patch ARM-AArch64/testsuite Neon intrinsics 01/13] Add vqmovn tests.

diff --git a/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqmovn.c b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqmovn.c new file mode 100644 index 000..b4d3198 --- /dev/null +++ b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqmovn.c @@ -0,0 +1,134 @@ +#include +#incl

[Patch ARM-AArch64/testsuite 00/13] Neon intrinsics executable tests

This patch series is a follow-up to the tests I already contributed, converted from my original testsuite. This series consists in 13 new files, which can be committed independently. Another series (hopefully final) will follow. Tested with qemu on arm*linux, aarch64-linux. I couldn't test on a

Re: [PATCH] Fix get_min_precision in __builtin_*_overflow expansion (PR target/66112)

On May 12, 2015 8:40:32 PM GMT+02:00, Jakub Jelinek wrote: >Hi! > >This patch fixes a thinko in get_min_precision. If we want UNSIGNED >number and arg_min is signed, but non-negative, calling >wi::min_precision >with SIGNED resulted in one bigger precision than necessary. If >get_min_precision i

Re: [PATCH][PING^2] Skip preprocessor directives in mklog

On 05/12/2015 09:19 AM, Yury Gribov wrote: On 04/30/2015 12:03 PM, Yury Gribov wrote: On 04/21/2015 02:26 PM, Yury Gribov wrote: Hi all, Contrib/mklog is currently faked by preprocessor directives inside functions to produce invalid ChangeLog. The attached patch fixes this. Tested with my lo

Re: [patch] Clean up detection of SJLJ exceptions in target libraries

On 05/12/2015 10:42 AM, Eric Botcazou wrote: Hi, 6 target libraries in the tree detect whether they are being compiled by a compiler configured for setjmp/longjmp exceptions: libada, libgcc, libgo, libjava, libobjc and libstdc++. They can be divided into 3 categories: 1) libada only checks th

Re: [PATCH, RFC]: Next stage1, refactoring: propagating rtx subclasses

On 05/11/2015 02:41 PM, Mikhail Maltsev wrote: On 09.05.2015 0:54, Jeff Law wrote: Both patches are approved. Please install onto the trunk. jeff Sorry for delay. When I started to work on this task, I wrote that I'll test the patches on couple of other platforms (not just x86). Probably I

[patch, fortran] Fix PR 66111

Hello world, this patch fixes a regression from the inline matmul patch by not inlining a case that is not yet handled, namely vector subscripts. Regression-tested. OK for trunk? Regards Thomas 2015-05-12 Thomas Koenig PR fortran/66111 * frontend-passes.c (has_dime

Re: [PATCH, ping 1] Move insns without introducing new temporaries in loop2_invariant

On 05/12/2015 03:32 AM, Thomas Preud'homme wrote: From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Thomas Preud'homme From: Jeff Law [mailto:l...@redhat.com] Sent: Tuesday, May 12, 2015 4:17 AM + + /* Check that all uses reached by the def in insn wou

Re: [PATCH, i386, PR target/66048] Fix mode switching ICE for functions returning bounds

On 05/12/2015 04:07 AM, Ilya Enkovich wrote: Hi, This patch moves up use insns for returned bounds and also marks bnd1 register as holding return value. This fixes problem in create_pre_exit for functions with no returned bounds initialization. Bootstrapped and regtested for x86_64-unknown-

Re: [PATCH] Add {u,}mulvhi4 patterns on i?86 (PR target/66112)

On Tue, May 12, 2015 at 8:43 PM, Jakub Jelinek wrote: > Hi! > > This patch improves expansion of __builtin_mul_overflow for HImode, both > signed and unsigned, on x86_64/i686. > > Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? > > 2015-05-12 Jakub Jelinek > > PR ta

Re: [PATCH][PING^2] Skip preprocessor directives in mklog

On 05/12/2015 10:14 AM, Yury Gribov wrote: On 05/12/2015 06:57 PM, Trevor Saunders wrote: On Tue, May 12, 2015 at 06:33:28PM +0300, Yury Gribov wrote: On 05/12/2015 06:23 PM, Diego Novillo wrote: The patch looks fine to me. I'm not really involved in GCC development anymore. I would suggest t

Re: [PATCH, ping 1] Move insns without introducing new temporaries in loop2_invariant

On 05/12/2015 02:55 AM, Thomas Preud'homme wrote: From: Jeff Law [mailto:l...@redhat.com] Sent: Tuesday, May 12, 2015 4:17 AM On 05/06/2015 03:47 AM, Thomas Preud'homme wrote: Ping? Something to consider as future work -- I'm pretty sure PRE sets up the same kind of problematical pattern with

Re: [PATCH] Add {u,}mulvhi4 patterns on i?86 (PR target/66112)

On Tue, May 12, 2015 at 7:43 PM, Jakub Jelinek wrote: > Hi! > > This patch improves expansion of __builtin_mul_overflow for HImode, both > signed and unsigned, on x86_64/i686. > > Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? > > 2015-05-12 Jakub Jelinek > > PR ta

[PATCH] Add {u,}mulvhi4 patterns on i?86 (PR target/66112)

Hi! This patch improves expansion of __builtin_mul_overflow for HImode, both signed and unsigned, on x86_64/i686. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2015-05-12 Jakub Jelinek PR target/66112 * config/i386/i386.md (mulv4, umulv4, *umulv4):

[PATCH] Fix get_min_precision in __builtin_*_overflow expansion (PR target/66112)

Hi! This patch fixes a thinko in get_min_precision. If we want UNSIGNED number and arg_min is signed, but non-negative, calling wi::min_precision with SIGNED resulted in one bigger precision than necessary. If get_min_precision is originally called with SIGNED and sign is then changed into UNSIG

Re: False ODR violation positives on anonymous namespace types

Hi, here is patch I LTO-bootstrapped and regtested on x86_64-linux and I am going to commit shortly. Thank you for useful discussion - I did not really realize before that types may or may not have linkage. This caused surprises indeed ;) Honza * ipa-devirt.c (type_with_linkage_p): New f

[PATCH, committed] Unbreak the JIT after conversion of md_asm_adjust to use vec<>

The patch: https://gcc.gnu.org/ml/gcc-patches/2015-05/msg00600.html broke the build of the jit: In file included from ../../src/gcc/jit/jit-builtins.c:23:0: ../../src/gcc/target.h:188:60: error: expected ‘;’ at end of member declaration #define DEFHOOK(NAME, DOC, TYPE, PARAMS, INIT) TYPE (* NA

Re: [patch 1/28] top-level: Use automake-1.11.6

On Tue, 12 May 2015, Michael Haubenwallner wrote: > It turns out that besides the top-level ones there are more > copied-from-automake > files in various library dirs - where some of them seem quite outdated > actually: > > libffi/mdate-sh > libjava/classpath/INSTALL > libjava/libltdl/con

Re: [PATCH 4/n] OpenMP 4.0 offloading infrastructure: lto-wrapper

On 05/12/2015 06:27 PM, Thomas Schwinge wrote: Patch variant 1: @@ -4266,7 +4266,7 @@ process_command (unsigned int decoded_op } gcc_assert (!IS_ABSOLUTE_PATH (tooldir_base_prefix)); - tooldir_prefix2 = concat (tooldir_base_prefix, spec_host_machine, + tooldir_prefix2 = concat (too

Re: [PATCH] Support vectorizing SLP with mixed plus/minus

On Tue, May 12, 2015 at 4:48 AM, Richard Biener wrote: > > The following makes it possible to vectorize complex multiplication > without unrolling by allowing mixed operations in a SLP node > (plus/minus for now). We vectorize that by computing both plus > and minus and then merge the result. > >

RE: [RFC]: Remove Mem/address type assumption in combiner

On Tue, 2015-05-12 at 05:21 +, Kumar, Venkataramanan wrote: > Hi Steve, > > Yes this is expected. As Segher pointed out, we need to change .md patterns > in > target to be based on shifts instead of mults. > > Regards, > Venkat. Here is my patch to change this. I tested it with the mips-

[patch] Clean up detection of SJLJ exceptions in target libraries

Hi, 6 target libraries in the tree detect whether they are being compiled by a compiler configured for setjmp/longjmp exceptions: libada, libgcc, libgo, libjava, libobjc and libstdc++. They can be divided into 3 categories: 1) libada only checks the preprocessor macro __USING_SJLJ_EXCEPTIONS__

[PATCH, ARM] Make tune params tables more self-documenting

This patch modifies the way that the tune_params data structures are constructed so that: 1) The initializers are (mostly) self documenting. 2) The compiler does a better job of detecting merge failures. Mostly this resolves around using C++ enums, such that we get type clashes if elements do not

Re: [patch 1/28] top-level: Use automake-1.11.6

On 05/12/2015 10:45 AM, Michael Haubenwallner wrote: > On 05/11/2015 07:55 PM, Joseph Myers wrote: >> On Sat, 9 May 2015, Bernd Edlinger wrote: >> >>> But maybe you would like it better if we update, for instance, to: >>> automake-1.14 _and_ autoconf-2.69 ? >> >> Updating to current automake and

[patch, nios2] use "trap" instruction instead of "break"

In connection with this discussion https://gcc.gnu.org/ml/gcc/2015-02/msg00163.html about bad behavior of -fisolate-erroneous-paths-dereference on nios2 bare-metal targets, I realized that we also had a serious problem on nios2-linux-gnu targets as well. The "trap" and "ctrapsi4" insns were

Re: My patch for GCC 5 directory names

On Tue, May 12, 2015 at 9:32 AM, Richard Biener wrote: > On May 12, 2015 6:11:45 PM GMT+02:00, "H.J. Lu" wrote: >>On Tue, May 12, 2015 at 9:09 AM, Richard Biener >> wrote: >>> On May 12, 2015 5:58:07 PM GMT+02:00, "H.J. Lu" >>wrote: On Tue, May 12, 2015 at 8:28 AM, Michael Matz wrote: >

Re: My patch for GCC 5 directory names

On May 12, 2015 6:11:45 PM GMT+02:00, "H.J. Lu" wrote: >On Tue, May 12, 2015 at 9:09 AM, Richard Biener > wrote: >> On May 12, 2015 5:58:07 PM GMT+02:00, "H.J. Lu" >wrote: >>>On Tue, May 12, 2015 at 8:28 AM, Michael Matz wrote: Hi, On Tue, 12 May 2015, H.J. Lu wrote: > >>

Re: [PATCH 4/n] OpenMP 4.0 offloading infrastructure: lto-wrapper

Hi Bernd! One question to you, as the one who has originally written this code (): On Thu, 2 Oct 2014 19:14:57 +0400, Ilya Verbin wrote: > With this patch lto-wrapper performs invocation of mkoffload tool for each > offload target. [...]

Re: False ODR violation positives on anonymous namespace types

> Looks good. Thank you! Testing looks good, too, no false positives on Firefox. I suppose the next challenge is to make type_with_linkage_p to return false on types that was not built by C++ FE as discussed in https://gcc.gnu.org/ml/gcc-patches/2015-05/msg01023.html What would be the preferred so

Re: [PATCH][PING^2] Skip preprocessor directives in mklog

On 05/12/2015 06:57 PM, Trevor Saunders wrote: On Tue, May 12, 2015 at 06:33:28PM +0300, Yury Gribov wrote: On 05/12/2015 06:23 PM, Diego Novillo wrote: The patch looks fine to me. I'm not really involved in GCC development anymore. I would suggest that this script should be maintained by whoe

Re: [PATCH][PING^2] Skip preprocessor directives in mklog

On Tue, May 12, 2015 at 11:50 AM, Tom de Vries wrote: > I'm not a good choice to be the maintainer of a perl script. I'm all kinds of sorry about the original choice of scripting language. I'd just spend a couple of hours re-writing it in python.

Re: My patch for GCC 5 directory names

On Tue, May 12, 2015 at 9:09 AM, Richard Biener wrote: > On May 12, 2015 5:58:07 PM GMT+02:00, "H.J. Lu" wrote: >>On Tue, May 12, 2015 at 8:28 AM, Michael Matz wrote: >>> Hi, >>> >>> On Tue, 12 May 2015, H.J. Lu wrote: >>> >> So we have >> >> experimental >> release >> p

Re: My patch for GCC 5 directory names

On May 12, 2015 5:58:07 PM GMT+02:00, "H.J. Lu" wrote: >On Tue, May 12, 2015 at 8:28 AM, Michael Matz wrote: >> Hi, >> >> On Tue, 12 May 2015, H.J. Lu wrote: >> >>> >> So we have >>> >> >>> >> experimental >>> >> release >>> >> post-release >>> >> >>> >> Why not just rename prerelease to post-rel

[patch, nios2] add IJMP_REGS reg class for indirect jumps

While testing something else, I noticed a couple compilation errors like: /tmp/ccCxOOoN.s: Assembler messages: /tmp/ccCxOOoN.s:164: Error: r31 cannot be used with jmp; use ret instead The assembler is correct in rejecting this code, since this is a documented restriction of the JMP instruction.

Re: False ODR violation positives on anonymous namespace types

Looks good. Jason

Re: [PATCH][PING^2] Skip preprocessor directives in mklog

On Tue, May 12, 2015 at 06:33:28PM +0300, Yury Gribov wrote: > On 05/12/2015 06:23 PM, Diego Novillo wrote: > >The patch looks fine to me. > > > >I'm not really involved in GCC development anymore. I would suggest > >that this script should be maintained by whoever's been hacking on it > >the most.

Re: My patch for GCC 5 directory names

On Tue, May 12, 2015 at 8:28 AM, Michael Matz wrote: > Hi, > > On Tue, 12 May 2015, H.J. Lu wrote: > >> >> So we have >> >> >> >> experimental >> >> release >> >> post-release >> >> >> >> Why not just rename prerelease to post-release? That is a one-line >> >> change. >> > >> > Why print anything

Re: [PATCH][PING^2] Skip preprocessor directives in mklog

On 12-05-15 17:33, Yury Gribov wrote: On 05/12/2015 06:23 PM, Diego Novillo wrote: The patch looks fine to me. I'm not really involved in GCC development anymore. I would suggest that this script should be maintained by whoever's been hacking on it the most. It's a simple script, so it shouldn'

Re: [PATCH, alpha]: Switch alpha to TARGET_SUPPORTS_WIDE_INT

On 05/12/2015 08:06 AM, Uros Bizjak wrote: > 2015-05-12 Uros Bizjak > > * config/alpha/alpha.h (TARGET_SUPPORTS_WIDE_INT): New define. > * config/alpha/alpha.c (alpha_rtx_costs): Handle CONST_WIDE_INT. > (alpha_extract_integer): Ditto. > (alpha_legitimate_constant_p): Ditto. >

Re: [PATCH][PING^2] Skip preprocessor directives in mklog

On 05/12/2015 06:23 PM, Diego Novillo wrote: The patch looks fine to me. I'm not really involved in GCC development anymore. I would suggest that this script should be maintained by whoever's been hacking on it the most. It's a simple script, so it shouldn't be hard to find a new maintainer for

Re: [PATCH, PR target/65103, 3/3] Change rtx cost for i386 address constants

Hello! >> 2015-03-10 Ilya Enkovich >> >> PR target/65103 >> * config/i386/i386.c (ix86_rtx_costs): We want to propagate >> link time constants into adress expressions and therefore set >> their cost to 0. >> >> gcc/testsuite/ >> >> 2015-03-10 Ilya Enkovich >>

Re: My patch for GCC 5 directory names

Hi, On Tue, 12 May 2015, H.J. Lu wrote: > >> So we have > >> > >> experimental > >> release > >> post-release > >> > >> Why not just rename prerelease to post-release? That is a one-line > >> change. > > > > Why print anything at all? 5.1.1 is after 5.1.0 in obvious ways. > > > > How can you te

Re: [patch 1/28] top-level: Use automake-1.11.6

On Tue, 12 May 2015, Michael Haubenwallner wrote: > On 05/11/2015 07:55 PM, Joseph Myers wrote: > > On Sat, 9 May 2015, Bernd Edlinger wrote: > > > >> But maybe you would like it better if we update, for instance, to: > >> automake-1.14 _and_ autoconf-2.69 ? > > > > Updating to current automak

Re: [PATCH][PING^2] Skip preprocessor directives in mklog

The patch looks fine to me. I'm not really involved in GCC development anymore. I would suggest that this script should be maintained by whoever's been hacking on it the most. It's a simple script, so it shouldn't be hard to find a new maintainer for it. Diegop. On Tue, May 12, 2015 at 11:19 AM

[PATCH][PING^2] Skip preprocessor directives in mklog

On 04/30/2015 12:03 PM, Yury Gribov wrote: On 04/21/2015 02:26 PM, Yury Gribov wrote: Hi all, Contrib/mklog is currently faked by preprocessor directives inside functions to produce invalid ChangeLog. The attached patch fixes this. Tested with my local mklog testsuite and http://paste.debian.

Re: [PATCH, 5/5] check_GNU_style.sh: Fix tab size in 80 characters check

On 05/12/2015 01:29 AM, Tom de Vries wrote: Hi, this patch fixes a problem in the 80 characters length check. Currently tab width is not properly calculated. The patch uses expand to interpret tabs properly. OK for trunk? Thanks, - Tom 0005-check_GNU_style.sh-Fix-tab-size-in-80-characters-

[PATCH, alpha]: Switch alpha to TARGET_SUPPORTS_WIDE_INT

Hello! The patch also further removes dead HWI < 64 code in a couple of places. 2015-05-12 Uros Bizjak * config/alpha/alpha.h (TARGET_SUPPORTS_WIDE_INT): New define. * config/alpha/alpha.c (alpha_rtx_costs): Handle CONST_WIDE_INT. (alpha_extract_integer): Ditto. (alpha_legitim

Re: [PATCH, 4/5] check_GNU_style.sh: Put stdin into temp file

On 05/12/2015 01:29 AM, Tom de Vries wrote: Hi, this patch makes sure we dump stdin into a file before we handle it in the rest of the script. This makes the stdin handling more like the handling of normal files. OK for trunk? Thanks, - Tom 0004-check_GNU_style.sh-Put-stdin-into-temp-file.pa

Re: [PATCH, 3/5] check_GNU_style.sh: Read either from stdin, or from files

On 05/12/2015 01:29 AM, Tom de Vries wrote: Hi, this patch makes sure we either handle: ... $ cat temp.patch | ./check_GNU_style.sh - ... or ... $ ./check_GNU_style.sh temp.patch ... but not: ... $ cat temp.patch | ./check_GNU_style.sh - temp2.patch ... OK for trunk? Thanks, - Tom 0003-check

[Ada] Implement new pragma/aspect Volatile_Full_Access

A new pragma (and equivalent aspect) Volatile_Full_Access is implemented. This is similar to Volatile except that there is a guarantee that every read and write access to an object with this aspect will always use a single instruction which reads or writes all the bits of the object. Note that this

Re: [PATCH, 2/5] check_GNU_style.sh: Check file presence

On 05/12/2015 01:28 AM, Tom de Vries wrote: Hi, this patch checks that the files specified on the check_GNU_style.sh command line are present. OK for trunk? Thanks, - Tom 0002-check_GNU_style.sh-Check-file-presence.patch [PATCH 2/5] check_GNU_style.sh: Check file presence 2015-05-11 Tom

[Ada] Legality checks on a formal derived type derived from previous formal.

This patch adds a missing legality check on the instantiation of a formal derived type whose parent type is a previous formal of the same generic unit, that is not a derived type. Compiling generic_testm.adb must yield: generic_testm.adb:7:26: expect type derived from "Integer" in instantiation

[Ada] Incomplete types and null procedures.

An untagged incomeplete type is legal in the profile of a null procedure. This patch remotes a spurious error on such, because the legality check for the use of an incomplete type was also applied to the subprogram body generated during expansion of a null procedure declaration. The following must

Re: [PATCH, 1/5] check_GNU_style.sh: Don't use filename prefix for one patch

On 05/12/2015 01:27 AM, Tom de Vries wrote: Hi, this patch makes sure that we don't print the patch filename in front of the error messages, if we're checking just one file. Making the lines shorter makes things easier to read. And since we're only processing one file, there's no information los

Re: [PATCH] Fix PR ipa/65557

On Tue, May 12, 2015 at 4:54 PM, Martin Liška wrote: > On 04/30/2015 11:11 PM, Uros Bizjak wrote: >> Hello! >> >>> 2015-03-29 Martin Liska >>> >>>PR ipa/65557 >>>* ipa-icf.c (sem_function::equals_wpa): Check if IPA CP >>>has already filled up function summary. >>>(sem_item_optim

Re: [PATCH] Fix PR ipa/65557

On 04/30/2015 11:11 PM, Uros Bizjak wrote: > Hello! > >> 2015-03-29 Martin Liska >> >>PR ipa/65557 >>* ipa-icf.c (sem_function::equals_wpa): Check if IPA CP >>has already filled up function summary. >>(sem_item_optimizer::update_hash_by_addr_refs): Likewise. >> >> gcc/testsuite/

Re: False ODR violation positives on anonymous namespace types

> /* Record, union and enumeration type have linkage that allows use >to check type_in_anonymous_namespace_p. Compound types >can be always compared structurally. >To save some work we compare builtin types properties of its main >variant. A special case are i

Re: My patch for GCC 5 directory names

On Tue, May 12, 2015 at 7:01 AM, Richard Biener wrote: > On Tue, 12 May 2015, H.J. Lu wrote: > >> On Tue, May 12, 2015 at 6:54 AM, Richard Biener wrote: >> > On Tue, 12 May 2015, H.J. Lu wrote: >> > >> >> On Tue, May 12, 2015 at 6:42 AM, Richard Biener wrote: >> >> > >> >> > I promised to send o

[PATCH, libgcc]: Commited: Add space between string literal and macro name.

... to avoid "C++11 requires a space between string literal and macro" warning. Plus fix lines over 80 chars. 2015-05-12 Uros Bizjak * libgcov-util.c: Add space between string literal and macro name. Tested on x86_64-linux-gnu and committed to mainline SVN. Uros. Index: ChangeLog ==

Re: [PATCH, PR target/65103, 3/3] Change rtx cost for i386 address constants

Ping 2015-03-10 19:14 GMT+03:00 Ilya Enkovich : > On 10 Mar 18:11, Ilya Enkovich wrote: >> Hi, >> >> This patch changes rtx cost for address constants to 0 similar to what we >> have in address cost hook beacuse we expect them to be propagated into >> address expression anyway. Bootstrapped and

Re: False ODR violation positives on anonymous namespace types

> On 05/11/2015 04:39 PM, Jan Hubicka wrote: > >What happens in LTO is that lto-symtab decide to merge the two declarations > >of > >foo. At this time it has chance to output the warning. For that it needs to > >be able to work out that these declarations are having different types, but > >becau

Re: My patch for GCC 5 directory names

On Tue, 12 May 2015, H.J. Lu wrote: > On Tue, May 12, 2015 at 6:54 AM, Richard Biener wrote: > > On Tue, 12 May 2015, H.J. Lu wrote: > > > >> On Tue, May 12, 2015 at 6:42 AM, Richard Biener wrote: > >> > > >> > I promised to send out my pat^Whack. Before building I introduce > >> > gcc/FULL-VER

Re: [PATCH 0/6] Getting rid of some zero_ext* patterns

On Sun, May 10, 2015 at 09:13:49AM -0700, Segher Boessenkool wrote: > This series teaches combine to behave a bit better, and then takes > advantage of that in the rs6000 backend. > > I'll test the combine patches on x86_64-linux before committing, > and will wait a bit for comments anyway. Done

Re: My patch for GCC 5 directory names

On Tue, May 12, 2015 at 6:54 AM, Richard Biener wrote: > On Tue, 12 May 2015, H.J. Lu wrote: > >> On Tue, May 12, 2015 at 6:42 AM, Richard Biener wrote: >> > >> > I promised to send out my pat^Whack. Before building I introduce >> > gcc/FULL-VER as copy of gcc/BASE-VER and adjust gcc/BASE-VER to

Re: My patch for GCC 5 directory names

On Tue, 12 May 2015, H.J. Lu wrote: > On Tue, May 12, 2015 at 6:42 AM, Richard Biener wrote: > > > > I promised to send out my pat^Whack. Before building I introduce > > gcc/FULL-VER as copy of gcc/BASE-VER and adjust gcc/BASE-VER to > > just the major number. Then I only need the following sma

Re: My patch for GCC 5 directory names

On Tue, May 12, 2015 at 6:42 AM, Richard Biener wrote: > > I promised to send out my pat^Whack. Before building I introduce > gcc/FULL-VER as copy of gcc/BASE-VER and adjust gcc/BASE-VER to > just the major number. Then I only need the following small > patch (where I don't speak enough tcl for

Re: False ODR violation positives on anonymous namespace types

On 05/11/2015 04:39 PM, Jan Hubicka wrote: What happens in LTO is that lto-symtab decide to merge the two declarations of foo. At this time it has chance to output the warning. For that it needs to be able to work out that these declarations are having different types, but because LTO merge can

[PATCH] Fix PR66110

The following patch fixes PR66110 - there is no reason why two alias-sets conflict just because one or the other has a child with alias-set zero. We need to paper over strict-aliasing bugs in dfp.c (re-using struct real_value unsigned long sig[] as decimal128) by adding -Wno-strict-aliasing, othe

My patch for GCC 5 directory names

I promised to send out my pat^Whack. Before building I introduce gcc/FULL-VER as copy of gcc/BASE-VER and adjust gcc/BASE-VER to just the major number. Then I only need the following small patch (where I don't speak enough tcl for fixing libjava.exp "properly"). Without the FULL-VER trick the p

[PATCH] Fix PR66101

This fixes PR66101. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2015-05-12 Richard Biener PR tree-optimization/66101 * tree-ssa-dce.c (remove_dead_stmt): Properly mark loops for fixup if we turn a loop exit edge to a fallthru edge.

  1   2   >