Re: Move "X +- C1 CMP C2 to X CMP C2 -+ C1" to match.pd

Here is the current patch (passed regtest), after moving defer/undefer from forwprop to fold_stmt_1. I am not sure if checking no_warning at the end of fold_stmt_1 is safe, or if I should save its value at the beginning of the function, in case some of the transformations clear it. On Tue, 26

Re: [RFA] add DW_LANG_Rust

On 04/26/2016 04:05 PM, Tom Tromey wrote: Hi. I've been working on Rust support for gdb, and I needed this patch as a prerequisite. DW_LANG_Rust is only defined in DWARF 5, which isn't released yet. However, the value is mentioned in the DWARF issue requesting the addition; and similar

Re: [PATCH] boehm-gc: add supprt for aarch64 ILP32

On 03/15/2016 08:46 AM, Andreas Schwab wrote: * include/private/gcconfig.h [AARCH64] (ALIGNMENT, CPP_WORDSZ): Define for __ILP32__. --- boehm-gc/include/private/gcconfig.h | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) Similarly, this should be going to upstream

Re: [PATCH] libffi: define FFI_SIZEOF_JAVA_RAW for aarch64 ILP32

On 03/15/2016 08:18 AM, Andreas Schwab wrote: Like x32, aarch64 ILP32 needs to define FFI_SIZEOF_JAVA_RAW. This fixes the java interpreter. Andreas. * src/aarch64/ffitarget.h (FFI_SIZEOF_JAVA_RAW) [__ILP32__]: Define. Doesn't libffi come from an upstream project? If so,

Re: [PATCH v2] [libatomic] Add RTEMS support

On 04/24/2016 11:40 PM, Sebastian Huber wrote: Ok, what about the GCC trunk? On 20/04/16 14:35, Sebastian Huber wrote: Hello, I know that I am pretty late, but is there a chance to get this into the GCC 6.1 release? On 19/04/16 14:56, Sebastian Huber wrote: v2: Do not use architecture

Re: IRA costs tweaks, PR 56069

On 04/25/2016 03:11 AM, Bernd Schmidt wrote: On 03/02/2016 10:53 PM, Vladimir Makarov wrote: The patch is ok for me. But I'd wait for GCC7. People are sensitive to their code performance degradation. Even in most cases the patch improves performance, in some case it can worsen code and

Re: IRA costs tweaks, PR 56069

On 03/02/2016 02:53 PM, Vladimir Makarov wrote: The overall effect of this patch seems to be fairly minimal in practice, which is slightly disappointing. On the whole, it seems to be a very slight net positive, with few instances where we generate additional instructions. I am just

Re: [PATCH 07/18] loop-iv.c: make cond_list a vec

On Mon, Apr 25, 2016 at 04:25:27PM +0200, Bernd Schmidt wrote: > On 04/25/2016 04:21 PM, Bernd Schmidt wrote: > >On 04/25/2016 03:30 PM, Trevor Saunders wrote: > >>On Mon, Apr 25, 2016 at 02:28:51PM +0200, Bernd Schmidt wrote: > >>>On 04/20/2016 08:22 AM, tbsaunde+...@tbsaunde.org wrote: >

Re: [RFC][PR68217] Improve value range for signed & sign-bit-CST

On 27/04/16 00:14, Richard Biener wrote: On Fri, Apr 15, 2016 at 12:44 PM, kugan wrote: As pointed out by Richard, for signed & sign-bit-CST value range should be [-INF, 0] range, not a [-INF, INF] range as happens now. This patch fixes this. I

Re: [PATCH] Fix a recent warning in reorg.c

On Tue, Apr 26, 2016 at 09:03:03AM -0600, Jeff Law wrote: > On 04/26/2016 07:08 AM, Jakub Jelinek wrote: > >Hi! > > > >I've noticed a warning during bootstrap: > >../../gcc/reorg.c: In function ‘void try_merge_delay_insns(rtx_insn*, > >rtx_insn*)’: > >../../gcc/reorg.c:1431:12: warning: name

Re: Fix some i386 testcases for -frename-registers

On Tue, Apr 26, 2016 at 3:11 PM, Bernd Schmidt wrote: > On 04/26/2016 09:39 PM, H.J. Lu wrote: >> >> make check-gcc RUNTESTFLAGS="--target_board='unix{-mx32}' >> i386.exp=avx512vl-vmovdqa64-1.c" > > > Unfortunately, that doesn't work: > > /usr/include/gnu/stubs.h:13:28: fatal

[PATCH] c++/66561 - __builtin_LINE at al. should yield constant expressions

PR c++/66639 asked to declare __func__ , __FUNCTION__ and __PRETTY_FUNCTION__ as constexpr​. With the request fulfilled sometime in the 6.0 time frame (possibly as a result of fixing c++/70353), the attached patch implements the corresponding change suggested in c++/66561 - __builtin_LINE at al.

