Re: movmem pattern and missed alignment

2018-10-09 Thread Richard Biener
On Tue, Oct 9, 2018 at 1:53 PM Joseph Myers wrote: > > On Tue, 9 Oct 2018, Richard Biener wrote: > > > It was repeatedly suggested that we _could_ derive alignment info from > > function parameter types since we rely on precise typing there for example > > for points-to analysis (albeit only for

Re: movmem pattern and missed alignment

2018-10-09 Thread Joseph Myers
On Tue, 9 Oct 2018, Richard Biener wrote: > It was repeatedly suggested that we _could_ derive alignment info from > function parameter types since we rely on precise typing there for example > for points-to analysis (albeit only for restrict qualification processing and > for DECL_BY_REFERENCE

[Bug middle-end/63155] [6/7/8 Regression] memory hog

2018-10-09 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63155 --- Comment #45 from Richard Biener --- Author: rguenth Date: Tue Oct 9 11:43:46 2018 New Revision: 264956 URL: https://gcc.gnu.org/viewcvs?rev=264956=gcc=rev Log: 2018-10-09 Richard Biener PR tree-optimization/63155 *

[Bug gcov-profile/77698] Unrolled loop not considered hot after profiling

2018-10-09 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77698 Martin Liška changed: What|Removed |Added Status|REOPENED|ASSIGNED --- Comment #6 from Martin

Re: [PATCHv2] Handle not explicitly zero terminated strings in merge sections

2018-10-09 Thread Rainer Orth
Hi Eric, >> Besides, the patch seems to have produced more fallout on Solaris: I see >> many new Go testsuite failures on Solaris 10 which probably are related, >> and Solaris bootstrap with Ada included is broken due to the extended >> usage of string merging. I'm currently investigating what's

[PATCH] Help PR63155 PTA hog a bit

2018-10-09 Thread Richard Biener
This helps us throw away constraints from uninitialized stuff earlier. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2018-10-09 Richard Biener PR tree-optimization/63155 * tree-ssa-structalias.c: Include tree-ssa.h.

Re: [PATCH] Come up with gcc/testsuite/g++.target/i386/i386.dg and move there some tests.

2018-10-09 Thread Martin Liška
On 10/9/18 11:03 AM, Rainer Orth wrote: > Hi Martin, > >> rename from gcc/testsuite/g++.dg/ext/pr82625.C >> rename to gcc/testsuite/g++.target/i386/pr82625.C >> index 59b174f8c51..0eb70baed5e 100644 >> --- a/gcc/testsuite/g++.dg/ext/pr82625.C >> +++ b/gcc/testsuite/g++.target/i386/pr82625.C >> @@

[Bug middle-end/24639] [meta-bug] bug to track all Wuninitialized issues

2018-10-09 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=24639 Bug 24639 depends on bug 87564, which changed state. Bug 87564 Summary: Missing -Wuninitialized with -O0 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87564 What|Removed |Added

[Bug middle-end/87564] Missing -Wuninitialized with -O0

2018-10-09 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87564 Richard Biener changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug libstdc++/87544] alloc-size-larger-than incorrectly triggered

2018-10-09 Thread glisse at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87544 --- Comment #10 from Marc Glisse --- (In reply to Jonathan Wakely from comment #9) > Do we want something like this as well? (and for malloc_allocator too) I think so. Changing allocator_traits as LWG seems likely to agree won't help much until

[Bug bootstrap/87551] [9 regression] libgnat-9.so fails to link on Solaris

2018-10-09 Thread ro at CeBiTec dot Uni-Bielefeld.DE
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87551 --- Comment #3 from ro at CeBiTec dot Uni-Bielefeld.DE --- > --- Comment #2 from ro at CeBiTec dot Uni-Bielefeld.DE Uni-Bielefeld.DE> --- >> --- Comment #1 from Bernd Edlinger --- >> Rainer, can you try this? > > Looks good so far: an

[PATCH] Fix cut fallout

2018-10-09 Thread Richard Biener
Committed as obvious. Richard. 2018-10-09 Richard Biener * tree-vectorizer.c (dump_stmt_cost): Fix cut missing replacements. diff --git a/gcc/tree-vectorizer.c b/gcc/tree-vectorizer.c index 0ab366b79a3..60ee7f6380c 100644 --- a/gcc/tree-vectorizer.c +++

[Bug target/87561] [9 Regression] 416.gamess is slower by ~10% starting from r264866 with -Ofast

2018-10-09 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87561 --- Comment #2 from Richard Biener --- OK, so on haswell I see (- is bad, + is good): -0x2342ca0 _40 + _45 1 times scalar_stmt costs 12 in body +0x2342ca0 _40 + _45 1 times scalar_stmt costs 4 in body so a simple add changes cost from 4 to 12

Re: [PATCH] Make std::list::iterator == and != global inline friend

2018-10-09 Thread Jonathan Wakely
On 09/10/18 07:25 +0200, François Dumont wrote: As we talked one day I would like to make all iterator operators global for consistency. So here is the patch to do so for std::list iterators. By "global" you mean "non-member", right? Thanks to this change the operators ==(iterator,

[Bug middle-end/57832] compiling sha-256 code (xz 5.0.5) generates false warnings when using -march=native on Atom CPU

2018-10-09 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=57832 Martin Liška changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug c/85870] [6/7/8/9 Regression][LTO1] ICE in linemap_line_start, at libcpp/line-map.c:794

2018-10-09 Thread ramana at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85870 --- Comment #8 from Ramana Radhakrishnan --- (In reply to Martin Liška from comment #5) > (In reply to Ramana Radhakrishnan from comment #4) > > (In reply to Martin Liška from comment #3) > > > Can't reproduce with GCC 7.3.0 on x86_64: > > > >

[Bug c/85870] [6/7/8/9 Regression][LTO1] ICE in linemap_line_start, at libcpp/line-map.c:794

2018-10-09 Thread sudi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85870 --- Comment #7 from sudi at gcc dot gnu.org --- It is not failing on x86_64 trunk anymore but with 8.0.1 + TARGET=x86_64-pc-linux-gnu + GCC_INSTALL=/work/x86-trunk/bld + GCC=/work/x86-trunk/bld/bin/x86_64-pc-linux-gnu-gcc-8.0.1 +

[Bug libstdc++/87544] alloc-size-larger-than incorrectly triggered

2018-10-09 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87544 --- Comment #9 from Jonathan Wakely --- Do we want something like this as well? (and for malloc_allocator too) --- a/libstdc++-v3/include/ext/new_allocator.h +++ b/libstdc++-v3/include/ext/new_allocator.h @@ -130,7 +130,13 @@

[Bug c/85870] [6/7/8/9 Regression][LTO1] ICE in linemap_line_start, at libcpp/line-map.c:794

2018-10-09 Thread sudi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85870 --- Comment #6 from sudi at gcc dot gnu.org --- Still fails for me on aarch64-none-linux-gnu-gcc and aarch64-none-elf-gcc on trunk and gcc-8.2.1 with the same error Reading object files: test_1.o test_2.olto1: internal compiler error: in

[Bug c++/85114] -fstack-check causes internal compiler error

2018-10-09 Thread ebotcazou at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85114 Eric Botcazou changed: What|Removed |Added CC||ebotcazou at gcc dot gnu.org ---

[Bug middle-end/87564] New: Missing -Wuninitialized with -O0

2018-10-09 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87564 Bug ID: 87564 Summary: Missing -Wuninitialized with -O0 Product: gcc Version: 9.0 Status: UNCONFIRMED Keywords: diagnostic Severity: normal Priority: P3

[Bug target/87563] [9 regression ] ICE with -march=armv8-a+sve

2018-10-09 Thread ktkachov at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87563 ktkachov at gcc dot gnu.org changed: What|Removed |Added CC||ktkachov at gcc dot gnu.org

[Bug fortran/84487] [8/9 Regression] Large rodate section increase in 465.tonto with r254427

2018-10-09 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84487 --- Comment #9 from Martin Liška --- Can please anybody from Fotran community dig into this?

[Bug c++/85114] -fstack-check causes internal compiler error

2018-10-09 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85114 Martin Liška changed: What|Removed |Added Keywords|needs-reduction | CC|

[Bug c/85870] [6/7/8/9 Regression][LTO1] ICE in linemap_line_start, at libcpp/line-map.c:794

2018-10-09 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85870 --- Comment #5 from Martin Liška --- (In reply to Ramana Radhakrishnan from comment #4) > (In reply to Martin Liška from comment #3) > > Can't reproduce with GCC 7.3.0 on x86_64: > > > > + gcc-7 -O2 -flto -c test_1.i -o test_1.o > > + gcc-7 -O2

[Bug c/85870] [6/7/8/9 Regression][LTO1] ICE in linemap_line_start, at libcpp/line-map.c:794

2018-10-09 Thread ramana at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85870 Ramana Radhakrishnan changed: What|Removed |Added Status|WAITING |NEW --- Comment #4 from Ramana

[Bug c/85870] [6/7/8/9 Regression][LTO1] ICE in linemap_line_start, at libcpp/line-map.c:794

2018-10-09 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85870 Martin Liška changed: What|Removed |Added Status|NEW |WAITING --- Comment #3 from Martin Liška

[Bug c++/84191] [7 Regression] Compiler ICEs when trying to resolve impossible arithmetic operations

2018-10-09 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84191 Martin Liška changed: What|Removed |Added CC||marxin at gcc dot gnu.org Known to

[Bug gcov-profile/87553] [9 regression] g++.dg/tree-prof/inline_mismatch_args.C etc. FAIL

2018-10-09 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87553 --- Comment #9 from Martin Liška --- (In reply to r...@cebitec.uni-bielefeld.de from comment #8) > > --- Comment #7 from Martin Liška --- > [...] > > You can use gcov-dump -l to dump content of the files. However, it's not > > problem as the

[Bug tree-optimization/87562] [9 Regression] ICE in in linemap_position_for_line_and_column, at libcpp/line-map.c:848

2018-10-09 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87562 --- Comment #1 from David Malcolm --- linemap_position_for_line_and_column(line_maps*, line_map_ordinary const*, unsigned int, unsigned int) at libcpp/line-map.c:848 is: linemap_assert (ORDINARY_MAP_STARTING_LINE_NUMBER (ord_map) <= line); I

[Bug c++/87547] G++ reports bad type names for bit-field members

2018-10-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87547 Jakub Jelinek changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned

[Bug c++/87559] Storage duration + lambda captures: Discrepancy in behavior between g++-7 and clang++-6.0

2018-10-09 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87559 --- Comment #4 from Jonathan Wakely --- Yes. Started to ICE with r253266 and was fixed by r261121.

[Bug target/87563] [9 regression ] ICE with -march=armv8-a+sve

2018-10-09 Thread ramana at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87563 Ramana Radhakrishnan changed: What|Removed |Added Target||aarch64-none-elf Target

[Bug target/87563] [9 regression ] ICE with -march=armv8-a+sve

2018-10-09 Thread ramana at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87563 Ramana Radhakrishnan changed: What|Removed |Added Keywords||ice-on-valid-code

[Bug target/87563] New: [9 regression ] ICE with -march=armv8-a+sve

2018-10-09 Thread ramana at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87563 Bug ID: 87563 Summary: [9 regression ] ICE with -march=armv8-a+sve Product: gcc Version: 9.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: target

[Bug c++/87559] Storage duration + lambda captures: Discrepancy in behavior between g++-7 and clang++-6.0

2018-10-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87559 --- Comment #3 from Jakub Jelinek --- Yeah. I think it is r261121 aka PR85761 that fixed the ICE. Wonder if it would be useful to add the #c1 testcase into testsuite or if lambda-const8.C is close enough that it covers it.

[C++ Patch PING] Re: [C++ Patch] PR 84423 ("[6/7/8/9 Regression] [concepts] ICE with invalid using declaration")

2018-10-09 Thread Paolo Carlini
Hi, gently pinging the below... On 29/09/18 21:27, Paolo Carlini wrote: Hi again, On 9/28/18 9:15 PM, Paolo Carlini wrote: Thanks. About the location, you are certainly right, but doesn't seem trivial. Something we can do *now* is using declspecs->locations[ds_typedef] and

[Bug c++/87559] Storage duration + lambda captures: Discrepancy in behavior between g++-7 and clang++-6.0

2018-10-09 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87559 --- Comment #2 from Jonathan Wakely --- (In reply to Jonathan Wakely from comment #1) > But GCC 8 gets them all right 8.1 crashes with an ICE (which makes bisection hard), 8.2 gets them right.

[Bug fortran/58787] ICE (error recovery) in check_proc_interface

2018-10-09 Thread juergen.reuter at desy dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58787 --- Comment #14 from Jürgen Reuter --- (In reply to Dominique d'Humieres from comment #13) > > (null):0: confused by earlier errors, bailing out > > Your compiler is configured with --enable-checking=release (either > explicitly or because your

[Bug fortran/58787] ICE (error recovery) in check_proc_interface

2018-10-09 Thread dominiq at lps dot ens.fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58787 --- Comment #13 from Dominique d'Humieres --- > (null):0: confused by earlier errors, bailing out Your compiler is configured with --enable-checking=release (either explicitly or because your are using a release). The above message is the

[Bug gcov-profile/87553] [9 regression] g++.dg/tree-prof/inline_mismatch_args.C etc. FAIL

2018-10-09 Thread ro at CeBiTec dot Uni-Bielefeld.DE
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87553 --- Comment #8 from ro at CeBiTec dot Uni-Bielefeld.DE --- > --- Comment #7 from Martin Liška --- [...] > You can use gcov-dump -l to dump content of the files. However, it's not > problem as the file exists. The warning should be only shown

[Bug c++/87559] Storage duration + lambda captures: Discrepancy in behavior between g++-7 and clang++-6.0

2018-10-09 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87559 --- Comment #1 from Jonathan Wakely --- Any gcc bugs seem to be fixed in current trunk. As a single testcase: extern "C" int puts(const char*); constexpr char top_doc[] = ""; void f1() { constexpr auto& doc = top_doc; [](int) { puts(doc);

[Bug fortran/58787] ICE (error recovery) in check_proc_interface

2018-10-09 Thread juergen.reuter at desy dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58787 --- Comment #12 from Jürgen Reuter --- (In reply to Dominique d'Humieres from comment #11) > > I can confirm that this ICEs on Linux, but not on MACOSX. > > I get the ICE with MACOSX: > > ... > Error: Expecting END SUBROUTINE statement at (1)

[Bug fortran/58787] ICE (error recovery) in check_proc_interface

2018-10-09 Thread dominiq at lps dot ens.fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58787 --- Comment #11 from Dominique d'Humieres --- > I can confirm that this ICEs on Linux, but not on MACOSX. I get the ICE with MACOSX: ... Error: Expecting END SUBROUTINE statement at (1) f951: internal compiler error: Segmentation fault: 11

Re: introduce --enable-mingw-full32 to default to --large-address-aware

2018-10-09 Thread JonY
On 10/09/2018 04:59 AM, Alexandre Oliva wrote: > On Oct 5, 2018, Joseph Myers wrote: > >> A new configure option needs documenting in install.texi. > > Ah, yes, thanks for the reminder. > > On Oct 6, 2018, JonY <10wa...@gmail.com> wrote: > >> They're both OK as far as I can see. I just

[Bug fortran/86576] [F03][OOP] Sourced allocation of object array fails with SEGFAULT

2018-10-09 Thread pault at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86576 --- Comment #4 from Paul Thomas --- (In reply to Dominique d'Humieres from comment #3) > AFAICT the test in comment 2 has been fixed between revisions r264451 > (2018-09-20) and r264486 (2018-09-21), may be r264485 (pr87359). Unfortunately, the

[Bug fortran/58787] ICE (error recovery) in check_proc_interface

2018-10-09 Thread juergen.reuter at desy dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58787 Jürgen Reuter changed: What|Removed |Added CC||juergen.reuter at desy dot de ---

[Bug fortran/86576] [F03][OOP] Sourced allocation of object array fails with SEGFAULT

2018-10-09 Thread dominiq at lps dot ens.fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86576 Dominique d'Humieres changed: What|Removed |Added CC||pault at gcc dot gnu.org ---

Re: movmem pattern and missed alignment

2018-10-09 Thread Richard Biener
On Tue, Oct 9, 2018 at 11:23 AM Jakub Jelinek wrote: > > On Tue, Oct 09, 2018 at 11:08:44AM +0200, Richard Biener wrote: > > On Tue, Oct 9, 2018 at 11:00 AM Alexander Monakov > > wrote: > > > > > > On Tue, 9 Oct 2018, Richard Biener wrote: > > > > > > > > then we cannot set the alignment of i_1

[Bug middle-end/63155] [6/7/8 Regression] memory hog

2018-10-09 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63155 --- Comment #44 from Richard Biener --- (In reply to Richard Biener from comment #43) > This makes CCP the main > offender again but as said the rectification would probably mean pulling > back the SSA SCC discovery code from SCCVN and use that

Re: movmem pattern and missed alignment

2018-10-09 Thread Jakub Jelinek
On Tue, Oct 09, 2018 at 11:08:44AM +0200, Richard Biener wrote: > On Tue, Oct 9, 2018 at 11:00 AM Alexander Monakov wrote: > > > > On Tue, 9 Oct 2018, Richard Biener wrote: > > > > > > then we cannot set the alignment of i_1 at/after k = *i_1 because doing > > > so would > > > affect the

[Bug c++/87410] internal compiler error: in fold_convert_loc, at fold-const.c:2530

2018-10-09 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87410 Martin Liška changed: What|Removed |Added Keywords||ice-on-valid-code

[Bug fortran/55735] ICE with deferred-length strings in COMMON

2018-10-09 Thread juergen.reuter at desy dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55735 --- Comment #10 from Jürgen Reuter --- Interestingly, nagfor rejects this code with the message "Inconsistent definitions of COMMON block FOO in program-units $block and BAR". Both ifort and pgfortran compile the code, and the program issues

[Bug target/87561] [9 Regression] 416.gamess is slower by ~10% starting from r264866 with -Ofast

2018-10-09 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87561 Richard Biener changed: What|Removed |Added Target||x86_64-*-*, i?86-*-*

[Bug c++/86740] [8/9 Regression] ICE with hana and nested lambdas (likely a regression, tsubst_copy, at cp/pt.c:15325)

2018-10-09 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86740 Martin Liška changed: What|Removed |Added Keywords|needs-reduction | CC|

[Bug c/87562] New: ICE in in linemap_position_for_line_and_column, at libcpp/line-map.c:848

2018-10-09 Thread ktkachov at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87562 Bug ID: 87562 Summary: ICE in in linemap_position_for_line_and_column, at libcpp/line-map.c:848 Product: gcc Version: unknown Status: UNCONFIRMED Keywords:

[Bug c/87562] [9 Regression] ICE in in linemap_position_for_line_and_column, at libcpp/line-map.c:848

2018-10-09 Thread ktkachov at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87562 ktkachov at gcc dot gnu.org changed: What|Removed |Added Target Milestone|--- |9.0 Summary|ICE

Re: movmem pattern and missed alignment

2018-10-09 Thread Richard Biener
On Tue, Oct 9, 2018 at 11:00 AM Alexander Monakov wrote: > > On Tue, 9 Oct 2018, Richard Biener wrote: > > > > then we cannot set the alignment of i_1 at/after k = *i_1 because doing so > > would > > affect the alignment test which we'd then optimize away. We'd need to > > introduce > > a SSA

Re: [PATCH] Come up with gcc/testsuite/g++.target/i386/i386.dg and move there some tests.

2018-10-09 Thread Rainer Orth
Hi Martin, > rename from gcc/testsuite/g++.dg/ext/pr82625.C > rename to gcc/testsuite/g++.target/i386/pr82625.C > index 59b174f8c51..0eb70baed5e 100644 > --- a/gcc/testsuite/g++.dg/ext/pr82625.C > +++ b/gcc/testsuite/g++.target/i386/pr82625.C > @@ -1,7 +1,7 @@ > /* { dg-do compile } */ > /* {

Re: movmem pattern and missed alignment

2018-10-09 Thread Alexander Monakov
On Tue, 9 Oct 2018, Richard Biener wrote: > > then we cannot set the alignment of i_1 at/after k = *i_1 because doing so > would > affect the alignment test which we'd then optimize away. We'd need to > introduce > a SSA copy to get a new SSA name but that would be optimized away quickly. We

Re: [PATCH] Come up with gcc/testsuite/g++.target/i386/i386.dg and move there some tests.

2018-10-09 Thread Martin Liška
Hi. There's another move of C++ tests, this time these that have dg-require-ifunc. Key question is whether we want to make subfolders for i386 tests (ext, other, ..)? Survives make check -k RUNTESTFLAGS="i386.exp" Martin >From d36db4d5b8306dcbe2d63762bc8596e05132e46a Mon Sep 17 00:00:00 2001

Re: [PATCH, ARM] Clean up arm backend using the @ construct for MD patterns

2018-10-09 Thread Ramana Radhakrishnan
On 09/10/2018 09:27, Mihail Ionescu wrote: > Hi all, > > This patch removes some of the machine mode checks from the arm backend when > emitting instructions by using the '@' construct (Parameterized Names[2]). It > is based on the previous AArch64 patch[1]. > >

Re: movmem pattern and missed alignment

2018-10-09 Thread Eric Botcazou
> It was repeatedly suggested that we _could_ derive alignment info from > function parameter types since we rely on precise typing there for example > for points-to analysis (albeit only for restrict qualification processing > and for DECL_BY_REFERENCE "pointers"). That would fix the simple

[Bug target/87561] New: [9 Regression] 416.gamess is slower by ~10% starting from r264866 with -Ofast

2018-10-09 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87561 Bug ID: 87561 Summary: [9 Regression] 416.gamess is slower by ~10% starting from r264866 with -Ofast Product: gcc Version: 9.0 Status: UNCONFIRMED Severity:

[Bug fortran/55735] ICE with deferred-length strings in COMMON

2018-10-09 Thread pault at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55735 --- Comment #9 from Paul Thomas --- (In reply to Jürgen Reuter from comment #7) > Ah sorry, I think I moved around the block data and then it wasn't valid > Fortran anymore. I think, both the block data and the subroutine are > external to the

Re: movmem pattern and missed alignment

2018-10-09 Thread Richard Biener
On Tue, Oct 9, 2018 at 10:02 AM Andrew Haley wrote: > > On 10/08/2018 07:38 PM, Paul Koning wrote: > > > > > >> On Oct 8, 2018, at 1:29 PM, Andrew Haley wrote: > >> > >> On 10/08/2018 06:20 PM, Michael Matz wrote: > >>> Only if you somewhere visibly add accesses to *i and *j. Without them you >

[Bug fortran/55735] ICE with deferred-length strings in COMMON

2018-10-09 Thread pault at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55735 --- Comment #8 from Paul Thomas --- (In reply to Jürgen Reuter from comment #7) > Ah sorry, I think I moved around the block data and then it wasn't valid > Fortran anymore. I think, both the block data and the subroutine are > external to the

Re: [PATCH] Come up with --param asan-stack-small-redzone (PR sanitizer/81715).

2018-10-09 Thread Martin Liška
PING^1 On 9/26/18 11:33 AM, Martin Liška wrote: > On 9/25/18 5:53 PM, Jakub Jelinek wrote: >> On Tue, Sep 25, 2018 at 05:26:44PM +0200, Martin Liška wrote: >>> The only missing piece is how to implement asan_emit_redzone_payload more >>> smart. >>> It means doing memory stores with 8,4,2,1 sizes

Re: movmem pattern and missed alignment

2018-10-09 Thread Richard Biener
On Tue, Oct 9, 2018 at 8:41 AM Eric Botcazou wrote: > > > It's not quite obvious what SSE has to do with this - any hint please? > > SSE introduced alignment constraints into the non-strict-alignment target x86 > so people didn't really want to play by the rules of strict-alignment targets.

Re: [PATCH] ASAN: emit line information of stack variables.

2018-10-09 Thread Jakub Jelinek
On Thu, Sep 27, 2018 at 10:55:10AM +0200, Martin Liška wrote: > @@ -1281,15 +1284,30 @@ asan_emit_stack_protection (rtx base, rtx pbase, > unsigned int alignb, >pp_space (_pp); >pp_wide_integer (_pp, offsets[l - 1] - offsets[l]); >pp_space (_pp); > + > +

[PATCH] Remove dead functions and fix VMS target by moving back some functions.

2018-10-09 Thread Martin Liška
Hi. Utilizing rtags' --find-dead-functions I'm suggesting a removal of part of the functions reported with the script. I built all cross compilers defined in contrib/config-list.mk and I fixed VMS targets that I broke in previous removal. If the folks are happy with the removal, I can probably

Re: [PATCH] ASAN: emit line information of stack variables.

2018-10-09 Thread Martin Liška
PING^1 On 9/27/18 10:55 AM, Martin Liška wrote: > Hi. > > I've noticed ASAN can inform user about location of stack variables > when a stack violation is detected. > > Sample example: > > ... > This frame has 3 object(s): > [32, 36) 'counter' (line 3) <== Memory access at offset 36

[Bug middle-end/63155] [6/7/8 Regression] memory hog

2018-10-09 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63155 --- Comment #43 from Richard Biener --- We're now down to tree PTA : 3.92 ( 16%) 0.12 ( 36%) 4.02 ( 16%) 12445 kB ( 2%) tree CCP : 7.43 ( 30%) 0.02 ( 6%) 7.44 ( 29%) 646

Re: movmem pattern and missed alignment

2018-10-09 Thread Andrew Haley
On 10/08/2018 07:38 PM, Paul Koning wrote: > > >> On Oct 8, 2018, at 1:29 PM, Andrew Haley wrote: >> >> On 10/08/2018 06:20 PM, Michael Matz wrote: >>> Only if you somewhere visibly add accesses to *i and *j. Without them you >>> only have the "accesses" via memcpy, and as Richi says, those

[Bug fortran/80931] ICE on move_alloc in gimplify_expr, at gimplify.c:11335

2018-10-09 Thread pault at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80931 --- Comment #9 from Paul Thomas --- Author: pault Date: Tue Oct 9 07:46:48 2018 New Revision: 264949 URL: https://gcc.gnu.org/viewcvs?rev=264949=gcc=rev Log: 2018-10-09 Paul Thomas PR fortran/87151 * trans-array.c

[Bug fortran/87151] allocating array of character

2018-10-09 Thread pault at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87151 --- Comment #5 from Paul Thomas --- Author: pault Date: Tue Oct 9 07:46:48 2018 New Revision: 264949 URL: https://gcc.gnu.org/viewcvs?rev=264949=gcc=rev Log: 2018-10-09 Paul Thomas PR fortran/87151 * trans-array.c

[Bug target/87550] Intrinsics for rdpmc (__rdpmc, __builtin_ia32_rdpmc) are interpreted as pure functions

2018-10-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87550 Jakub Jelinek changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned

[Bug testsuite/87557] New test case g++.dg/ext/pr82625.C in r264845 doesn't compile

2018-10-09 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87557 --- Comment #2 from Martin Liška --- Yes, but I forgot to move the file to newly created ./gcc/testsuite/g++.target/i386/ I'll do that.

[Bug ipa/87554] [8/9 Regression] internal compiler error: in record_reference, at cgraphbuild.c:64

2018-10-09 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87554 Martin Liška changed: What|Removed |Added CC||jason at gcc dot gnu.org --- Comment #5

[Bug target/87560] ICE in curr_insn_transform, at lra-constraints.c:3892

2018-10-09 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87560 Segher Boessenkool changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug testsuite/87557] New test case g++.dg/ext/pr82625.C in r264845 doesn't compile

2018-10-09 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87557 Richard Biener changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

RE: [PATCH][GCC][mid-end] Add a hook to support telling the mid-end when to probe the stack [patch (2/6)]

2018-10-09 Thread Richard Biener
On Tue, 9 Oct 2018, Tamar Christina wrote: > Hi All, > > I'm looking for permission to backport this patch to the GCC-8 branch > to fix PR86486. > > OK for backport? This doesn't seem to fix a regression and it's been on trunk only for a few days. Richard. > Thanks, > Tamar > > >

[Bug gcov-profile/87553] [9 regression] g++.dg/tree-prof/inline_mismatch_args.C etc. FAIL

2018-10-09 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87553 --- Comment #7 from Martin Liška --- (In reply to r...@cebitec.uni-bielefeld.de from comment #6) > > --- Comment #5 from Martin Liška --- > [...] > >> Sorry, I've been doing too many things at once and not been paying close > >> enough

[Bug target/86383] [9 Regression] arm-netbsdelf cross compiler fails in selftests

2018-10-09 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86383 --- Comment #9 from Martin Liška --- Any progress on that please?

[Bug ipa/87554] [8/9 Regression] internal compiler error: in record_reference, at cgraphbuild.c:64

2018-10-09 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87554 --- Comment #4 from Martin Liška --- I'm reducing that ...

[Bug target/87560] New: ICE in curr_insn_transform, at lra-constraints.c:3892

2018-10-09 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87560 Bug ID: 87560 Summary: ICE in curr_insn_transform, at lra-constraints.c:3892 Product: gcc Version: 9.0 Status: UNCONFIRMED Keywords: ice-on-valid-code Severity: normal

FW: PROPOSAL: Extend inline asm syntax with size spec

2018-10-09 Thread David Laight
Resend to gcc@gcc.gnu.org to avoid spam filter > From: Michael Matz > > Sent: 07 October 2018 16:53 > ... > > I think the examples I saw from Boris were all indirect inlines: > > > > static inline void foo() { asm("large-looking-but-small-asm"); } > > static void bar1() { ... foo() ... } > >

Re: movmem pattern and missed alignment

2018-10-09 Thread Eric Botcazou
> It's not quite obvious what SSE has to do with this - any hint please? SSE introduced alignment constraints into the non-strict-alignment target x86 so people didn't really want to play by the rules of strict-alignment targets. > (according to my quick check this changed between gcc-4.5 and

RE: [PATCH][GCC][front-end][build-machinery][opt-framework] Allow setting of stack-clash via configure options. [Patch (4/6)]

2018-10-09 Thread Tamar Christina
Hi All, I'm looking for permission to backport this patch to the GCC-8 branch to fix PR86486. OK for backport? Thanks, Tamar > -Original Message- > From: Jeff Law > Sent: Friday, August 3, 2018 19:03 > To: Tamar Christina ; Joseph Myers > > Cc: gcc-patches@gcc.gnu.org; nd ;

RE: [PATCH][GCC][AArch64] Cleanup the AArch64 testsuite when stack-clash is on [Patch (7/7)]

2018-10-09 Thread Tamar Christina
Hi All, I'm looking for permission to backport this patch to the GCC-8 branch to fix PR86486. OK for backport? Thanks, Tamar > -Original Message- > From: gcc-patches-ow...@gcc.gnu.org > On Behalf Of Tamar Christina > Sent: Friday, September 28, 2018 17:36 > To: Jeff Law ;

RE: [PATCH][GCC][AArch64] Set default values for stack-clash and do basic validation in back-end. [Patch (5/6)]

2018-10-09 Thread Tamar Christina
Hi All, I'm looking for permission to backport this patch to the GCC-8 branch to fix PR86486. OK for backport? Thanks, Tamar > -Original Message- > From: James Greenhalgh > Sent: Tuesday, July 31, 2018 22:02 > To: Tamar Christina > Cc: gcc-patches@gcc.gnu.org; nd ; Richard Earnshaw >

RE: [PATCH][GCC][mid-end] Add a hook to support telling the mid-end when to probe the stack [patch (2/6)]

2018-10-09 Thread Tamar Christina
Hi All, I'm looking for permission to backport this patch to the GCC-8 branch to fix PR86486. OK for backport? Thanks, Tamar > -Original Message- > From: Jeff Law > Sent: Wednesday, July 11, 2018 19:53 > To: Tamar Christina ; gcc-patches@gcc.gnu.org > Cc: nd ; rguent...@suse.de;

RE: [PATCH 8/8][GCC][AArch64] stack-clash: Add LR assert to layout_frame.

2018-10-09 Thread Tamar Christina
Hi All, I'm looking for permission to backport this patch to the GCC-8 branch to fix PR86486. OK for backport? Thanks, Tamar > -Original Message- > From: gcc-patches-ow...@gcc.gnu.org > On Behalf Of Tamar Christina > Sent: Wednesday, September 26, 2018 09:30 > To:

RE: [PATCH][GCC][AArch64] Ensure that outgoing argument size is at least 8 bytes when alloca and stack-clash. [Patch (3/6)]

2018-10-09 Thread Tamar Christina
Hi All, I'm looking for permission to backport this patch to the GCC-8 branch to fix PR86486. OK for backport? Thanks, Tamar > -Original Message- > From: James Greenhalgh > Sent: Tuesday, August 7, 2018 17:18 > To: Tamar Christina > Cc: Jeff Law ; gcc-patches@gcc.gnu.org; nd > ;

RE: [PATCH][GCC][AArch64] Updated stack-clash implementation supporting 64k probes. [patch (1/7)]

2018-10-09 Thread Tamar Christina
Hi All, I'm looking for permission to backport this patch to the GCC-8 branch to fix PR86486. OK for backport? Thanks, Tamar > -Original Message- > From: James Greenhalgh > Sent: Tuesday, September 11, 2018 16:56 > To: Tamar Christina > Cc: Richard Sandiford ; Jeff Law > ;

RE: [PATCH][GCC][AArch64] Add support for SVE stack clash probing [patch (2/7)]

2018-10-09 Thread Tamar Christina
Hi All, I'm looking for permission to backport this patch to the GCC-8 branch to fix PR86486. OK for backport? Thanks, Tamar > -Original Message- > From: Richard Sandiford > Sent: Friday, September 28, 2018 18:18 > To: Tamar Christina > Cc: gcc-patches@gcc.gnu.org; nd ; James

<    1   2