Re: gcc does not reduce the function call to the result if called function is not static when using -O2, only with -O3, clang and msvc do the optimization also with -O2

2020-12-05 Thread Dennis Luehring via Gcc
Am 05.12.2020 um 14:25 schrieb Eric Botcazou: > can someone explain to me why the -O2 optimizer is not able(allowed) to > reduce this small sample the same way as clang/msvc? Change the name of the function to something else than "main". that works, thanks!

Re: gcc does not reduce the function call to the result if called function is not static when using -O2, only with -O3, clang and msvc do the optimization also with -O2

2020-12-05 Thread Dennis Luehring via Gcc
Am 05.12.2020 um 13:04 schrieb Jan Hubicka: > gcc does not reduce to call result if called function is not static in > -O2 (will do with -O2) > clang and msvc does it also in -O2 regardless of the function beeing > static or not > > can someone explain to me why the -O2 optimizer is not

[Bug c/98159] Compiler generates infinite loop

2020-12-05 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98159 Andrew Pinski changed: What|Removed |Added Resolution|--- |INVALID Status|UNCONFIRMED

[Bug c/98159] New: Compiler generates infinite loop

2020-12-05 Thread olme8 at mail dot ru via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98159 Bug ID: 98159 Summary: Compiler generates infinite loop Product: gcc Version: 10.2.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: c

[PATCH] PR target/98152: Checking python is available before using

2020-12-05 Thread Kito Cheng
We'll try to canonicalize the arch string for --with-arch, and the script is written in python, however it will turns out GCC require python to build for RISC-V port, it's not expect as the GCC requirement. So this patch is made this as optional, detect python and only use it when it available,

[Bug target/97969] [9/10/11 Regression][ARM/Thumb] Certain combo of codegen options leads to compilation infinite loop with growing memory use

2020-12-05 Thread haoxintu at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97969 Haoxin Tu changed: What|Removed |Added CC||haoxintu at gmail dot com --- Comment #7

Re: [PATCH] RISC-V: Canonicalize --with-arch

2020-12-05 Thread Kito Cheng via Gcc-patches
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98152 Andreas Schwab has created a bug entry for this issue, using awk or shell should be fine to get the same functionality, but it might take some time, so I plan to add some checking to detect python, python2 or python3, and skip this step if none

[Bug target/98152] [11 regression] /usr/bin/env: 'python': No such file or directory

2020-12-05 Thread kito at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98152 Kito Cheng changed: What|Removed |Added Status|NEW |ASSIGNED CC|

[Bug preprocessor/96391] [10/11 Regression] internal compiler error: in linemap_compare_locations, at libcpp/line-map.c:1359

2020-12-05 Thread mike at cchtml dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96391 --- Comment #8 from Michael Cronenworth --- (In reply to Jan Smets from comment #5) > Similar issue @ https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96935 I applied the patch from that bug but it did not fix this issue. Anything I can do to help

[Bug c++/98158] New: Gcc generates warning about its own generated move assignment operator

2020-12-05 Thread darklythinking at yahoo dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98158 Bug ID: 98158 Summary: Gcc generates warning about its own generated move assignment operator Product: gcc Version: 10.2.0 Status: UNCONFIRMED Severity: normal

[Bug c++/98157] New: internal compiler error: Segmentation, gcc 10.2, -std=gnu++2a

2020-12-05 Thread michael_knoerzer at gmx dot net via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98157 Bug ID: 98157 Summary: internal compiler error: Segmentation, gcc 10.2, -std=gnu++2a Product: gcc Version: 10.2.0 Status: UNCONFIRMED Severity: normal

gcc-10-20201205 is now available

2020-12-05 Thread GCC Administrator via Gcc
Snapshot gcc-10-20201205 is now available on https://gcc.gnu.org/pub/gcc/snapshots/10-20201205/ and on various mirrors, see http://gcc.gnu.org/mirrors.html for details. This snapshot has been generated from the GCC 10 git branch with the following options: git://gcc.gnu.org/git/gcc.git branch

[Bug target/98139] varasm.c fails to compile on AIX 7.2: -Werror=unused-variable

2020-12-05 Thread dje at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98139 David Edelsohn changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug c++/97645] Rejects valid subscript expression on array of unknown bound in constant expression