Re: Allow embedded timestamps by C/C++ macros to be set externally (3)

On 04/26/2016 11:28 PM, Dhole wrote: I've fixed all the spaces issues. I've also changed the "unsigned long long" to "time_t" as you suggested. Also, to improve reliability I'm now using strtoll rather than strtoull, so that negative values can be detected in SOURCE_DATE_EPOCH, which are

[PATCH] Fix up inchash::add_expr to match more closely operand_equal_p (PR sanitizer/70683, take 2)

On Tue, Apr 26, 2016 at 03:02:38PM +0200, Jakub Jelinek wrote: > I've been using the attached hack; without this patch during x86_64-linux > and i686-linux yes,extra,rtl checking bootstraps there were 66931 > notes (surprisingly only from the ivopts and gimple-ssa-strength-reduction > hash tables,

[PATCH] operand_equal_p checking (PR sanitizer/70683)

On Tue, Apr 26, 2016 at 03:02:38PM +0200, Jakub Jelinek wrote: > The debugging hack is too ugly and slows down the compiler (by artificially > increasing number of collisions), so it is not appropriate, but perhaps we > can add some internal only use OEP_* flag, pass it to the recursive calls > of

Re: Fix some i386 testcases for -frename-registers

On 04/26/2016 09:39 PM, H.J. Lu wrote: make check-gcc RUNTESTFLAGS="--target_board='unix{-mx32}' i386.exp=avx512vl-vmovdqa64-1.c" Unfortunately, that doesn't work: /usr/include/gnu/stubs.h:13:28: fatal error: gnu/stubs-x32.h: No such file or directory compilation terminated. Trying to

[RFA] add DW_LANG_Rust

Hi. I've been working on Rust support for gdb, and I needed this patch as a prerequisite. DW_LANG_Rust is only defined in DWARF 5, which isn't released yet. However, the value is mentioned in the DWARF issue requesting the addition; and similar DWARF-5-only constants have already been added to

Re: RFA: Prevent an ICE when redeclaring a static function as weak

On 02/17/2016 07:24 AM, Nick Clifton wrote: Hi Guys, Redefining a previously defined static function as both public and weak triggers an ICE in ipa-visibility.c: internal compiler error: in function_and_variable_visibility, at ipa-visibility.c:518 This bug has been discussed and patch

Re: [PATCH][rtlanal.c] Convert conditional compilation on WORD_REGISTER_OPERATIONS

On 12/15/2015 10:25 AM, Kyrill Tkachov wrote: Hi all, This converts the preprocessor check for WORD_REGISTER_OPERATIONS into a runtime one in rtlanal.c. Since this one was in combination with an "#if defined" and used to guard an if-statement I'd appreciate it if someone gave it a double-check

Re: Allow embedded timestamps by C/C++ macros to be set externally (3)

Hi Bernd, On 16-04-25 12:15:50, Bernd Schmidt wrote: > On 04/18/2016 02:26 PM, Dhole wrote: > >A few months ago I submited a patch to allow the embedded timestamps by > >C/C++ macros to be set externally [2], which was already an improvement > >over [1]. I was told to wait until the GCC 7 stage

Re: C, C++: New warning for memset without multiply by elt size

On 04/25/2016 03:07 AM, Bernd Schmidt wrote: On 04/22/2016 03:57 PM, Jason Merrill wrote: This looks good, but can we move the code into c-common rather than duplicate it? That would be this patch. Also passes testing on x86_64-linux. Hi Bernd, I like the idea behind the patch! So much so

Go patch committed: Add Enclosed_var_expression

This patch by Chris Manghane adds Enclosed_var_expression to the Go frontend. This represents a reference from a function literal to a variable defined in the enclosing function. This simplifies the existing code, and provides a base for future work. Bootstrapped and ran Go testsuite on

Re: Combine simplify_set WORD_REGISTER_OPERATIONS

On 01/31/2016 03:16 PM, Alan Modra wrote: The comment says this test is supposed to prevent "a narrower operation than requested", but it actually only allows a larger subreg, not one the same size. Fix that. Bootstrapped and regression tested powerpc64-linux. OK for stage1? Note that this

Re: [Patch, testsuite] Require int32plus and scheduling support for some tests

On 04/04/2016 06:00 AM, Senthil Kumar Selvaraj wrote: 2016-04-04 Senthil Kumar Selvaraj * gcc.c-torture/compile/pr69102.c: Require scheduling support. * gcc.c-torture/compile/pr37669.c: Require >=32 bit integers. *

Re: [PATCH IRA] save a call to init_reload

On 04/11/2016 08:10 PM, Vladimir Makarov wrote: On 04/10/2016 11:24 PM, Zhouyi Zhou wrote: save a function call to init_reload when using lra, also remove the a type error in reload1.c because init_reload is called only once when compile process, the performance reduction may not be

Re: Allow redefinition of libcilkrts debug macros

