[Bug c++/88003] ICE on outside definition of inner function-local class in poplevel_class, at cp/name-lookup.c:4325

2020-08-24 Thread nathan at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88003 Nathan Sidwell changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |nathan at gcc dot gnu.org

Re: [PATCH 0/6] Parallelize Intra-Procedural Optimizations using the LTO Engine.

2020-08-24 Thread Giuliano Belinassi via Gcc-patches
Ho, Josh. On 08/24, Josh Triplett wrote: > On Sat, Aug 22, 2020 at 06:04:48PM -0300, Giuliano Belinassi wrote: > > Hi, Josh > > > > On 08/21, Josh Triplett wrote: > > > On Thu, Aug 20, 2020 at 07:00:13PM -0300, Giuliano Belinassi wrote: > > > > This patch series add a new flag "-fparallel-jobs="

[Bug target/96772] Power VSX libmvec implementation for OpenMP SIMD

2020-08-24 Thread dje at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96772 David Edelsohn changed: What|Removed |Added Status|UNCONFIRMED |NEW Ever confirmed|0

[Bug target/96772] New: Power VSX libmvec implementation for OpenMP SIMD

2020-08-24 Thread dje at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96772 Bug ID: 96772 Summary: Power VSX libmvec implementation for OpenMP SIMD Product: gcc Version: unknown Status: UNCONFIRMED Severity: enhancement Priority: P3

Re: [PATCH] sra: Bail out when encountering accesses with negative offsets (PR 96730)

2020-08-24 Thread Richard Biener
On August 24, 2020 5:07:56 PM GMT+02:00, Martin Jambor wrote: >Hi, > >I must admit I was quite surprised to see that SRA does not disqualify >an aggregate from any transformations when it encounters an offset for >which get_ref_base_and_extent returns a negative offset. It may not >matter too

Re: Clobber REG_CC only for some constraint alternatives?