2020-12-05 Thread leni536 at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97645 --- Comment #2 from Lénárd Szolnoki --- (In reply to Lénárd Szolnoki from comment #0) > This was possibly impossible prior to C++20 to subscript an array of unknown > bound in constexpr context. As of P0388 conversion from T[N] to T[] can >

Re: introduce overridable clear_cache emitter

2020-12-05 Thread Jakub Jelinek via Gcc-patches
On Sat, Dec 05, 2020 at 06:01:59PM -0300, Alexandre Oliva wrote: > On Dec 5, 2020, Andreas Schwab wrote: > > > ../../../../libffi/src/aarch64/ffi.c: In function 'ffi_prep_closure_loc': > > ../../../../libffi/src/aarch64/ffi.c:67:3: internal compiler error: in > > emit_library_call_value_1, at

[Bug target/83466] Wrong TLS GD sequence for ILP32

2020-12-05 Thread schwab--- via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83466 --- Comment #6 from Andreas Schwab --- The same issue exists for SYMBOL_SMALL_TLSIE.

Re: introduce overridable clear_cache emitter

2020-12-05 Thread Alexandre Oliva
On Dec 5, 2020, Andreas Schwab wrote: > ../../../../libffi/src/aarch64/ffi.c: In function 'ffi_prep_closure_loc': > ../../../../libffi/src/aarch64/ffi.c:67:3: internal compiler error: in > emit_library_call_value_1, at calls.c:5300 >67 | __builtin___clear_cache (start, end); > |

[Bug testsuite/98156] New: [Coarray] alloc_comp_1.f90 tests for wrong condition

2020-12-05 Thread tkoenig at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98156 Bug ID: 98156 Summary: [Coarray] alloc_comp_1.f90 tests for wrong condition Product: gcc Version: 11.0 Status: UNCONFIRMED Severity: normal Priority: P3

[C PATCH] fix atomic loads [PR 97981]

2020-12-05 Thread Uecker, Martin
Hi Joseph, the patch to drop qualifiers during lvalue conversion was broken, because the code to emit atomic loads did not trigger anymore. I now added a test that scans for "atomic_load". I should have taken the new warning for _Atomic int y; y; // warning statement with no effect as a

[Bug c++/97093] ICE on C++20 code when chaining requirements (in hashtab_chk_error, at hash-table.c:137)

2020-12-05 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97093 --- Comment #3 from CVS Commits --- The master branch has been updated by Patrick Palka : https://gcc.gnu.org/g:40234200864b6c0d0079abbcdc7a4139b60257ff commit r11-5799-g40234200864b6c0d0079abbcdc7a4139b60257ff Author: Patrick Palka Date:

[Bug c++/97093] ICE on C++20 code when chaining requirements (in hashtab_chk_error, at hash-table.c:137)

2020-12-05 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97093 --- Comment #2 from CVS Commits --- The master branch has been updated by Patrick Palka : https://gcc.gnu.org/g:904ac8577521b8152b97e9b549c1a1ca569a3d1f commit r11-5798-g904ac8577521b8152b97e9b549c1a1ca569a3d1f Author: Patrick Palka Date:

Re: Updating the backend status for h8300 on the wiki

2020-12-05 Thread John Paul Adrian Glaubitz
Hi Jeff! On 11/30/20 8:22 PM, Jeff Law wrote: > I also took care of updating cris and mn103 status WRT cc0. Attached is > what was pushed to the gcc-wwwdocs trunk. Now that VAX has been converted as well, the entry can be updated here as well. Now there are only AVR and CR16 that need to be

[Bug target/95294] [vax] Convert the backend to MODE_CC so it can be kept in future releases

2020-12-05 Thread glaubitz at physik dot fu-berlin.de via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95294 --- Comment #5 from John Paul Adrian Glaubitz --- (In reply to Maciej W. Rozycki from comment #4) > Conversion complete. Great. The Bountysource campaign closes the moment you close this bug report. Then every supporter can vote whether they

Re: [PATCH 31/31] PR target/95294: VAX: Add test cases for MODE_CC representation

2020-12-05 Thread Maciej W. Rozycki
On Fri, 20 Nov 2020, Jeff Law wrote: > Sweet.  OK once the prereqs are all ACK'd. Thank you for your review. I have applied all the changes now, as posted in their most recent versions, except for 01/31 where I have noticed an anomaly in the included

[Bug target/95294] [vax] Convert the backend to MODE_CC so it can be kept in future releases