On 04/26/2016 08:04 AM, Rainer Orth wrote: When working on a couple of Cilk Plus issues lately (PRs target/60290, target/68945), I noticed that you have to modify the libcilkplus sources to enable various debugging output. This seems silly, and the following patch allows defining them from the

Re: [RFC] introduce --param max-lto-partition for having an upper bound on partition size

On 26 April 2016 at 16:31, Richard Biener wrote: > On Mon, 25 Apr 2016, Prathamesh Kulkarni wrote: > >> On 6 April 2016 at 14:54, Richard Biener wrote: >> > On Wed, 6 Apr 2016, Richard Biener wrote: >> > >> >> On Wed, 6 Apr 2016, Prathamesh Kulkarni wrote:

Re: Patches to fix GCC’s C++ exception handling on NetBSD/VAX

On 03/26/2016 08:38 AM, Jake Hamby wrote: Unfortunately, my previous patch that included a change to gcc/config/vax/vax.h that increased FIRST_PSEUDO_REGISTER from 16 to 17 breaks the C++ exception handling that I’d worked so hard to get right with the rest of the patch. I believe I need to

Re: Patches to fix optimizer bug & C++ exceptions for GCC VAX backend

On 04/01/2016 01:06 PM, Jake Hamby wrote: My theory is that it has to do with liveness detection and a write into the stack frame being incorrectly seen as updating a local variable, but that could be completely wrong. I was hoping that by cc'ing gcc-patches that somebody more familiar with why

Re: Patches to fix optimizer bug & C++ exceptions for GCC VAX backend

On 03/31/2016 02:29 PM, Jake Hamby wrote: The failures all seem to be related to trying to read a value from an array of 64-bit values and loading it into a 32-bit register. It seems like there should be a special insn defined for this sort of array access, since VAX has mova* and pusha*

Re: Move "X +- C1 CMP C2 to X CMP C2 -+ C1" to match.pd

On Tue, 26 Apr 2016, Richard Biener wrote: On Sun, Apr 24, 2016 at 7:14 PM, Marc Glisse wrote: Hello, trying to move a first pattern from fold_comparison. I first tried without single_use. It brought the number of 'free' in g++.dg/tree-ssa/pr61034.C down to 11, changed

Re: Patches to fix GCC’s C++ exception handling on NetBSD/VAX

On 03/26/2016 05:56 AM, Jake Hamby wrote: On Mar 23, 2016, at 05:56, Christos Zoulas wrote: In article , Jake Hamby wrote: Hi, Thanks a lot for your patch. I applied it to our gcc-5 in the tree.

Re: [RFC] Update gmp/mpfr/mpc minimum versions

On Tue, 26 Apr 2016, Bernd Edlinger wrote: > Hi, > > as we all know, it's high time now to adjust the minimum supported > gmp/mpfr/mpc versions for gcc-7. I think updating the minimum versions (when using previously built libraries, not in-tree) is only appropriate when it allows some cleanup

Re: [PATCH] PR target/70155: Use SSE for TImode load/store

On Tue, Apr 26, 2016 at 11:42 AM, H.J. Lu wrote: > On Tue, Apr 26, 2016 at 9:33 AM, H.J. Lu wrote: >> On Tue, Apr 26, 2016 at 9:27 AM, Ilya Enkovich >> wrote: >>> 2016-04-26 19:20 GMT+03:00 Ilya Enkovich

Re: Fix some i386 testcases for -frename-registers

On Tue, Apr 26, 2016 at 12:32 PM, Bernd Schmidt wrote: > On 04/26/2016 08:35 PM, H.J. Lu wrote: >> >> This >> >> /* { dg-final { scan-assembler-times "vmovdqa64\[ >> \\t\]+\[^\{\n\]*%ymm\[0-9\]+\[^\nxy\]*\\(.{5}(?:\n|\[ \\t\]+#)" 1 { >> target nonpic } } } */ >> >> fails on

Re: Fix some i386 testcases for -frename-registers

On 04/26/2016 08:35 PM, H.J. Lu wrote: This /* { dg-final { scan-assembler-times "vmovdqa64\[ \\t\]+\[^\{\n\]*%ymm\[0-9\]+\[^\nxy\]*\\(.{5}(?:\n|\[ \\t\]+#)" 1 { target nonpic } } } */ fails on x32 since x32 with 32-bit pointers has (%r10d) instead of (%r10). .{5} doesn't match. What is

Re: [RFC] Update gmp/mpfr/mpc minimum versions

On Tue, 26 Apr 2016, Bernd Edlinger wrote: For instance PR libstdc++/69881: gmp.h did this: #define __need_size_t /* tell gcc stddef.h we only want size_t */ #include /* for size_t */ I've persuaded Jonathan to work around that in libstdc++. Of course the in-tree build does work with

Re: constexpr function caching

OK. Jason