2020-08-24 Thread Jeff Law via Gcc
On Thu, 2020-08-20 at 21:36 +0530, Senthil Kumar Selvaraj via Gcc wrote: > Pip Cet writes: > > > On Tue, Aug 18, 2020 at 6:52 AM Senthil Kumar Selvaraj > > wrote: > > > > recognize such insns, but as it stands that define_insn would > > > > recognize the incorrect insn: > > > > > > > > [(set

[committed] doc: Switch valgrind.com to https

2020-08-24 Thread Gerald Pfeifer
Pushed. Jonathan, I found two more references to valgrind.com without https (which lead to a redirect) at http://gcc.gnu.org/onlinedocs/libstdc++/manual/debug.html http://valgrind.org/docs/manual/drd-manual.html http://valgrind.org/docs/manual/hg-manual.html Would you mind having a look,

[committed] wwwdocs: Switch www.validlab.com to https

2020-08-24 Thread Gerald Pfeifer
Pushed. Gerald. --- htdocs/bugs/index.html | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/htdocs/bugs/index.html b/htdocs/bugs/index.html index a6631d8a..88fba1b9 100644 --- a/htdocs/bugs/index.html +++ b/htdocs/bugs/index.html @@ -300,7 +300,7 @@ to round to the nearest

Re: [PATCH] cmpelim: recognize extra clobbers in insns

2020-08-24 Thread Jeff Law via Gcc-patches
On Thu, 2020-08-06 at 12:42 +, Pip Cet via Gcc-patches wrote: > I'm working on the AVR cc0 -> CCmode conversion (bug#92729). One > problem is that the cmpelim pass is currently very strict in requiring > insns of the form > > (parallel [(set (reg:SI) (op:SI ... ...)) >(clobber

Re: [PATCH 1/6] Modify gcc driver for parallel compilation

2020-08-24 Thread Giuliano Belinassi via Gcc-patches
Hi, Richi. On 08/24, Richard Biener wrote: > On Fri, Aug 21, 2020 at 12:00 AM Giuliano Belinassi > wrote: > > > > Update the driver for parallel compilation. This process work as > > follows: > > > > When calling gcc, the driver will check if the flag > > "-fparallel-jobs" was provided by the

Re: reorg.c (fill_slots_from_thread): Improve for TARGET_FLAGS_REGNUM targets

2020-08-24 Thread Jeff Law via Gcc-patches
On Fri, 2020-08-14 at 07:27 +0200, Hans-Peter Nilsson via Gcc-patches wrote: > Originally I thought to bootstrap this patch on MIPS and SPARC > since they're both delayed-branch-slot targets but I > reconsidered, as neither is a TARGET_FLAGS_REGNUM target. It > seems only visium and CRIS has this

Re: PING [Patch][Middle-end]Add -fzero-call-used-regs=[skip|used-gpr|all-gpr|used|all]

2020-08-24 Thread Qing Zhao via Gcc-patches
> On Aug 24, 2020, at 12:49 PM, Segher Boessenkool > wrote: > > On Wed, Aug 19, 2020 at 06:27:45PM -0500, Qing Zhao wrote: >>> On Aug 19, 2020, at 5:57 PM, Segher Boessenkool >>> wrote: >>> Numbers on how expensive this is (for what arch, in code size and in >>> execution time) would be

Re: PING [Patch][Middle-end]Add -fzero-call-used-regs=[skip|used-gpr|all-gpr|used|all]

2020-08-24 Thread Segher Boessenkool
[ Please quote correctly. I fixed this up a bit. ] On Mon, Aug 24, 2020 at 02:47:22PM +, Rodriguez Bahena, Victor wrote: > > The call-clobbered regs are the only ones you *can* touch. That does > > not mean you should clear them all (it doesn't help much at all in some > > cases). Only the

Re: PING [Patch][Middle-end]Add -fzero-call-used-regs=[skip|used-gpr|all-gpr|used|all]

2020-08-24 Thread Segher Boessenkool
On Wed, Aug 19, 2020 at 06:27:45PM -0500, Qing Zhao wrote: > > On Aug 19, 2020, at 5:57 PM, Segher Boessenkool > > wrote: > > Numbers on how expensive this is (for what arch, in code size and in > > execution time) would be useful. If it is so expensive that no one will > > use it, it helps

[Bug middle-end/96733] std::clamp for floats and doubles produces worse code than a combo of std::min / std::max

2020-08-24 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96733 --- Comment #4 from Jakub Jelinek --- (In reply to Jonathan Wakely from comment #3) > Shouldn't this help in the library then? > > if (__hi < __lo) > __builtin_unreachable(); For integers, I guess it could if Andrew's/Aldy's

[Bug middle-end/96733] std::clamp for floats and doubles produces worse code than a combo of std::min / std::max

2020-08-24 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96733 --- Comment #3 from Jonathan Wakely --- Shouldn't this help in the library then? if (__hi < __lo) __builtin_unreachable();

Re: [PATCH] gimple-fold: Don't optimize wierdo floating point value reads [PR95450]

2020-08-24 Thread Jeff Law via Gcc-patches
On Mon, 2020-08-24 at 12:51 +0200, Richard Biener wrote: > On Wed, 12 Aug 2020, Jakub Jelinek wrote: > > > On Wed, Aug 12, 2020 at 04:30:35PM +0200, Richard Biener wrote: > > > Not a final review but if we care for this kind of normalization at all > > > the we should do so consistently, thus for

Re: [PATCH 3/5] Add DWARF5 variants of assembly scan tests that use DW_FORM_implicit_const

2020-08-24 Thread Jakub Jelinek via Gcc-patches
On Mon, Aug 24, 2020 at 02:56:56PM +0200, Mark Wielaard wrote: > Some DWARF tests scan the assembly output looking for constant values. > When using DWARF5 those constants might use DW_FORM_implicit_const, > which are output (in the comments) after the attribute instead of > before. To make sure

Re: [PATCH 2/5] Make sure that static data member constexpr isn't optimized away in test.

2020-08-24 Thread Jakub Jelinek via Gcc-patches
On Mon, Aug 24, 2020 at 02:56:55PM +0200, Mark Wielaard wrote: > In DWARF5 class variables (static data members) are represented with a > DW_TAG_variable instead of a DW_TAG_member. Make sure the variable isn't > optimized away in the constexpr-var-1.C testcase so we can still match (2) >

Re: [PATCH 1/5] Don't enable -gvariable-location-views by default for DWARF5.

2020-08-24 Thread Jakub Jelinek via Gcc-patches
On Mon, Aug 24, 2020 at 02:56:54PM +0200, Mark Wielaard wrote: > DWARF5 makes it possible to read loclists tables without consulting > the debuginfo tree by introducing a table header. Adding location views > breaks this (at least for binutils and elfutils). So don't enable >

Re: [PATCH] improve validation of attribute arguments (PR c/78666)

2020-08-24 Thread Martin Sebor via Gcc-patches
On 8/24/20 4:59 AM, Aldy Hernandez wrote: On 8/21/20 1:37 AM, Martin Sebor wrote: On 8/20/20 3:00 PM, Aldy Hernandez wrote: Regardless, here are some random comments. Thanks for the careful review! diff --git a/gcc/c-family/c-attribs.c b/gcc/c-family/c-attribs.c index

Re: [PATCH 0/6] Parallelize Intra-Procedural Optimizations using the LTO Engine.

2020-08-24 Thread Josh Triplett
On Sat, Aug 22, 2020 at 06:04:48PM -0300, Giuliano Belinassi wrote: > Hi, Josh > > On 08/21, Josh Triplett wrote: > > On Thu, Aug 20, 2020 at 07:00:13PM -0300, Giuliano Belinassi wrote: > > > This patch series add a new flag "-fparallel-jobs=" to control if the > > > compiler should try to

Re: [PATCH] Fortran : ICE for division by zero in declaration PR95882

2020-08-24 Thread Thomas Koenig via Gcc-patches
Hi Mark, OK to commit and backport? The test cases mentioned in the ChangeLog are not in the patch, instead there is the test case for PR 96624. Could you correct that? Best regards Thomas

[PATCH] x86: Use target("baseline-isas-only") in

2020-08-24 Thread H.J. Lu via Gcc-patches
On Mon, Aug 24, 2020 at 7:55 AM Uros Bizjak wrote: > > On Mon, Aug 24, 2020 at 3:23 PM H.J. Lu wrote: > > > > Speaking of pragmas, these should be added outside cpuid.h, like: > > > > > > #pragma GCC push_options > > > #pragma GCC target("general-regs-only") > > > > > > #include > > > > > >

Re: [PATCH] hppa: Improve expansion of ashldi3 when !TARGET_64BIT

2020-08-24 Thread Jeff Law via Gcc-patches
On Fri, 2020-08-21 at 13:53 +0100, Roger Sayle wrote: > This patch improves the code generated on PA-RISC for DImode > (double word) left shifts by small constants (1-31). This target > has a very cool shd instruction that can be recognized by combine > for simple shifts, but relying on combine

[Bug middle-end/96733] std::clamp for floats and doubles produces worse code than a combo of std::min / std::max

2020-08-24 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96733 Jakub Jelinek changed: What|Removed |Added CC||jakub at gcc dot gnu.org,

Re: [PATCH] arm: Fix -mpure-code support/-mslow-flash-data for armv8-m.base [PR94538]

2020-08-24 Thread Christophe Lyon via Gcc-patches
On Mon, 24 Aug 2020 at 11:09, Christophe Lyon wrote: > > On Sat, 22 Aug 2020 at 00:44, Ramana Radhakrishnan > wrote: > > > > On Wed, Aug 19, 2020 at 10:32 AM Christophe Lyon via Gcc-patches > > wrote: > > > > > > armv8-m.base (cortex-m23) has the movt instruction, so we need to > > > disable

[Bug middle-end/96771] New: arm/pr32920-2.c fails since svn r228175 / f11a7b6d57f6fcba1bf2e5a0403dc49120195320

2020-08-24 Thread clyon at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96771 Bug ID: 96771 Summary: arm/pr32920-2.c fails since svn r228175 / f11a7b6d57f6fcba1bf2e5a0403dc49120195320 Product: gcc Version: 10.0 Status: UNCONFIRMED

[committed] libstdc++: Add deduction guide for std::ranges::join_view [LWG 3474]

2020-08-24 Thread Jonathan Wakely via Gcc-patches
This implements the proposed resolution for LWG 3474. libstdc++-v3/ChangeLog: * include/std/ranges (join_view): Add deduction guide (LWG 3474). * testsuite/std/ranges/adaptors/join_lwg3474.cc: New test. Tested powerpc64le-linux. Committed to trunk. commit

[committed] libstdc++: Fix std::indirectly_readable ambiguity [LWG 3446]

2020-08-24 Thread Jonathan Wakely via Gcc-patches
This implements the proposed resolution of LWG 3446. I'm also adding another new constrained specialization which isn't proposed by 3446, to resolve the ambiguity when a type has both value_type and element_type but denoting different types. libstdc++-v3/ChangeLog: *

Re: [PATCH] libstdc++: Fix iota_view::size() to avoid overflow

2020-08-24 Thread Jonathan Wakely via Gcc-patches
On 20/08/20 19:54 +0100, Jonathan Wakely wrote: This avoids overfow that occurs when negating the most negative value of an integral type. Also prevent returning signed int when the values have lower rank and promote to int. libstdc++-v3/ChangeLog: * include/std/ranges

[committed] libstdc++: Make variant_npos conversions explicit [PR 96766]

2020-08-24 Thread Jonathan Wakely via Gcc-patches
libstdc++-v3/ChangeLog: PR libstdc++/96766 * include/std/variant (_Variant_storage): Replace implicit conversions from size_t to __index_type with explicit casts. This fixes a silly Clang UBsan warning from its silly unsigned-integer-overflow sanitizer which complains

[Bug libstdc++/96766] std::swap(std::variant, std::variant) triggers undefined behavior sanitizer

2020-08-24 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96766 --- Comment #4 from CVS Commits --- The master branch has been updated by Jonathan Wakely : https://gcc.gnu.org/g:074436cf8cdd2a9ce75cadd36deb8301f00e55b9 commit r11-2822-g074436cf8cdd2a9ce75cadd36deb8301f00e55b9 Author: Jonathan Wakely Date:

Re: [PATCH 0/6] Parallelize Intra-Procedural Optimizations using the LTO Engine.

2020-08-24 Thread Giuliano Belinassi via Gcc-patches
Hi, Richi. On 08/24, Richard Biener wrote: > On Fri, Aug 21, 2020 at 12:00 AM Giuliano Belinassi > wrote: > > > > This patch series add a new flag "-fparallel-jobs=" to control if the > > compiler should try to compile the current file in parallel. > > > > There are three modes which is

Re: [PATCH] Fix libstdc++ testsuite to handle VxWorks gthreads implementation

2020-08-24 Thread Jonathan Wakely via Gcc-patches
On 21/08/20 09:37 -0300, Alexandre Oliva wrote: On Dec 20, 2019, Jonathan Wakely wrote: On 10/12/19 15:58 +0100, Corentin Gay wrote: This patch was tested on x86_64-linux and is part of our nightly testing on all platforms, including VxWorks. Was it tested on AIX? I think

[Bug tree-optimization/96730] [10/11 Regression] ICE on x86_64-linux-gnu with `-O1` to `-O3` (in verify_sra_access_forest, at tree-sra.c:2352) since r10-6320-g5b9e89c922dc2e7e

2020-08-24 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96730 --- Comment #3 from Martin Jambor --- I have proposed a fix on the mailing list: https://gcc.gnu.org/pipermail/gcc-patches/2020-August/552488.html

[PATCH] sra: Bail out when encountering accesses with negative offsets (PR 96730)

2020-08-24 Thread Martin Jambor
Hi, I must admit I was quite surprised to see that SRA does not disqualify an aggregate from any transformations when it encounters an offset for which get_ref_base_and_extent returns a negative offset. It may not matter too much because I sure hope such programs always have undefined behavior

Re: [PATCH] Fix ICE.

2020-08-24 Thread Uros Bizjak via Gcc-patches
On Mon, Aug 24, 2020 at 3:03 PM Hongtao Liu wrote: > > Hi: > This patch is to fix a typo in my last patch [1]. > [1] https://gcc.gnu.org/pipermail/gcc-patches/2020-August/551982.html > > Bootstrap is ok, gcc regression test hosted on CLX for i386/x86-64 > backend is ok. > Ok for trunk? >

Re: [PATCH] x86: Use -march=x86-64/-march=i386 in

2020-08-24 Thread Uros Bizjak via Gcc-patches
On Mon, Aug 24, 2020 at 3:23 PM H.J. Lu wrote: > > Speaking of pragmas, these should be added outside cpuid.h, like: > > > > #pragma GCC push_options > > #pragma GCC target("general-regs-only") > > > > #include > > > > void cpuid_check () > > ... > > > > #pragma GCC pop_options > > > >

Re: PING [Patch][Middle-end]Add -fzero-call-used-regs=[skip|used-gpr|all-gpr|used|all]

2020-08-24 Thread Qing Zhao via Gcc-patches
> On Aug 24, 2020, at 5:50 AM, Richard Biener wrote: > > On Tue, 11 Aug 2020, Qing Zhao wrote: > >> Hi, Alexandre, >> >> CC’ing Richard for his comments on this. >> >> >>> On Aug 10, 2020, at 9:39 PM, Alexandre Oliva wrote: I think that moving how to zeroing the registers part to

Re: PING [Patch][Middle-end]Add -fzero-call-used-regs=[skip|used-gpr|all-gpr|used|all]

2020-08-24 Thread Rodriguez Bahena, Victor via Gcc-patches
From: Qing Zhao Date: Wednesday, August 19, 2020 at 6:28 PM To: Segher Boessenkool , "Rodriguez Bahena, Victor" Cc: Richard Biener , Jeff Law , Uros Bizjak , "H. J. Lu" , Jakub Jelinek , GCC Patches , Kees Cook Subject: Re: PING [Patch][Middle-end]Add

Re: Alloca documentation

2020-08-24 Thread Jonathan Wakely via Gcc
On Mon, 24 Aug 2020 at 15:39, Philip R Brenan via Gcc wrote: > > Hi *GCC*: > > Given the example for *alloca*() at: > > https://www.gnu.org/software/libc/manual/html_mono/libc.html#Alloca-Example > > May I recommend that the code below portrays the use of alloca() more > completely in that it

Alloca documentation

2020-08-24 Thread Philip R Brenan via Gcc
Hi *GCC*: Given the example for *alloca*() at: https://www.gnu.org/software/libc/manual/html_mono/libc.html#Alloca-Example May I recommend that the code below portrays the use of alloca() more completely in that it demonstrates that it is exiting the containing function, not the {} of the first

Re: PING [Patch][Middle-end]Add -fzero-call-used-regs=[skip|used-gpr|all-gpr|used|all]

2020-08-24 Thread Rodriguez Bahena, Victor via Gcc-patches
-Original Message- From: Segher Boessenkool Date: Wednesday, August 19, 2020 at 5:58 PM To: Qing Zhao Cc: Richard Biener , Jeff Law , Uros Bizjak , "H. J. Lu" , Jakub Jelinek , GCC Patches , Kees Cook , "Rodriguez Bahena, Victor" Subject: Re: PING [Patch][Middle-end]Add

[Bug target/96744] [11 Regression] FAIL: gcc.target/i386/avx512bitalgvl-vpopcntb-1.c execution test

2020-08-24 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96744 --- Comment #6 from Uroš Bizjak --- (In reply to Uroš Bizjak from comment #5) > However, the patch assumes that avx512vp2intersect implies mavx512dq, > otherwise there is no direct QImode move from mask register to memory > available. This is

[Bug tree-optimization/96715] Failure to optimize copysign of variable by negated variable

2020-08-24 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96715 Jakub Jelinek changed: What|Removed |Added Last reconfirmed||2020-08-24

[Bug target/96744] [11 Regression] FAIL: gcc.target/i386/avx512bitalgvl-vpopcntb-1.c execution test

2020-08-24 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96744 --- Comment #5 from Uroš Bizjak --- (In reply to Hongtao.liu from comment #2) > Need to add define_insn for movp2qi/movp2hi? Yes, this is needed to cover some corner cases. Please see attachment 49114. However, the patch assumes that

[Bug target/96744] [11 Regression] FAIL: gcc.target/i386/avx512bitalgvl-vpopcntb-1.c execution test

2020-08-24 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96744 --- Comment #4 from Uroš Bizjak --- Created attachment 49114 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=49114=edit Double-reg mask moves

[Bug fortran/96486] get_environment_variable crashes for environment variables that are empty strings

2020-08-24 Thread markeggleston at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96486 markeggleston at gcc dot gnu.org changed: What|Removed |Added Resolution|--- |FIXED

Re: Peephole optimisation: isWhitespace()

2020-08-24 Thread Stefan Kanthak
"Richard Biener" wrote: > On Mon, Aug 24, 2020 at 1:22 PM Stefan Kanthak > wrote: >> >> "Richard Biener" wrote: >> >> > On Mon, Aug 17, 2020 at 7:09 PM Stefan Kanthak >> > wrote: >> >> >> >> "Allan Sandfeld Jensen" wrote: >> >> >> >> > On Freitag, 14. August 2020 18:43:12 CEST Stefan

[Bug fortran/96486] get_environment_variable crashes for environment variables that are empty strings

2020-08-24 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96486 --- Comment #31 from CVS Commits --- The releases/gcc-9 branch has been updated by Mark Eggleston : https://gcc.gnu.org/g:bbe17767c602f1ff08a1520a1d989c6b86b536fd commit r9-8829-gbbe17767c602f1ff08a1520a1d989c6b86b536fd Author: Mark Eggleston

[Bug tree-optimization/96722] [8/9/10/11 Regression] Clobbers on NULL since r8-1519

2020-08-24 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96722 --- Comment #6 from Richard Biener --- I don't have a good idea either but eventually something along the following - if we remove any control stmt in a clobbers control dependence chain force-remove the clobber. Obviously as written it's

c++: overload dumper

2020-08-24 Thread Nathan Sidwell
I frequently need to look at overload sets, and debug_node spews more information than is useful, most of the time. Here's a dumper for overloads, that just tells you their full name and where they came from. gcc/cp * ptree.c (debug_overload): New. pushed -- Nathan Sidwell diff

[Bug target/94538] [9/10/11 Regression] ICE: in extract_constrain_insn_cached, at recog.c:2223 (insn does not satisfy its constraints) with -mcpu=cortex-m23 -mslow-flash-data

2020-08-24 Thread clyon at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94538 --- Comment #21 from Christophe Lyon --- I filed PR96767, PR96768, PR96769, PR96770 to track the enhancements discussed here. The ICE is now fixed in trunk.

[Bug target/96770] New: -mpure-code produces suboptimal code for relocations with small offset for thumb-1

2020-08-24 Thread clyon at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96770 Bug ID: 96770 Summary: -mpure-code produces suboptimal code for relocations with small offset for thumb-1 Product: gcc Version: 10.0 Status: UNCONFIRMED

[Bug target/96769] New: -mpure-code produces suboptimal code for immediate generation for thumb-1

2020-08-24 Thread clyon at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96769 Bug ID: 96769 Summary: -mpure-code produces suboptimal code for immediate generation for thumb-1 Product: gcc Version: 10.0 Status: UNCONFIRMED Severity:

[PATCH] x86: Use -march=x86-64/-march=i386 in

2020-08-24 Thread H.J. Lu via Gcc-patches
On Sun, Aug 23, 2020 at 9:03 AM Uros Bizjak wrote: > > On Sun, Aug 23, 2020 at 5:23 PM H.J. Lu wrote: > > > > On Sun, Aug 23, 2020 at 10:18:28AM +0200, Uros Bizjak wrote: > > > On Sat, Aug 22, 2020 at 9:09 PM H.J. Lu wrote: > > > > > > > > > Compile CPUID check with "-mno-sse -mfpmath=387" to

Re: [PATCH 4/6] Add `+' for Jobserver Integration

2020-08-24 Thread Richard Biener via Gcc-patches
On Fri, Aug 21, 2020 at 12:34 AM Joseph Myers wrote: > > On Thu, 20 Aug 2020, Giuliano Belinassi via Gcc-patches wrote: > > > libbacktrace/Makefile.in | 2 +- > > zlib/Makefile.in | 64 ++-- > > These directories use makefiles generated by automake. Rather than > modifying the

[Bug target/96768] New: -mpure-code produces switch tables for thumb-1

2020-08-24 Thread clyon at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96768 Bug ID: 96768 Summary: -mpure-code produces switch tables for thumb-1 Product: gcc Version: 10.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component:

Re: [PATCH 1/6] Modify gcc driver for parallel compilation

2020-08-24 Thread Richard Biener via Gcc-patches
On Fri, Aug 21, 2020 at 12:00 AM Giuliano Belinassi wrote: > > Update the driver for parallel compilation. This process work as > follows: > > When calling gcc, the driver will check if the flag > "-fparallel-jobs" was provided by the user. If yes, then we will > check what is the desired output,

[Bug target/96767] New: -mpure-code produces indirect loads for thumb-1

2020-08-24 Thread clyon at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96767 Bug ID: 96767 Summary: -mpure-code produces indirect loads for thumb-1 Product: gcc Version: 10.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component:

[Bug tree-optimization/96758] [10/11 Regression] strncmp miscompiles as memcmp since r10-3920-g27c14dbc6b01d5b7

2020-08-24 Thread gcc-bugs at vgkmb dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96758 --- Comment #3 from R. Evans --- Thanks for the quick patch! Applied to head (87c753ac) and confirmed that it passes the test case and fixes the problem with smartmontools-7.1.

[Bug c++/96765] Base class constructor cast to derived should cause a warning

2020-08-24 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96765 Jonathan Wakely changed: What|Removed |Added Last reconfirmed||2020-08-24 Ever confirmed|0

[Bug tree-optimization/96722] [8/9/10/11 Regression] Clobbers on NULL since r8-1519

2020-08-24 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96722 --- Comment #5 from Jakub Jelinek --- On the CDDCE side, I'm afraid I have no idea what can be done. The pass works by not considering clobber stmts necessary, and when about to remove (all) clobbers as unnecessary, it instead attempts to keep

[PATCH] Fix ICE.

2020-08-24 Thread Hongtao Liu via Gcc-patches
Hi: This patch is to fix a typo in my last patch [1]. [1] https://gcc.gnu.org/pipermail/gcc-patches/2020-August/551982.html Bootstrap is ok, gcc regression test hosted on CLX for i386/x86-64 backend is ok. Ok for trunk? gcc/ChangeLog: PR target/96755 * config/i386/sse.md:

[PATCH 5/5] Add --gdwarf-5 to ASM_SPEC

2020-08-24 Thread Mark Wielaard
This is needed to get DWARF version 5 .debug_line tables. It is also obviously wrong. It needs a check for whether as supports --gdwarf- for all versions we support and it should only be added when generating DWARF debug information for the specific DWARF version we are generating. It also needs

[PATCH 4/5] Default to DWARF5

2020-08-24 Thread Mark Wielaard
--- gcc/common.opt | 2 +- gcc/doc/invoke.texi | 5 +++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/gcc/common.opt b/gcc/common.opt index 513125f0c00b..05e4e0638ecb 100644 --- a/gcc/common.opt +++ b/gcc/common.opt @@ -3144,7 +3144,7 @@ Common Driver JoinedOrMissing

[PATCH 2/5] Make sure that static data member constexpr isn't optimized away in test.

2020-08-24 Thread Mark Wielaard
In DWARF5 class variables (static data members) are represented with a DW_TAG_variable instead of a DW_TAG_member. Make sure the variable isn't optimized away in the constexpr-var-1.C testcase so we can still match (2) const_expr in the the assembly output. Note that the same issue causes some

[PATCH 3/5] Add DWARF5 variants of assembly scan tests that use DW_FORM_implicit_const

2020-08-24 Thread Mark Wielaard
Some DWARF tests scan the assembly output looking for constant values. When using DWARF5 those constants might use DW_FORM_implicit_const, which are output (in the comments) after the attribute instead of before. To make sure these tests work introduce a -gdwarf-5 variant of these tests and

[PATCH 1/5] Don't enable -gvariable-location-views by default for DWARF5.

2020-08-24 Thread Mark Wielaard
DWARF5 makes it possible to read loclists tables without consulting the debuginfo tree by introducing a table header. Adding location views breaks this (at least for binutils and elfutils). So don't enable variable-location-views by default if DWARF5 or higher is selected. --- gcc/doc/invoke.texi

BoF DWARF5 patches

2020-08-24 Thread Mark Wielaard
Hi, This afternoon there will be a BoF at the virtual Cauldron about DWARF5 and beyond. https://linuxplumbersconf.org/event/7/contributions/746/ Here are some patches for GCC that I would like to discuss. I'll reply to them after the BoF with any comments people made. Cheers, Mark

Re: [PATCH 0/6] Parallelize Intra-Procedural Optimizations using the LTO Engine.

2020-08-24 Thread Richard Biener via Gcc-patches
On Fri, Aug 21, 2020 at 12:00 AM Giuliano Belinassi wrote: > > This patch series add a new flag "-fparallel-jobs=" to control if the > compiler should try to compile the current file in parallel. > > There are three modes which is supported by now: > > 1. -fparallel-jobs=: Try to compile the file

[Bug libstdc++/96766] std::swap(std::variant, std::variant) triggers undefined behavior sanitizer

2020-08-24 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96766 Jonathan Wakely changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |redi at gcc dot gnu.org

[Bug libstdc++/96766] std::swap(std::variant, std::variant) triggers undefined behavior sanitizer

2020-08-24 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96766 --- Comment #2 from Jonathan Wakely --- This is clang's stupid "unsigned overflow" sanitizer, which complains about correct code. The conversion here is intended, and does exactly the right thing, converting numeric_limits::max() to

[Bug libstdc++/96766] std::swap(std::variant, std::variant) triggers undefined behavior sanitizer

2020-08-24 Thread kndevl at outlook dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96766 --- Comment #1 from Karthik Nishanth --- Reproducer https://www.godbolt.org/z/Whz6ab

Re: Peephole optimisation: isWhitespace()

2020-08-24 Thread Richard Biener via Gcc
On Mon, Aug 24, 2020 at 1:22 PM Stefan Kanthak wrote: > > "Richard Biener" wrote: > > > On Mon, Aug 17, 2020 at 7:09 PM Stefan Kanthak > > wrote: > >> > >> "Allan Sandfeld Jensen" wrote: > >> > >> > On Freitag, 14. August 2020 18:43:12 CEST Stefan Kanthak wrote: > >> >> Hi @ll, > >> >> > >>

[Bug libstdc++/96766] New: std::swap(std::variant, std::variant) triggers undefined behavior sanitizer

2020-08-24 Thread kndevl at outlook dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96766 Bug ID: 96766 Summary: std::swap(std::variant, std::variant) triggers undefined behavior sanitizer Product: gcc Version: 10.2.0 Status: UNCONFIRMED Severity:

[Bug c++/96761] "error: call of overloaded ‘func(size_t)’ is ambiguous" when argument is size_t(0) and func(int) and func(const char *) are visible

2020-08-24 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96761 Jakub Jelinek changed: What|Removed |Added Last reconfirmed||2020-08-24 Ever confirmed|0

[Bug debug/96690] [10/11 Regression] ICE in write_type since r10-6087

2020-08-24 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96690 Richard Biener changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned

[Bug c/96760] Faulty optimization in nested loops with -O2

2020-08-24 Thread zhige.yu18 at imperial dot ac.uk
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96760 --- Comment #3 from Yu Zhige --- (In reply to Thomas Koenig from comment #1) > The loop > > for (a = 20; a; a++) { > > increases a, which is a char, beyond its value range, and then tests > against zero. > > This is undefined behavior. > >

[Bug tree-optimization/96722] [8/9/10/11 Regression] Clobbers on NULL since r8-1519

2020-08-24 Thread rguenther at suse dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96722 --- Comment #4 from rguenther at suse dot de --- On Mon, 24 Aug 2020, jakub at gcc dot gnu.org wrote: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96722 > > --- Comment #3 from Jakub Jelinek --- > Created attachment 49111 > -->

[Bug target/96744] [11 Regression] FAIL: gcc.target/i386/avx512bitalgvl-vpopcntb-1.c execution test

2020-08-24 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96744 --- Comment #3 from Uroš Bizjak --- Created attachment 49112 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=49112=edit Retune mask <-> general moves cost It looks to me that mask <-> general cost is too low, so the compiler now prefers

[Bug c++/96761] "error: call of overloaded ‘func(size_t)’ is ambiguous" when argument is size_t(0) and func(int) and func(const char *) are visible

2020-08-24 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96761 Jakub Jelinek changed: What|Removed |Added CC||jakub at gcc dot gnu.org --- Comment #1

Re: [PATCH] libstdc++: mark variables as possibly unused

2020-08-24 Thread Jonathan Wakely via Gcc-patches
On 24/08/20 13:26 +0200, Krystian Kuźniarek via Libstdc++ wrote: Hi, First of all, sorry, I must have sent it as quoted-printable so spaces and tabs are preserved. A description of the problem/bug and how your patch addresses it: I've got a small patch for -Wunused-variable and

[Bug debug/96690] [10/11 Regression] ICE in write_type since r10-6087

2020-08-24 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96690 --- Comment #6 from Jakub Jelinek --- Without LTO, gen_remaining_tmpl_value_param_die_attribute will try to get it, and will mangle the foo decl, but shortly after will throw it away due to const_ok_for_output failing on it. Your patch makes

[Bug analyzer/96764] [11 Regression] ICE in fold_convert_const_int_from_real, at fold-const.c:2038

2020-08-24 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96764 David Malcolm changed: What|Removed |Added Ever confirmed|0 |1 Status|UNCONFIRMED

[Bug debug/96690] [10/11 Regression] ICE in write_type since r10-6087

2020-08-24 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96690 --- Comment #5 from Richard Biener --- Forcing the mangling via generating garbage RTL early doesn't work since the /* ??? The C++ FE emits debug information for using decls, so putting gcc_unreachable here falls over. See PR31899.

[Bug analyzer/96763] [11 Regression] ICE in get_subregion_within_ctor, at analyzer/store.cc:379 on RANGE_EXPR index

2020-08-24 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96763 David Malcolm changed: What|Removed |Added Summary|[11 Regression] ICE in |[11 Regression] ICE in

[Bug c++/96765] New: Base class constructor cast to derived should cause a warning

2020-08-24 Thread jzwinck at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96765 Bug ID: 96765 Summary: Base class constructor cast to derived should cause a warning Product: gcc Version: 10.2.1 Status: UNCONFIRMED Severity: normal

[Bug analyzer/96764] New: [11 Regression] ICE in fold_convert_const_int_from_real, at fold-const.c:2038

2020-08-24 Thread asolokha at gmx dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96764 Bug ID: 96764 Summary: [11 Regression] ICE in fold_convert_const_int_from_real, at fold-const.c:2038 Product: gcc Version: 11.0 Status: UNCONFIRMED Severity:

Re: [PATCH] libstdc++: remove an ignored qualifier on function return type

2020-08-24 Thread Jonathan Wakely via Gcc-patches
On 24/08/20 13:26 +0200, Krystian Kuźniarek via Libstdc++ wrote: Hi, First of all, sorry, I must have sent it as quoted-printable so spaces and tabs are preserved. A description of the problem/bug and how your patch addresses it: I've got a small patch for -Wignored-qualifiers in system

[Bug fortran/96486] get_environment_variable crashes for environment variables that are empty strings

2020-08-24 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96486 --- Comment #30 from CVS Commits --- The releases/gcc-10 branch has been updated by Mark Eggleston : https://gcc.gnu.org/g:5effbd0733f9a4d42ddae965e4c28701be7811ac commit r10-8658-g5effbd0733f9a4d42ddae965e4c28701be7811ac Author: Mark

[Bug debug/96690] [10/11 Regression] ICE in write_type since r10-6087

2020-08-24 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96690 Richard Biener changed: What|Removed |Added Component|c++ |debug Keywords|

[Bug tree-optimization/96722] [8/9/10/11 Regression] Clobbers on NULL since r8-1519

2020-08-24 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96722 --- Comment #3 from Jakub Jelinek --- Created attachment 49111 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=49111=edit gcc11-pr96722.patch If *0 ={v} {CLOBBER}; is supposed to be a fancy nop, then we should ignore it during path

[PATCH] libstdc++: mark variables as possibly unused

2020-08-24 Thread Krystian Kuźniarek via Gcc-patches
Hi, First of all, sorry, I must have sent it as quoted-printable so spaces and tabs are preserved. A description of the problem/bug and how your patch addresses it: I've got a small patch for -Wunused-variable and -Wunused-parameter in system headers. These are needed either for: 1)

[Bug c++/96721] [11 Regression] pseudo-destructor calls on pointers since r11-2238

2020-08-24 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96721 --- Comment #3 from Jakub Jelinek --- Created attachment 49110 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=49110=edit gcc11-pr96721.patch Untested fix.

[PATCH] libstdc++: fix misleading indentation

2020-08-24 Thread Krystian Kuźniarek via Gcc-patches
Hi, First of all, sorry, I must have sent it as quoted-printable so spaces and tabs are preserved. A description of the problem/bug and how your patch addresses it: I've got a small patch for -Wmisleading-indentation in system headers. Testcases: N/A, it's only a warning. ChangeLog: Sorry,

[PATCH] libstdc++: add parentheses around assignments used as truth values

2020-08-24 Thread Krystian Kuźniarek via Gcc-patches
Hi, A description of the problem/bug and how your patch addresses it: I've got a small patch for -Wparentheses in system headers. Testcases: N/A, it's only a warning. ChangeLog: Sorry, contrib/mklog.py didn't quite work for me. For some reason after instruction in line 129: "diff =

[PATCH] libstdc++: remove unused variables

2020-08-24 Thread Krystian Kuźniarek via Gcc-patches
Hi, A description of the problem/bug and how your patch addresses it: I've got a small patch for -Wunused-variable in system headers. These aren't needed (i.e. no preprocessor conditions are there). Testcases: N/A, it's only a warning. ChangeLog: Sorry, contrib/mklog.py didn't quite work for

[PATCH] libstdc++: remove an ignored qualifier on function return type

2020-08-24 Thread Krystian Kuźniarek via Gcc-patches
Hi, First of all, sorry, I must have sent it as quoted-printable so spaces and tabs are preserved. A description of the problem/bug and how your patch addresses it: I've got a small patch for -Wignored-qualifiers in system headers. Testcases: N/A, it's only a warning. ChangeLog: Sorry,

<    1   2   3   >