2020-12-05 Thread macro--- via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95294 --- Comment #4 from Maciej W. Rozycki --- Conversion complete.

[Bug target/58901] vax bootstrap fails on subreg reload

2020-12-05 Thread macro--- via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58901 Maciej W. Rozycki changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |ma...@linux-mips.org

[Bug target/95294] [vax] Convert the backend to MODE_CC so it can be kept in future releases

2020-12-05 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95294 --- Comment #3 from CVS Commits --- The master branch has been updated by Maciej W. Rozycki : https://gcc.gnu.org/g:4f47fca1d0fc2c6efef791d6dd115abd9d221733 commit r11-5797-g4f47fca1d0fc2c6efef791d6dd115abd9d221733 Author: Maciej W. Rozycki

[Bug target/95294] [vax] Convert the backend to MODE_CC so it can be kept in future releases

2020-12-05 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95294 --- Comment #2 from CVS Commits --- The master branch has been updated by Maciej W. Rozycki : https://gcc.gnu.org/g:e552abe2ba27ccec0d7c0d050b4617f2f636dd63 commit r11-5796-ge552abe2ba27ccec0d7c0d050b4617f2f636dd63 Author: Maciej W. Rozycki

[Bug lto/67548] [5 Regression] LTO drops weak binding with "ld -r"

2020-12-05 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67548 --- Comment #17 from CVS Commits --- The master branch has been updated by Maciej W. Rozycki : https://gcc.gnu.org/g:76a553587f3181605c57801c37b0d3e94ce3aca5 commit r11-5795-g76a553587f3181605c57801c37b0d3e94ce3aca5 Author: Maciej W. Rozycki

[Bug lto/69866] lto1: internal compiler error: in add_symbol_to_partition_1, at lto/lto-partition.c:158

2020-12-05 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69866 --- Comment #17 from CVS Commits --- The master branch has been updated by Maciej W. Rozycki : https://gcc.gnu.org/g:76a553587f3181605c57801c37b0d3e94ce3aca5 commit r11-5795-g76a553587f3181605c57801c37b0d3e94ce3aca5 Author: Maciej W. Rozycki

[Bug target/58901] vax bootstrap fails on subreg reload

2020-12-05 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58901 --- Comment #8 from CVS Commits --- The master branch has been updated by Maciej W. Rozycki : https://gcc.gnu.org/g:a27d5f9a73978f20cfef1796a94f6a1a82438146 commit r11-5761-ga27d5f9a73978f20cfef1796a94f6a1a82438146 Author: Matt Thomas Date:

Re: testsuite: Adjust target requirements for sad-vectorize and signbit

2020-12-05 Thread David Edelsohn via Gcc-patches
On Sat, Dec 5, 2020 at 6:12 AM Alan Modra wrote: > > On Thu, Oct 29, 2020 at 10:10:58PM +1030, Alan Modra wrote: > > Fixes > > FAIL: gcc.target/powerpc/signbit-1.c scan-assembler-not stxvd2x > > FAIL: gcc.target/powerpc/signbit-1.c scan-assembler-times mfvsrd 3 > > FAIL:

[Bug target/98152] [11 regression] /usr/bin/env: 'python': No such file or directory

2020-12-05 Thread marxin at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98152 Martin Liška changed: What|Removed |Added Last reconfirmed||2020-12-05 CC|

[Bug rtl-optimization/97421] aarch64: Wrong code with -O2 -fmodulo-sched

2020-12-05 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97421 --- Comment #9 from CVS Commits --- The master branch has been updated by Roman Zhuykov : https://gcc.gnu.org/g:4eb8f93d026eaa1de9b4820337069f3ce3465cd0 commit r11-5760-g4eb8f93d026eaa1de9b4820337069f3ce3465cd0 Author: Roman Zhuykov Date:

Testsuite integration for devel/coarray_native

2020-12-05 Thread Thomas König
Hi, I'm currently trying to put together a testsuite for the shared coarray branch. What I have so far is a directory gcc/testsuite/gfortran.dg/caf-shared which contains the attached dejagnu file plus the test cases This works with the command make check-fortran

[Bug lto/98155] New: [11 Regression] ICE during GIMPLE pass: slp, in vect_init_pattern_stmt

2020-12-05 Thread doko at debian dot org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98155 Bug ID: 98155 Summary: [11 Regression] ICE during GIMPLE pass: slp, in vect_init_pattern_stmt Product: gcc Version: 11.0 Status: UNCONFIRMED Severity: normal