Re: [RFC] Update gmp/mpfr/mpc minimum versions

On 26.04.2016 20:39, Marc Glisse wrote: > On Tue, 26 Apr 2016, Bernd Edlinger wrote: > >> as we all know, it's high time now to adjust the minimum supported >> gmp/mpfr/mpc versions for gcc-7. >> >> So this attached patch is now targeting at only supporting the currently >> latest relased

Re: [RFC] Update gmp/mpfr/mpc minimum versions

On April 26, 2016 8:39:28 PM GMT+02:00, Marc Glisse wrote: >On Tue, 26 Apr 2016, Bernd Edlinger wrote: > >> as we all know, it's high time now to adjust the minimum supported >> gmp/mpfr/mpc versions for gcc-7. >> >> So this attached patch is now targeting at only

Re: [PATCH] PR target/70155: Use SSE for TImode load/store

On Tue, Apr 26, 2016 at 9:33 AM, H.J. Lu wrote: > On Tue, Apr 26, 2016 at 9:27 AM, Ilya Enkovich wrote: >> 2016-04-26 19:20 GMT+03:00 Ilya Enkovich : >>> 2016-04-26 19:12 GMT+03:00 H.J. Lu : On Tue,

Re: [RFC] Update gmp/mpfr/mpc minimum versions

On 26.04.2016 20:29, Jakub Jelinek wrote: > On Tue, Apr 26, 2016 at 06:23:17PM +, Bernd Edlinger wrote: >> as we all know, it's high time now to adjust the minimum supported >> gmp/mpfr/mpc versions for gcc-7. > > I'm not saying we shouldn't bump the minimum supported versions, but bumping >

Re: [middle-end][PATCH] Update alignment_for_piecewise_move

On Tue, Apr 26, 2016 at 11:31 AM, Bernd Schmidt wrote: > On 04/26/2016 08:21 PM, Richard Sandiford wrote: >> >> "H.J. Lu" writes: >>> >>> I am working a patch to enable SSE, AVX and AVX512 for memcpy/memset >>> optimization. x86 backend defines

Re: [RFC] Update gmp/mpfr/mpc minimum versions

On Tue, 26 Apr 2016, Bernd Edlinger wrote: as we all know, it's high time now to adjust the minimum supported gmp/mpfr/mpc versions for gcc-7. So this attached patch is now targeting at only supporting the currently latest relased gmp-6.1.0, mpfr-3.1.4 and mpc-1.0.3, instead of trying to

Re: Fix some i386 testcases for -frename-registers

On Tue, Apr 26, 2016 at 5:26 AM, Bernd Schmidt wrote: > On 01/29/2016 01:19 PM, Uros Bizjak wrote: >>> >>> * gcc.target/i386/avx512bw-vptestmb-1.c: Correct [xyz]mm register >>> number scans. >>> * gcc.target/i386/avx512bw-vptestmw-1.c: Likewise. >>> *

Re: [PATCH 02/18] make avail_stores a vec

Another style nit on top of the ones already posted, sorry: tbsaunde+...@tbsaunde.org writes: > +void > +print_rtx_insn_vec (FILE *file, const vec ) > +{ > + fputc('{', file); missing space before '('. Richard

Re: [middle-end][PATCH] Update alignment_for_piecewise_move

On 04/26/2016 08:21 PM, Richard Sandiford wrote: "H.J. Lu" writes: I am working a patch to enable SSE, AVX and AVX512 for memcpy/memset optimization. x86 backend defines MAX_BITSIZE_MODE_ANY_INT to 128 to keep the OI and XI modes from confusing the compiler into thinking

Re: [RFC] Update gmp/mpfr/mpc minimum versions

On Tue, Apr 26, 2016 at 06:23:17PM +, Bernd Edlinger wrote: > as we all know, it's high time now to adjust the minimum supported > gmp/mpfr/mpc versions for gcc-7. I'm not saying we shouldn't bump the minimum supported versions, but bumping them immediately to the latest releases is IMHO

[RFC] Update gmp/mpfr/mpc minimum versions

Hi, as we all know, it's high time now to adjust the minimum supported gmp/mpfr/mpc versions for gcc-7. So this attached patch is now targeting at only supporting the currently latest relased gmp-6.1.0, mpfr-3.1.4 and mpc-1.0.3, instead of trying to support all previous versions, especially

Re: [middle-end][PATCH] Update alignment_for_piecewise_move

"H.J. Lu" writes: > I am working a patch to enable SSE, AVX and AVX512 for memcpy/memset > optimization. x86 backend defines MAX_BITSIZE_MODE_ANY_INT to 128 > to keep the OI and XI modes from confusing the compiler into thinking > that these modes could actually be used for

Re: RFD: annotate iterator patterns with expanded forms

Bernd Schmidt writes: > On 01/01/2016 07:02 PM, Hans-Peter Nilsson wrote: >> On Tue, 1 Dec 2015, Bernd Schmidt wrote: > >>> The automatic Makefile approach might look something like this. The effect >>> is >>> similar to what happens when you edit tm.texi.in, except the

constexpr function caching

Jason. we currently clone constexpr function bodies when evaluating them (via the reuse cache that caused such fun). The cloning is necessary so decls in different (recursive) evaluations are distinct. We only need to clone for recursive evaluations though -- the outermost evaluation could

Re: [PATCH] PR target/70155: Use SSE for TImode load/store

On Tue, Apr 26, 2016 at 9:27 AM, Ilya Enkovich wrote: > 2016-04-26 19:20 GMT+03:00 Ilya Enkovich : >> 2016-04-26 19:12 GMT+03:00 H.J. Lu : >>> On Tue, Apr 26, 2016 at 9:07 AM, Ilya Enkovich >>> wrote:

Re: [PATCH] PR target/70155: Use SSE for TImode load/store

On Tue, Apr 26, 2016 at 9:20 AM, Ilya Enkovich wrote: > 2016-04-26 19:12 GMT+03:00 H.J. Lu : >> On Tue, Apr 26, 2016 at 9:07 AM, Ilya Enkovich >> wrote: >>> 2016-04-26 18:39 GMT+03:00 H.J. Lu : On

Re: Patches to fix optimizer bug & C++ exceptions for GCC VAX backend

On 04/01/2016 05:37 AM, Bernd Schmidt wrote: Cc'ing Matt Thomas who is listed as the vax maintainer; most of the patch should be reviewed by him IMO. If he is no longer active I'd frankly rather deprecate the port rather than invest effort in keeping it running. Index: gcc/except.c

Re: [PATCH] PR target/70155: Use SSE for TImode load/store

2016-04-26 19:20 GMT+03:00 Ilya Enkovich : > 2016-04-26 19:12 GMT+03:00 H.J. Lu : >> On Tue, Apr 26, 2016 at 9:07 AM, Ilya Enkovich >> wrote: >>> 2016-04-26 18:39 GMT+03:00 H.J. Lu : On Tue, Apr 26,

Re: Patches to fix optimizer bug & C++ exceptions for GCC VAX backend

On 04/04/2016 08:51 AM, Maciej W. Rozycki wrote: On Thu, 31 Mar 2016, Jake Hamby wrote: There's one more thing that's broken in the VAX backend which I'd *really* like to fix: GCC can't compile many of its own files at -O2, as well as a few other .c files in the NetBSD tree, because it can't

Re: [PATCH] PR target/70155: Use SSE for TImode load/store

2016-04-26 19:12 GMT+03:00 H.J. Lu : > On Tue, Apr 26, 2016 at 9:07 AM, Ilya Enkovich wrote: >> 2016-04-26 18:39 GMT+03:00 H.J. Lu : >>> On Tue, Apr 26, 2016 at 8:21 AM, Ilya Enkovich >>> wrote:

Re: [PATCH][AArch64] Improve aarch64_case_values_threshold setting

On 04/26/16 11:14, Wilco Dijkstra wrote: Evandro Menezes wrote: True, but the results when running on A53 could be quite different. GCC is ~1.2% faster on Cortex-A53 built for generic, but there is no difference in perlbench. Looks good, then. Fine by me. Thanks for your patience, --

Re: [PATCH][AArch64] Improve aarch64_case_values_threshold setting

Evandro Menezes wrote: > > True, but the results when running on A53 could be quite different. GCC is ~1.2% faster on Cortex-A53 built for generic, but there is no difference in perlbench. Wilco

Re: [PATCH] PR target/70155: Use SSE for TImode load/store

On Tue, Apr 26, 2016 at 9:07 AM, Ilya Enkovich wrote: > 2016-04-26 18:39 GMT+03:00 H.J. Lu : >> On Tue, Apr 26, 2016 at 8:21 AM, Ilya Enkovich >> wrote: >>> 2016-04-26 18:12 GMT+03:00 H.J. Lu : On

Re: [PATCH] PR target/70155: Use SSE for TImode load/store

2016-04-26 18:39 GMT+03:00 H.J. Lu : > On Tue, Apr 26, 2016 at 8:21 AM, Ilya Enkovich wrote: >> 2016-04-26 18:12 GMT+03:00 H.J. Lu : >>> On Tue, Apr 26, 2016 at 8:05 AM, Ilya Enkovich >>> wrote:

Re: Patches to fix GCC's C++ exception_handling on NetBSD/VAX

On 03/27/2016 04:09 PM, Jake Hamby wrote: Thank you for the offer. I already tested it on an Amiga 3000 w/ 68040 card when I made the fix. The bug manifested as the cross-compiler crashing with a failure to find a suitable insn, because it couldn’t find the correct FP instruction to expand to. I

Re: C, C++: New warning for memset without multiply by elt size

On Tue, Apr 26, 2016 at 9:04 AM, Bernd Schmidt wrote: > On 04/25/2016 07:55 PM, Jason Merrill wrote: >> >> On 04/25/2016 05:07 AM, Bernd Schmidt wrote: >>> >>> +if (TREE_CODE (arg2) == CONST_DECL) >>> + arg2 = DECL_INITIAL (arg2); >>> +int

Re: match.pd: unsigned A - B > A --> A < B

On Tue, 26 Apr 2016, Richard Biener wrote: On Sun, Apr 24, 2016 at 7:42 PM, Marc Glisse wrote: Hello, the first part is something that was discussed last stage3, and Jakub argued in favor of single_use. The second part is probably less useful, it notices that if we

Re: C++ PATCH to implement C++17 maybe_unused attribute

Fixed. Jason On Tue, Apr 26, 2016 at 5:46 AM, Andreas Schwab <sch...@linux-m68k.org> wrote: > /usr/local/gcc/gcc-20160426/gcc/testsuite/c-c++-common/cpp/pr63831-1.c:5:14: > error: size of array 'T1' is negative > /usr/local/gcc/gcc-20160426/gcc/testsuite/c-c++-common/cpp/pr

Re: [PATCH] Fix a recent warning in reorg.c

On 04/26/2016 05:41 PM, Jakub Jelinek wrote: On Tue, Apr 26, 2016 at 03:13:32PM +0200, Bernd Schmidt wrote: On 04/26/2016 03:08 PM, Jakub Jelinek wrote: ^ ../../gcc/reorg.c:1413:25: warning: matches this ‘i’ under old rules for (unsigned int i = len - 1; i < len; i--)

Re: Bad CC0 optimizer in VAX backend (was Re: Patches to fix GCC’s C++ exception handling on NetBSD/VAX)

On 03/28/2016 04:29 PM, Jake Hamby wrote: I have some bad news and some good news. The bad news is that there has been a nasty optimizer bug lurking in the VAX backend for GCC for many years, which has to do with over-optimistic removal of necessary tst/cmp instructions under certain

Re: [PATCH] Fix a recent warning in reorg.c

On Tue, Apr 26, 2016 at 03:13:32PM +0200, Bernd Schmidt wrote: > On 04/26/2016 03:08 PM, Jakub Jelinek wrote: > >^ > >../../gcc/reorg.c:1413:25: warning: matches this ‘i’ under old rules > >for (unsigned int i = len - 1; i < len; i--) > > ^ > > Oh, and

Re: [PATCH] PR target/70155: Use SSE for TImode load/store

On Tue, Apr 26, 2016 at 8:21 AM, Ilya Enkovich wrote: > 2016-04-26 18:12 GMT+03:00 H.J. Lu : >> On Tue, Apr 26, 2016 at 8:05 AM, Ilya Enkovich >> wrote: >>> 2016-04-26 17:55 GMT+03:00 H.J. Lu : On

[middle-end][PATCH] Update alignment_for_piecewise_move

I am working a patch to enable SSE, AVX and AVX512 for memcpy/memset optimization. x86 backend defines MAX_BITSIZE_MODE_ANY_INT to 128 to keep the OI and XI modes from confusing the compiler into thinking that these modes could actually be used for computation. But the OI and XI modes can be

Re: Patches to fix GCC’s C++ exception handling on NetBSD/VAX

On 03/26/2016 06:02 AM, Jake Hamby wrote: As an added bonus, I see that my patch set also included an old m68k patch that had been sitting in my tree, which fixes a crash when -m68040 is defined. I may have submitted it to port-m68k before. It hasn’t been tested with the new compiler either.

Re: [PATCH] PR target/70155: Use SSE for TImode load/store

On Tue, Apr 26, 2016 at 2:35 AM, Richard Biener wrote: > On Tue, 26 Apr 2016, Uros Bizjak wrote: > >> On Tue, Apr 26, 2016 at 11:17 AM, Richard Biener wrote: >> > On Mon, 25 Apr 2016, Uros Bizjak wrote: >> > >> >> On Mon, Apr 25, 2016 at 4:47 PM, H.J. Lu

Re: [PATCH] PR target/70155: Use SSE for TImode load/store

2016-04-26 18:12 GMT+03:00 H.J. Lu : > On Tue, Apr 26, 2016 at 8:05 AM, Ilya Enkovich wrote: >> 2016-04-26 17:55 GMT+03:00 H.J. Lu : >>> On Tue, Apr 26, 2016 at 7:15 AM, Ilya Enkovich >>> wrote:

Re: [PATCH] PR target/70155: Use SSE for TImode load/store

On 04/26/2016 03:17 AM, Richard Biener wrote: On Mon, 25 Apr 2016, Uros Bizjak wrote: On Mon, Apr 25, 2016 at 4:47 PM, H.J. Lu wrote: On Mon, Apr 25, 2016 at 7:18 AM, Uros Bizjak wrote: On Mon, Apr 25, 2016 at 2:51 PM, H.J. Lu

Re: [PATCH] [libatomic] Add RTEMS support

On 04/19/2016 01:24 AM, Sebastian Huber wrote: gcc/ * config/rtems.h (LIB_SPEC): Add -latomic. libatomic/ * configure.tgt (*-*-rtems*): New supported target. * config/rtems/host-config.h: New file. * config/rtems/lock.c: Likewise. IMHO, as the rtems port

Re: [PATCH] PR target/70155: Use SSE for TImode load/store

On Tue, Apr 26, 2016 at 8:05 AM, Ilya Enkovich wrote: > 2016-04-26 17:55 GMT+03:00 H.J. Lu : >> On Tue, Apr 26, 2016 at 7:15 AM, Ilya Enkovich >> wrote: >>> 2016-04-26 17:07 GMT+03:00 H.J. Lu : On

Re: [PATCH] PR target/70155: Use SSE for TImode load/store

2016-04-26 17:55 GMT+03:00 H.J. Lu : > On Tue, Apr 26, 2016 at 7:15 AM, Ilya Enkovich wrote: >> 2016-04-26 17:07 GMT+03:00 H.J. Lu : >>> On Mon, Apr 25, 2016 at 9:13 AM, Ilya Enkovich >>> wrote:

Re: [PATCH] Fix a recent warning in reorg.c

On 04/26/2016 07:08 AM, Jakub Jelinek wrote: Hi! I've noticed a warning during bootstrap: ../../gcc/reorg.c: In function ‘void try_merge_delay_insns(rtx_insn*, rtx_insn*)’: ../../gcc/reorg.c:1431:12: warning: name lookup of ‘i’ changed for (i = 0; i < XVECLEN (PATTERN (insn), 0); i++)

[C PATCH] Better location for -Wnested-externs (PR c/70791)

This patch improves the location info for -Wnested-externs. Bootstrapped/regtested on x86_64-linux, applying to trunk. 2016-04-26 Marek Polacek PR c/70791 * c-decl.c (pushdecl): Pass LOCUS down to warning. * gcc.dg/Wnested-externs-2.c: New test.

Re: [PATCH] PR target/70155: Use SSE for TImode load/store

On Tue, Apr 26, 2016 at 7:15 AM, Ilya Enkovich wrote: > 2016-04-26 17:07 GMT+03:00 H.J. Lu : >> On Mon, Apr 25, 2016 at 9:13 AM, Ilya Enkovich >> wrote: >>> 2016-04-25 18:27 GMT+03:00 H.J. Lu :

Re: [C PATCH] Follow-up fix to the misclassified token problem (PR c/67784)

On Tue, 26 Apr 2016, Marek Polacek wrote: > Ah, right, that revealed two more places that were missing the > c_parser_maybe_reclassify_token call. > > Bootstrapped/regtested on x86_64-linux, ok for trunk? OK. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH] Replace old AWK script (utilizing bc) with Python implementation