[Bug c++/98151] integer output gives different results with -O2 and -O3

2020-12-05 Thread bradley_bell at yahoo dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98151 --- Comment #5 from Brad Bell --- Compiling with g++ -fsanitize=undefined -O2 int_output.cpp -o int_output does in fact give a very useful message I scanned the page https://www.gnu.org/software/gcc/bugs/ and looked for things that

[Bug fortran/98141] Segmentation fault with empty string sourced allocation

2020-12-05 Thread dominiq at lps dot ens.fr via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98141 Dominique d'Humieres changed: What|Removed |Added Ever confirmed|0 |1 Status|UNCONFIRMED

[Bug fortran/98016] Host association problem

2020-12-05 Thread pault at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98016 Paul Thomas changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug fortran/98016] Host association problem

2020-12-05 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98016 --- Comment #9 from CVS Commits --- The master branch has been updated by Paul Thomas : https://gcc.gnu.org/g:7ae210d5db5eb96989b2f639e65678b5597e61f0 commit r11-5759-g7ae210d5db5eb96989b2f639e65678b5597e61f0 Author: Paul Thomas Date: Sat

[Bug c/98154] avr-size (avr-binutils v2.35) don't' accept --formart=avr

2020-12-05 Thread redi at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98154 --- Comment #2 from Jonathan Wakely --- Binutils is not part of GCC. It has its own bugzilla at https://sourceware.org/bugzilla

[Bug c++/98151] integer output gives different results with -O2 and -O3

2020-12-05 Thread redi at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98151 --- Comment #4 from Jonathan Wakely --- Compiling with -fsanitize=undefined (as suggested by the bug reporting guidelines) would have shown the undefined behaviour.

[Bug target/98147] [11 Regression] ICE in emit_library_call_value_1, at calls.c:5296 since r11-5725-g442b6fb7c09a39577261de90413cc4db366f1c5f

2020-12-05 Thread jakub at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98147 Jakub Jelinek changed: What|Removed |Added CC||jakub at gcc dot gnu.org --- Comment #1

[Bug target/97865] libtool needs to be updated for Darwin20.

2020-12-05 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97865 --- Comment #28 from CVS Commits --- The master branch has been updated by Iain D Sandoe : https://gcc.gnu.org/g:1352bc88a0525743c952197fb2db6e4f8c091cde commit r11-5758-g1352bc88a0525743c952197fb2db6e4f8c091cde Author: Iain Sandoe Date:

Re: gcc does not reduce the function call to the result if called function is not static when using -O2, only with -O3, clang and msvc do the optimization also with -O2

2020-12-05 Thread Eric Botcazou
> can someone explain to me why the -O2 optimizer is not able(allowed) to > reduce this small sample the same way as clang/msvc? Change the name of the function to something else than "main". -- Eric Botcazou

[Bug c/98154] avr-size (avr-binutils v2.35) don't' accept --formart=avr

2020-12-05 Thread lul at isep dot ipp.pt via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98154 --- Comment #1 from Luis --- avr-binutils version 2.35 with version 2.32 work's fine

[Bug c/98154] New: avr-size (avr-binutils v2.35) don't' accept --formart=avr

2020-12-05 Thread lul at isep dot ipp.pt via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98154 Bug ID: 98154 Summary: avr-size (avr-binutils v2.35) don't' accept --formart=avr Product: gcc Version: unknown Status: UNCONFIRMED Severity: normal

Re: [PATCH] phiopt, v2: Handle bool in two_value_replacement [PR796232]

2020-12-05 Thread Richard Biener
On December 5, 2020 11:59:22 AM GMT+01:00, Jakub Jelinek wrote: >On Sat, Dec 05, 2020 at 11:22:24AM +0100, Richard Biener wrote: >> >two_value_replacement and didn't restrict it to !early_p. Shall we >> >instead >> >do two_value_replacement only in late phiopt? >> >> Yeah, I guess that would

Re: [PATCH] match.pd: Improve conditional_replacement for x ? 0 : -1 [PR796232]

2020-12-05 Thread Richard Biener
On December 5, 2020 11:57:46 AM GMT+01:00, Jakub Jelinek wrote: >On Sat, Dec 05, 2020 at 11:20:11AM +0100, Richard Biener wrote: >> >As mentioned in the PR, for boolean x we currently optimize >> >in phiopt x ? 0 : -1 into -(int)!x but it can be optimized as >> >(int) x - 1 which is one less