> On 04/25/2016 09:01 PM, Matthias Klose wrote: > > please could you make the shebang python3? Not sure if it's good to replace > > one old implementation with a soon to become old implementation. > > > > Matthias > > Sure, thanks for pointing out. > > Attaching v2, where I changed: > +

Re: [PATCH] PR target/70155: Use SSE for TImode load/store

2016-04-26 17:07 GMT+03:00 H.J. Lu : > On Mon, Apr 25, 2016 at 9:13 AM, Ilya Enkovich wrote: >> 2016-04-25 18:27 GMT+03:00 H.J. Lu : >>> >>> Ilya, can you take a look? >>> >>> Thanks. >>> >>> -- >>> H.J. >> >> Hi, >> >> Algorithmic

Re: [RFC][PR68217] Improve value range for signed & sign-bit-CST

On Fri, Apr 15, 2016 at 12:44 PM, kugan wrote: > As pointed out by Richard, for signed & sign-bit-CST value range should be > [-INF, 0] range, not a [-INF, INF] range as happens now. > > This patch fixes this. I bootstrapped and regression tested for >

Re: [PATCH] PR target/70155: Use SSE for TImode load/store

On Mon, Apr 25, 2016 at 9:13 AM, Ilya Enkovich wrote: > 2016-04-25 18:27 GMT+03:00 H.J. Lu : >> >> Ilya, can you take a look? >> >> Thanks. >> >> -- >> H.J. > > Hi, > > Algorithmic part of the patch looks OK to me except the following piece of > code.

Allow redefinition of libcilkrts debug macros

When working on a couple of Cilk Plus issues lately (PRs target/60290, target/68945), I noticed that you have to modify the libcilkplus sources to enable various debugging output. This seems silly, and the following patch allows defining them from the command line. Tested on i386-pc-solaris2.12

Re: [PATCH] Replace old AWK script (utilizing bc) with Python implementation

On 04/25/2016 09:01 PM, Matthias Klose wrote: > please could you make the shebang python3? Not sure if it's good to replace > one old implementation with a soon to become old implementation. > > Matthias Sure, thanks for pointing out. Attaching v2, where I changed: + switched from python2 to

Re: [PING][PATCH] New plugin event when evaluating a constexpr call

Hi, thanks for answering, 2016-04-25 16:21 GMT-03:00 Jason Merrill : > Let's create a constexpr.h rather than expose constexpr internals to all of > the front end. Really, I'd prefer to avoid exposing them at all. Why does > what you want to do require all this implementation

Re: [PATCH][AArch64] Replace insn to zero up SIMD registers

Evandro Menezes wrote: >On 03/10/16 10:37, James Greenhalgh wrote: >> Thanks for sticking with it. This is OK for GCC 7 when development >> opens. >> >> Remember to mention the most recent changes in your Changelog entry >> (Remove "fp" attribute from *movhf_aarch64 and *movtf_aarch64). > > > OK