Re: [PATCH] Avoid atomic for guard acquire when that is expensive

2020-12-05 Thread Bernd Edlinger
On 12/2/20 7:57 PM, Jason Merrill wrote: > On 12/1/20 1:28 PM, Bernd Edlinger wrote: >> On 11/24/20 11:10 PM, Jason Merrill wrote: >>> On 11/22/20 3:05 AM, Bernd Edlinger wrote: Hi, this avoids the need to use -fno-threadsafe-statics on arm-none-eabi or working around that

[Bug lto/98145] gcc with nvptx offloading on Windows: lto-wrapper can't find accel/nvptx-none/mkoffload

2020-12-05 Thread brechtsanders at users dot sourceforge.net via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98145 --- Comment #3 from Brecht Sanders --- Fixing gcc/config/nvptx/mkoffload.c got me one step further, but now the parameters seem to be the issue: lto1.exe: error: unrecognized command-line option '-mgomp' lto1.exe: error: '-foffload-abi' option

Re: gcc does not reduce the function call to the result if called function is not static when using -O2, only with -O3, clang and msvc do the optimization also with -O2

2020-12-05 Thread Jan Hubicka
> gcc does not reduce to call result if called function is not static in > -O2 (will do with -O2) > clang and msvc does it also in -O2 regardless of the function beeing > static or not > > can someone explain to me why the -O2 optimizer is not able(allowed) to > reduce this small sample the same

gcc does not reduce the function call to the result if called function is not static when using -O2, only with -O3, clang and msvc do the optimization also with -O2

2020-12-05 Thread Dennis Luehring via Gcc
gcc does not reduce to call result if called function is not static in -O2 (will do with -O2) clang and msvc does it also in -O2 regardless of the function beeing static or not can someone explain to me why the -O2 optimizer is not able(allowed) to reduce this small sample the same way as

Re: testsuite: Adjust target requirements for sad-vectorize and signbit

2020-12-05 Thread Alan Modra via Gcc-patches
On Thu, Oct 29, 2020 at 10:10:58PM +1030, Alan Modra wrote: > Fixes > FAIL: gcc.target/powerpc/signbit-1.c scan-assembler-not stxvd2x > FAIL: gcc.target/powerpc/signbit-1.c scan-assembler-times mfvsrd 3 > FAIL: gcc.target/powerpc/signbit-1.c scan-assembler-times srdi 3 > FAIL:

[PATCH] phiopt, v2: Handle bool in two_value_replacement [PR796232]

2020-12-05 Thread Jakub Jelinek via Gcc-patches
On Sat, Dec 05, 2020 at 11:22:24AM +0100, Richard Biener wrote: > >two_value_replacement and didn't restrict it to !early_p. Shall we > >instead > >do two_value_replacement only in late phiopt? > > Yeah, I guess that would make sense. I don't remember exactly but I saw > regressions with early

[PATCH] match.pd: Improve conditional_replacement for x ? 0 : -1 [PR796232]

2020-12-05 Thread Jakub Jelinek via Gcc-patches
On Sat, Dec 05, 2020 at 11:20:11AM +0100, Richard Biener wrote: > >As mentioned in the PR, for boolean x we currently optimize > >in phiopt x ? 0 : -1 into -(int)!x but it can be optimized as > >(int) x - 1 which is one less operation both in GIMPLE and in x86 > >assembly. > > >

Re: [PATCH] phiopt: Handle bool in two_value_replacement [PR796232]

2020-12-05 Thread Richard Biener
On December 5, 2020 10:14:49 AM GMT+01:00, Jakub Jelinek wrote: >Hi! > >The following patch improves code generation on the included testcase >by >enabling two_value_replacement on booleans. It does that only for >arg0/arg1 >values that conditional_replacement doesn't handle, and only does it

Re: [PATCH] phiopt: Improve conditional_replacement for x ? 0 : -1 [PR796232]

2020-12-05 Thread Richard Biener
On December 5, 2020 10:10:25 AM GMT+01:00, Jakub Jelinek wrote: >Hi! > >As mentioned in the PR, for boolean x we currently optimize >in phiopt x ? 0 : -1 into -(int)!x but it can be optimized as >(int) x - 1 which is one less operation both in GIMPLE and in x86 >assembly. >