[Patch AArch64] Set TARGET_OMIT_STRUCT_RETURN_REG to true.

As $SUBJECT. The reason this caught my eye on aarch64 is because the return value register (x0) is not identical to the register in which the hidden parameter for AArch64 is set (x8). Thus setting this to true seems to be quite reasonable and shaves off 100 odd mov x0, x8's from cc1 in a bootstrap

Re: [PATCH] Fix a recent warning in reorg.c

On 04/26/2016 03:08 PM, Jakub Jelinek wrote: ^ ../../gcc/reorg.c:1413:25: warning: matches this ‘i’ under old rules for (unsigned int i = len - 1; i < len; i--) ^ Oh, and also - I flagged this while reviewing other parts of Trevor's changes, this

Re: [PATCH] Fix a recent warning in reorg.c

On 04/26/2016 03:08 PM, Jakub Jelinek wrote: It is not fatal, but still ugly. The problem is that the function has int i; ... for (i = 0; ...) ... for (unsigned int i = ... ) ... for (i = 0; ...) This patch just declares the var in the only affected loop, so that the warning is not

Re: [C PATCH] Follow-up fix to the misclassified token problem (PR c/67784)

On Tue, Apr 26, 2016 at 03:06:16PM +0200, Marek Polacek wrote: > On Tue, Apr 26, 2016 at 11:44:52AM +, Joseph Myers wrote: > > On Tue, 26 Apr 2016, Marek Polacek wrote: > > > > > This PR was reopened, because the exact same problem with treating a > > > TYPENAME > > > wrongly as an ID was

[PATCH] Fix a recent warning in reorg.c

Hi! I've noticed a warning during bootstrap: ../../gcc/reorg.c: In function ‘void try_merge_delay_insns(rtx_insn*, rtx_insn*)’: ../../gcc/reorg.c:1431:12: warning: name lookup of ‘i’ changed for (i = 0; i < XVECLEN (PATTERN (insn), 0); i++) ^ ../../gcc/reorg.c:1263:7: warning:

Re: [PATCH] add support for placing variables in shared memory

On 04/25/16 13:49, Alexander Monakov wrote: On Mon, 25 Apr 2016, Nathan Sidwell wrote: acceptable? No, that really doesn't sound viable. You'd need to somehow take into account every instance where the compiler attempts to switch sections internally (.text/.data/.bss,

Re: [C PATCH] Follow-up fix to the misclassified token problem (PR c/67784)

On Tue, Apr 26, 2016 at 11:44:52AM +, Joseph Myers wrote: > On Tue, 26 Apr 2016, Marek Polacek wrote: > > > This PR was reopened, because the exact same problem with treating a > > TYPENAME > > wrongly as an ID was found when using just if-clause, without an enclosing > > for > > loop.

Re: C, C++: New warning for memset without multiply by elt size

On 04/25/2016 07:55 PM, Jason Merrill wrote: On 04/25/2016 05:07 AM, Bernd Schmidt wrote: +if (TREE_CODE (arg2) == CONST_DECL) + arg2 = DECL_INITIAL (arg2); +int literal_mask = ((!!integer_zerop (arg1) << 1) +| (!!integer_zerop (arg2) << 2)); Are

Re: C/C++ PATCH to add -Wdangling-else option

On 04/26/2016 02:39 PM, Jakub Jelinek wrote: I support that change, and -Wparentheses will still enable this, it just gives more fine-grained control and be in line with what clang does. Bernd, how much are you against this change? Don't really care that much, I just don't quite see the

[PATCH] Fix up inchash::add_expr to match more closely operand_equal_p (PR sanitizer/70683)

Hi! As mentioned in the PR, the bug that has been eventually worked around by making the C++ constexpr hash tables non-deletable is that operand_equal_p, which is used in lots of hash tables as the equal function (with last argument 0), may return true, i.e. a match, even for trees that have

  1   2   >