Re: introduce overridable clear_cache emitter

2020-12-05 Thread Andreas Schwab
../../../../libffi/src/aarch64/ffi.c: In function 'ffi_prep_closure_loc': ../../../../libffi/src/aarch64/ffi.c:67:3: internal compiler error: in emit_library_call_value_1, at calls.c:5300 67 | __builtin___clear_cache (start, end); | ^~~~ Andreas. --

Re: [PATCH] [X86_64]: Enable support for next generation AMD Zen3 CPU

2020-12-05 Thread Jan Hubicka
> [AMD Public Use] > > > As per https://gcc.gnu.org/codingconventions.html#ChangeLogs > > --Snip-- > ChangeLogs > ChangeLog entries are part of git commit messages and are automatically put > into a corresponding ChangeLog file. > --Snip-- > > This means Changelog files will be updated

[Bug c++/64236] [c++11] last alignas overrides stricter on a class definition

2020-12-05 Thread hahnjo at hahnjo dot de via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64236 Jonas Hahnfeld changed: What|Removed |Added CC||hahnjo at hahnjo dot de --- Comment #6

[Bug go/98153] New: [11 Regression] libgo ftbfs on i686-gnu

2020-12-05 Thread doko at debian dot org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98153 Bug ID: 98153 Summary: [11 Regression] libgo ftbfs on i686-gnu Product: gcc Version: 11.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: go

[Bug c++/98151] integer output gives different results with -O2 and -O3

2020-12-05 Thread jakub at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98151 Jakub Jelinek changed: What|Removed |Added Status|WAITING |RESOLVED Resolution|---

[PATCH] phiopt: Handle bool in two_value_replacement [PR796232]

2020-12-05 Thread Jakub Jelinek via Gcc-patches
Hi! The following patch improves code generation on the included testcase by enabling two_value_replacement on booleans. It does that only for arg0/arg1 values that conditional_replacement doesn't handle, and only does it if not in the early phiopt pass, because conditional_replacement isn't

Re: [PATCH 0/8] [RS6000] rs6000_rtx_costs V2

2020-12-05 Thread Alan Modra via Gcc-patches
Hi Segher, I've been holding off pinging these knowing you had a lot of other review work, but maybe that's settling down now? You already OK'd 1/8, 2/8 and 6/8. [PATCH 3/8] [RS6000] rs6000_rtx_costs tidy AND https://gcc.gnu.org/pipermail/gcc-patches/2020-October/555754.html [PATCH 4/8]

[PATCH] phiopt: Improve conditional_replacement for x ? 0 : -1 [PR796232]

2020-12-05 Thread Jakub Jelinek via Gcc-patches
Hi! As mentioned in the PR, for boolean x we currently optimize in phiopt x ? 0 : -1 into -(int)!x but it can be optimized as (int) x - 1 which is one less operation both in GIMPLE and in x86 assembly. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? And/or, shall we have a

Re: [PATCH] [X86_64]: Enable support for next generation AMD Zen3 CPU

2020-12-05 Thread Jakub Jelinek via Gcc-patches
On Sat, Dec 05, 2020 at 08:22:23AM +, Kumar, Venkataramanan via Gcc-patches wrote: > [AMD Public Use] > > > As per https://gcc.gnu.org/codingconventions.html#ChangeLogs > > --Snip-- > ChangeLogs > ChangeLog entries are part of git commit messages and are automatically put > into a

[Bug target/98152] [11 regression] /usr/bin/env: 'python': No such file or directory

2020-12-05 Thread schwab--- via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98152 Andreas Schwab changed: What|Removed |Added Target Milestone|--- |11.0 Keywords|

[Bug target/98152] New: [11 regression] /usr/bin/env: 'python': No such file or directory

2020-12-05 Thread schwab--- via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98152 Bug ID: 98152 Summary: [11 regression] /usr/bin/env: 'python': No such file or directory Product: gcc Version: 11.0 Status: UNCONFIRMED Severity: normal

RE: [PATCH] [X86_64]: Enable support for next generation AMD Zen3 CPU

2020-12-05 Thread Kumar, Venkataramanan via Gcc-patches
[AMD Public Use] As per https://gcc.gnu.org/codingconventions.html#ChangeLogs --Snip-- ChangeLogs ChangeLog entries are part of git commit messages and are automatically put into a corresponding ChangeLog file. --Snip-- This means Changelog files will be updated automatically? I did not do