[Bug middle-end/19987] [meta-bug] fold missing optimizations in general

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=19987 Bug 19987 depends on bug 94916, which changed state. Bug 94916 Summary: Failure to optimize pattern into difference or zero selector https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94916 What|Removed |Added

[Bug tree-optimization/94916] Failure to optimize pattern into difference or zero selector

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94916 Andrew Pinski changed: What|Removed |Added Target Milestone|--- |13.0 See Also|

[Bug rtl-optimization/3507] appalling optimisation with sub/cmp on multiple targets

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=3507 Andrew Pinski changed: What|Removed |Added Status|ASSIGNED|NEW Assignee|rask at gcc dot

Re: More C type errors by default for GCC 14

2023-05-12 Thread Po Lu via Gcc
Thomas Koenig writes: > C99, 6.7.2, "Type specifiers" > > # Constraints > > # At least one type specifier shall be given in the declaration > # specifiers in each declaration, and in the specifier-qualifier > # list in each struct declaration and type name. And? > In C99 and onwards, this is

[Bug tree-optimization/95408] Failure to optimize bitwise and with negated conditional using the same operand to conditional with decremented operand

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95408 Andrew Pinski changed: What|Removed |Added Ever confirmed|0 |1 Assignee|unassigned at gcc

Re: More C type errors by default for GCC 14

2023-05-12 Thread Thomas Koenig via Gcc
On 13.05.23 02:45, Po Lu via Gcc wrote: Gabriel Ravier writes: ...You're joking, right ? You can't possibly be seriously arguing this, you have to be kidding... right ? No, I'm not. The meaning of a variable declaration with only a storage class specifier is extremely clear: the type of

[Bug c++/106943] GCC building clang/llvm with LTO flags causes ICE in clang

2023-05-12 Thread xry111 at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106943 --- Comment #30 from Xi Ruoyao --- https://reviews.llvm.org/D150505.

[Bug tree-optimization/94893] Sign function not getting optimized to simple compare

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94893 --- Comment #3 from Andrew Pinski --- Note we also don't optimize: inline int sign1(int x) { return (x < 0 ? -1 : 0) | (x > 0 ? 1 : 0); } bool f1(int x) { return sign1(x) < 1; } To be just `x <= 0`

[Bug middle-end/98710] missing optimization (x | c) & ~(y | c) -> x & ~(y | c)

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98710 Andrew Pinski changed: What|Removed |Added Last reconfirmed|2021-04-26 00:00:00 |2023-5-12 --- Comment #2 from Andrew

[Bug middle-end/104069] Wuse-after-free=2 -O0 false positive "may be used"

2023-05-12 Thread mkaracsony81 at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104069 --- Comment #34 from Miklos Karacsony --- (In reply to Sergei Trofimovich from comment #32) > Created attachment 55068 [details] > gzip.c.c > > > > You should be able to extract preprocessed file using > > > https://gcc.gnu.org/bugs/#need: you

[Bug middle-end/104069] Wuse-after-free=2 -O0 false positive "may be used"

2023-05-12 Thread mkaracsony81 at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104069 --- Comment #33 from Miklos Karacsony --- Created attachment 55073 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=55073=edit gcc 12.3.1 preprocessed source file

[Bug tree-optimization/94911] Failure to optimize comparisons of VLA sizes

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94911 Andrew Pinski changed: What|Removed |Added Last reconfirmed||2023-05-13 Ever confirmed|0

Re: [PATCH] libstdc++: use __bool_constant instead of integral_constant

2023-05-12 Thread Ken Matsui via Gcc-patches
It appears that GCC 13 has been released, but I am wondering if there are any issues preventing this patch from being merged yet. Can you provide any information on this? On Sat, Apr 8, 2023 at 2:08 PM Ken Matsui wrote: > > I see. Thank you! > > On Sat, Apr 8, 2023 at 12:52 AM Jonathan Wakely

[Bug c++/106943] GCC building clang/llvm with LTO flags causes ICE in clang

2023-05-12 Thread xry111 at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106943 --- Comment #29 from Xi Ruoyao --- (In reply to Xi Ruoyao from comment #28) > (In reply to Alexander Monakov from comment #21) > > (In reply to Xi Ruoyao from comment #18) > > > Maybe. Should we send a patch? > > > > Yes, if we have a

[Bug c++/106943] GCC building clang/llvm with LTO flags causes ICE in clang

2023-05-12 Thread xry111 at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106943 --- Comment #28 from Xi Ruoyao --- (In reply to Alexander Monakov from comment #21) > (In reply to Xi Ruoyao from comment #18) > > Maybe. Should we send a patch? > > Yes, if we have a volunteer. I'm creating it, but from the description of

Re: [PATCH] RISC-V: Optimize vsetvl AVL for VLS VLMAX auto-vectorization

2023-05-12 Thread Kito Cheng via Gcc-patches
Hmmm here is alternative approach for this: diff --git a/gcc/config/riscv/riscv-v.cc b/gcc/config/riscv/riscv-v.cc index b8dc333f54e1..c88056024e7d 100644 --- a/gcc/config/riscv/riscv-v.cc +++ b/gcc/config/riscv/riscv-v.cc @@ -50,6 +50,21 @@ using namespace riscv_vector; namespace riscv_vector {

[Bug tree-optimization/109834] [14 Regression] internal compiler error: tree check: expected class ‘type’, have ‘exceptional’ (ssa_name) in gimple_simplify_191 when building harfbuzz

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

[Bug tree-optimization/109834] [14 Regression] internal compiler error: tree check: expected class ‘type’, have ‘exceptional’ (ssa_name) in gimple_simplify_191 when building harfbuzz

2023-05-12 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109834 --- Comment #12 from CVS Commits --- The trunk branch has been updated by Andrew Pinski : https://gcc.gnu.org/g:99488a6048745a7b999c22f46e5814d02ebf88d9 commit r14-804-g99488a6048745a7b999c22f46e5814d02ebf88d9 Author: Andrew Pinski Date:

[PATCH] MATCH: Fix PR 109834, ICE with popcount combined with bswap

2023-05-12 Thread Andrew Pinski via Gcc-patches
After r14-673-gc0dd80e4c4c3, there was a check in the match patterns which was checking the type is unsigned but instead of using the type, the patch used the expression. This adds the needed TREE_TYPE so get the correct answer and don't ICE. Committed as obvious after a bootstrap/test on

Re: [PATCH] i386: Allow -mlarge-data-threshold with -mcmodel=large

2023-05-12 Thread Fangrui Song via Gcc-patches
On Wed, May 10, 2023 at 2:58 AM Uros Bizjak wrote: > > On Fri, Apr 28, 2023 at 2:47 AM Fangrui Song wrote: > > > > When using -mcmodel=medium, large data is placed into .l* sections. GNU ld > > places .l* sections into separate output sections. If small and medium > > code model object files

[Bug analyzer/109839] New: -Wanalyzer-fd-leak false positive with routine dup2

2023-05-12 Thread eggert at cs dot ucla.edu via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109839 Bug ID: 109839 Summary: -Wanalyzer-fd-leak false positive with routine dup2 Product: gcc Version: 13.1.1 Status: UNCONFIRMED Severity: normal Priority: P3

[PATCH] RISC-V: Optimize vsetvl AVL for VLS VLMAX auto-vectorization

2023-05-12 Thread juzhe . zhong
From: Juzhe-Zhong This patch is optimizing the AVL for VLS auto-vectorzation. Consider such case: typedef int8_t vnx2qi __attribute__ ((vector_size (2))); __attribute__ ((noipa)) void f_vnx2qi (int8_t a, int8_t b, int8_t *out) { vnx2qi v = {a, b}; *(vnx2qi *) out = v; } Before this

[Bug libstdc++/109818] std::trunc() requires a hack after building DJGPP

2023-05-12 Thread janezz55 at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109818 --- Comment #24 from Janez Zemva --- I'll go libm shopping them, I know just the thing to try out first: https://www.ibiblio.org/pub/micro/pc-stuff/freedos/files/devel/libs/libm/

Re: [committed] Convert xstormy16 to LRA

2023-05-12 Thread Hans-Peter Nilsson via Gcc-patches
> From: Hans-Peter Nilsson > Date: Sat, 13 May 2023 02:56:39 +0200 > > > From: "Roger Sayle" > > Date: Fri, 12 May 2023 15:04:03 +0100 > > > Hi H-P, > > This patch should now already be on trunk: > > https://gcc.gnu.org/git/?p=gcc.git;a=commit;h=d8a6945c6ea22efa4d5e42fe1922d2 > > b27953c8cd >

Re: [committed] Convert xstormy16 to LRA

2023-05-12 Thread Hans-Peter Nilsson via Gcc-patches
> From: "Roger Sayle" > Date: Fri, 12 May 2023 15:04:03 +0100 > Hi H-P, > This patch should now already be on trunk: > https://gcc.gnu.org/git/?p=gcc.git;a=commit;h=d8a6945c6ea22efa4d5e42fe1922d2 > b27953c8cd > Many thanks to Jeff for the review/approval. > There have been no reported adverse

Re: More C type errors by default for GCC 14

2023-05-12 Thread Po Lu via Gcc
Gabriel Ravier writes: > ...You're joking, right ? You can't possibly be seriously arguing > this, you have to be kidding... right ? No, I'm not. The meaning of a variable declaration with only a storage class specifier is extremely clear: the type of the variable is `int'. There's absolutely

Re: Re: [PATCH V4] RISC-V: Using merge approach to optimize repeating sequence in vec_init

2023-05-12 Thread 钟居哲
Address comments. V5 patch: https://gcc.gnu.org/pipermail/gcc-patches/2023-May/618398.html Thanks. juzhe.zh...@rivai.ai From: Kito Cheng Date: 2023-05-13 00:16 To: juzhe.zhong CC: gcc-patches; palmer; jeffreyalaw Subject: Re: [PATCH V4] RISC-V: Using merge approach to optimize repeating

[PATCH V5] RISC-V: Using merge approach to optimize repeating sequence in vec_init

2023-05-12 Thread juzhe . zhong
From: Juzhe-Zhong 1. Remove magic number of V4 2. Remove unnecessary gcc_assert Consider this following case: typedef int64_t vnx32di __attribute__ ((vector_size (256))); __attribute__ ((noipa)) void f_vnx32di (int64_t a, int64_t b, int64_t *out) { vnx32di v = {a, b, a, b, a, b, a, b,

[Bug target/109825] [14 Regression] ICE in ix86_widen_mult_cost, at config/i386/i386.cc:20442 since r14-666-g608e7f3ab47fe7

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109825 Andrew Pinski changed: What|Removed |Added Status|UNCONFIRMED |NEW Ever confirmed|0

[Bug middle-end/109838] [14 Regression] ICE on libaom-3.6.0: in ix86_widen_mult_cost, at config/i386/i386.cc:20444

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

[Bug target/109825] [14 Regression] ICE in ix86_widen_mult_cost, at config/i386/i386.cc:20442 since r14-666-g608e7f3ab47fe7

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109825 Andrew Pinski changed: What|Removed |Added CC||slyfox at gcc dot gnu.org --- Comment

[Bug middle-end/109838] New: [14 Regression] ICE on libaom-3.6.0: in ix86_widen_mult_cost, at config/i386/i386.cc:20444

2023-05-12 Thread slyfox at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109838 Bug ID: 109838 Summary: [14 Regression] ICE on libaom-3.6.0: in ix86_widen_mult_cost, at config/i386/i386.cc:20444 Product: gcc Version: 14.0 Status: UNCONFIRMED

[Bug libstdc++/109818] std::trunc() requires a hack after building DJGPP

2023-05-12 Thread redi at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109818 --- Comment #23 from Jonathan Wakely --- No, the built-in functions just call the functions defined in libm. GCC doesn't implement them. Look at the code for a call to __builtin_acosh: https://godbolt.org/z/dPf46sKxx

[Bug c++/109824] aligned attribute lost when using it on a typedef of a struct if there was not an usage of the struct beforehand

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109824 Andrew Pinski changed: What|Removed |Added Ever confirmed|0 |1 Status|UNCONFIRMED

[Bug libstdc++/109818] std::trunc() requires a hack after building DJGPP

2023-05-12 Thread janezz55 at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109818 --- Comment #22 from Janez Zemva --- They are very sloppy, so I doubt even what they provide is working 100%. This is why I suggested the generative approach. gcc has many built-in functions, surely a rudimentary could be cobbled out of them.

[Bug c++/109824] aligned attribute lost when using it on a typedef of a struct if there was not an usage of the struct beforehand

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109824 Andrew Pinski changed: What|Removed |Added Summary|aligned attribute lost on |aligned attribute lost when

[PATCH] aarch64: Add SVE instruction types

2023-05-12 Thread Evandro Menezes via Gcc-patches
This patch adds the attribute `type` to most SVE1 instructions, as in the other instructions. -- Evandro Menezes 0002-aarch64-Add-SVE-instruction-types.patch Description: Binary data

[Bug libgomp/109837] New: [OpenMP] despite 'requires unified_address' there is segfault when 'is_device_ptr' is not used

2023-05-12 Thread burnus at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109837 Bug ID: 109837 Summary: [OpenMP] despite 'requires unified_address' there is segfault when 'is_device_ptr' is not used Product: gcc Version: 13.0 Status: UNCONFIRMED

[Bug c++/109824] aligned attribute lost on first usage

2023-05-12 Thread stsp at users dot sourceforge.net via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109824 Stas Sergeev changed: What|Removed |Added Status|RESOLVED|UNCONFIRMED

[Bug tree-optimization/109834] [14 Regression] internal compiler error: tree check: expected class ‘type’, have ‘exceptional’ (ssa_name) in gimple_simplify_191 when building harfbuzz

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109834 Andrew Pinski changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned

Re: Re: [PATCH V4] RISC-V: Using merge approach to optimize repeating sequence in vec_init

2023-05-12 Thread 钟居哲
>> What the magic 64 means? uint64_t mask = 0; 64 = sizeof (uint64_t) >> gcc_assert will removed at release mode, so it's not you want I guess? You mean I need to remove it? juzhe.zh...@rivai.ai From: Kito Cheng Date: 2023-05-13 00:16 To: juzhe.zhong CC: gcc-patches; palmer; jeffreyalaw

gcc-12-20230512 is now available

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

[Bug libstdc++/109814] [13/14 Regression] freestanding using cmath does not compile anymore

2023-05-12 Thread arsen at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109814 --- Comment #10 from Arsen Arsenović --- (In reply to lesto fante from comment #9) > To be fair I have no idea what would be the impact of removing freestanding, > from a quick test does not seems to have a realistic impact. > > I guess what

[Bug tree-optimization/109834] [14 Regression] internal compiler error: tree check: expected class ‘type’, have ‘exceptional’ (ssa_name) in gimple_simplify_191 when building harfbuzz

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109834 Andrew Pinski changed: What|Removed |Added Last reconfirmed||2023-05-12 Ever confirmed|0

[Bug tree-optimization/109834] [14 Regression] internal compiler error: tree check: expected class ‘type’, have ‘exceptional’ (ssa_name) in gimple_simplify_191 when building harfbuzz

2023-05-12 Thread sjames at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109834 --- Comment #9 from Sam James --- (In reply to Sam James from comment #8) > Created attachment 55071 [details] > graphite2-GlyphCache.cpp.ii (maybe related) I guess this is really the same given the operations it's doing (its own bswaps).

[Bug tree-optimization/109834] [14 Regression] internal compiler error: tree check: expected class ‘type’, have ‘exceptional’ (ssa_name) in gimple_simplify_191 when building harfbuzz

2023-05-12 Thread sjames at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109834 --- Comment #8 from Sam James --- Created attachment 55071 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=55071=edit graphite2-GlyphCache.cpp.ii (maybe related)

[Bug libstdc++/109818] std::trunc() requires a hack after building DJGPP

2023-05-12 Thread redi at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109818 --- Comment #21 from Jonathan Wakely --- Oh, checking FLT_EVAL_METHOD doesn't work, because that's defined in not . So another djgpp-specific "NO_FLOAT_TYPES" macro will be needed. Sigh.

Re: More C type errors by default for GCC 14

2023-05-12 Thread Florian Weimer
* Sam James: > Florian Weimer writes: > >> * Sam James: >> >>> Florian Weimer writes: >>> [...] In summary, all these seems to be good candidates for errors by default: * int-conversion as errors (already raised separately * -Wint-conversion for ?: * parameter

[Bug libstdc++/109818] std::trunc() requires a hack after building DJGPP

2023-05-12 Thread redi at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109818 --- Comment #20 from Jonathan Wakely --- Created attachment 55070 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=55070=edit Patch to make cmath configure tests more granular Something like this might work, but it's horrible.

Re: [PATCH] i386: Honour -mdirect-extern-access when calling __fentry__

2023-05-12 Thread Ard Biesheuvel via Gcc-patches
On Fri, 12 May 2023 at 19:05, Uros Bizjak wrote: > > On Fri, May 12, 2023 at 4:07 PM Ard Biesheuvel wrote: > > > > > > > Note that the GOT reference in question is in fact a data > > > > > > reference: we > > > > > > explicitly load the address of __fentry__ from the GOT, which > > > > > >

Re: More C type errors by default for GCC 14

2023-05-12 Thread Sam James via Gcc
Florian Weimer writes: > * Sam James: > >> Florian Weimer writes: >> >>> [...] >>> In summary, all these seems to be good candidates for errors by default: >>> >>> * int-conversion as errors (already raised separately >>> * -Wint-conversion for ?: >>> * parameter names in non-prototype

[Bug c/59905] Unfriendly abort when calling a fucntion via a function pointer cast

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59905 Andrew Pinski changed: What|Removed |Added Target Milestone|--- |4.9.0

Re: More C type errors by default for GCC 14

2023-05-12 Thread Florian Weimer
* Sam James: > Florian Weimer writes: > >> [...] >> In summary, all these seems to be good candidates for errors by default: >> >> * int-conversion as errors (already raised separately >> * -Wint-conversion for ?: >> * parameter names in non-prototype function declarations >> * the union wait

Re: More C type errors by default for GCC 14

2023-05-12 Thread Sam James via Gcc
Florian Weimer writes: > [...] > In summary, all these seems to be good candidates for errors by default: > > * int-conversion as errors (already raised separately > * -Wint-conversion for ?: > * parameter names in non-prototype function declarations > * the union wait function pointer

[Bug libstdc++/109818] std::trunc() requires a hack after building DJGPP

2023-05-12 Thread redi at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109818 --- Comment #19 from Jonathan Wakely --- (In reply to Jonathan Wakely from comment #15) > A proper fix would be to split up the configure test for USE_C99_MATH_TR1 to > be more fine-grained, so that we use the subset of functions that are >

Re: More C type errors by default for GCC 14

2023-05-12 Thread Sam James via Gcc
Florian Weimer writes: > * Jason Merrill: > >> On Fri, May 12, 2023 at 11:03 AM Florian Weimer wrote: >>> >>> * Joseph Myers: >>> >>> > On Wed, 10 May 2023, Eli Zaretskii via Gcc wrote: >>> > >>> >> That is not the case we are discussing, AFAIU. Or at least no one has >>> >> yet explained why

[Bug c++/109745] [13 Regression] Incorrect code generated with -O1 when having a constexpr object modifying a mutable member

2023-05-12 Thread carlosgalvezp at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109745 --- Comment #8 from Carlos Galvez --- Thanks a lot for the quick fix!

[Bug c/109835] -Wincompatible-function-pointer-types as a subset of -Wincompatible-pointer-types?

2023-05-12 Thread sjames at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109835 --- Comment #2 from Sam James --- Okay, fair point, I gave examples but not *motivating* examples. I have some non-harmless examples: 1. https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=1e0e5c4d289004fa779c86da9319cf2bb18548b1 (a nasty

[Bug tree-optimization/109834] [14 Regression] internal compiler error: tree check: expected class ‘type’, have ‘exceptional’ (ssa_name) in gimple_simplify_191 when building harfbuzz

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109834 --- Comment #7 from Andrew Pinski --- Reduced further: int split_subtables(char v) { return __builtin_popcount((int)__builtin_bswap16(v)); }

[Bug tree-optimization/109834] [14 Regression] internal compiler error: tree check: expected class ‘type’, have ‘exceptional’ (ssa_name) in gimple_simplify_191 when building harfbuzz

2023-05-12 Thread sjames at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109834 --- Comment #6 from Sam James --- Created attachment 55069 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=55069=edit reduced.ii I might reduce the graphite2 one for fun just to see how different it is but no promises.

Re: More C type errors by default for GCC 14

2023-05-12 Thread Florian Weimer
* Jason Merrill: > On Fri, May 12, 2023 at 11:03 AM Florian Weimer wrote: >> >> * Joseph Myers: >> >> > On Wed, 10 May 2023, Eli Zaretskii via Gcc wrote: >> > >> >> That is not the case we are discussing, AFAIU. Or at least no one has >> >> yet explained why accepting those old K programs will

[PATCH RFC] c-family: make -fno-permissive upgrade pedwarns

2023-05-12 Thread Jason Merrill via Gcc-patches
In the context of the recent discussion, it occurred to me that this semantic would be useful, but currently there is no easy way to access it. Bikeshedding welcome; the use of this flag is a bit odd, but it has the advantage of being accepted without error going back at least to 4.3. -- 8< --

[Bug tree-optimization/109834] [14 Regression] internal compiler error: tree check: expected class ‘type’, have ‘exceptional’ (ssa_name) in gimple_simplify_191 when building harfbuzz

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109834 --- Comment #5 from Andrew Pinski --- (In reply to Mikael Morin from comment #4) > (In reply to Andrew Pinski from comment #2) > > I am 99% sure it was introduced by r14-673-g5fdcfe3c5776 > > > I think the correct link is r14-673-gc0dd80e4c4c3

[Bug c++/109654] unnecessary "cannot bind packed field to reference" error when referenced type has aligned(1) attribute

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109654 Andrew Pinski changed: What|Removed |Added CC||stsp at users dot sourceforge.net ---

[Bug c++/109824] aligned attribute lost on first usage

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109824 Andrew Pinski changed: What|Removed |Added Resolution|--- |DUPLICATE See Also|

[Bug tree-optimization/109834] [14 Regression] internal compiler error: tree check: expected class ‘type’, have ‘exceptional’ (ssa_name) in gimple_simplify_191 when building harfbuzz

2023-05-12 Thread mikael at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109834 Mikael Morin changed: What|Removed |Added CC||mikael at gcc dot gnu.org --- Comment

Re: More C type errors by default for GCC 14

2023-05-12 Thread Florian Weimer
* Joseph Myers: > On Fri, 12 May 2023, Florian Weimer wrote: > >> This sone seems to be a good candidate for additional errors, though: >> >> warned_here = pedwarn >> (loc, warn_return_type >= 0 ? OPT_Wreturn_type : 0, >> "% with no value, in function returning non-void"); >> >> It's

[Bug c/109828] [13/14 Regression] C2x:static compound literal (with flexible array) in initializer leads to invalid size and ICE

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109828 --- Comment #6 from Andrew Pinski --- here is another example where we output a bogus `.zero` (though it does not ICE): struct s { int i; char c[]; }; const struct s *t = &(struct s) { .c = {'2','\0'}, }; We get: .size

[Bug libfortran/109662] bad namelist input but gfortran accepted it

2023-05-12 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109662 --- Comment #21 from CVS Commits --- The master branch has been updated by Jerry DeLisle : https://gcc.gnu.org/g:96cc09dea48b562a0fc93d43fb3b702ac20b89fd commit r14-802-g96cc09dea48b562a0fc93d43fb3b702ac20b89fd Author: Jerry DeLisle Date:

[Bug libfortran/109662] bad namelist input but gfortran accepted it

2023-05-12 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109662 --- Comment #20 from CVS Commits --- The master branch has been updated by Jerry DeLisle : https://gcc.gnu.org/g:cf3b032b8fb681516ccacbe3689f1cad43a1773a commit r14-801-gcf3b032b8fb681516ccacbe3689f1cad43a1773a Author: Jerry DeLisle Date:

[Bug analyzer/109577] -Wanalyzer-allocation-size mishandles __builtin_mul_overflow

2023-05-12 Thread eggert at cs dot ucla.edu via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109577 Paul Eggert changed: What|Removed |Added CC||eggert at cs dot ucla.edu --- Comment #1

Re: [patch, fortran] PR109662 Namelist input with comma after name accepted

2023-05-12 Thread Jerry D via Gcc-patches
I plan to commit the following as simple. The issue was a value was being modified on a short namelist read. After tthe first read gives the correct EOF, a second read would give the error but modify the variable. diff --git a/libgfortran/io/unit.c b/libgfortran/io/unit.c index

[Bug middle-end/104069] Wuse-after-free=2 -O0 false positive "may be used"

2023-05-12 Thread slyfox at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104069 --- Comment #32 from Sergei Trofimovich --- Created attachment 55068 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=55068=edit gzip.c.c > > You should be able to extract preprocessed file using > > https://gcc.gnu.org/bugs/#need: you

RE: [EXTERNAL] Re: [PATCH] Fixes and workarounds for warnings during autoprofiledbootstrap build

2023-05-12 Thread Eugene Rozenfeld via Gcc-patches
Thank you, Richard. I went with your suggestion. New patch: [PATCH] Disable warnings as errors for STAGEautofeedback. Compilation during STAGEautofeedback produces additional warnings since inlining decisions with -fauto-profile are different from other builds. This patches disables warnings

[Bug ada/106168] Errors with empty array aggregate.

2023-05-12 Thread ebotcazou at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106168 Eric Botcazou changed: What|Removed |Added Status|REOPENED|SUSPENDED --- Comment #5 from Eric

Re: More C type errors by default for GCC 14

2023-05-12 Thread Jason Merrill via Gcc
On Fri, May 12, 2023 at 11:03 AM Florian Weimer wrote: > > * Joseph Myers: > > > On Wed, 10 May 2023, Eli Zaretskii via Gcc wrote: > > > >> That is not the case we are discussing, AFAIU. Or at least no one has > >> yet explained why accepting those old K programs will adversely > >> affect the

[Bug tree-optimization/109829] Optimizing __builtin_signbit(x) ? -x : x or abs for FP

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109829 --- Comment #6 from Andrew Pinski --- (In reply to Jakub Jelinek from comment #4) > Or does match.pd try to invert all the COND_EXPR conditions and swap the > operands? match does not try but phiopt does in gimple_simplify_phiopt . Or even

[pushed] wwwdocs: gcc-13/buildstat: Remove trace of XHTML

2023-05-12 Thread Gerald Pfeifer
Hi Jakub, do you recall how this snuck in? None of other other pages has had <..." /> instead of <..."> for a while. Not a biggie at all, just curious. Pushed. On a related note, the buildstat pages for GCC 9, 10, 11, 12, and 13 all are empty and I suggest to remove them. Any concerns?

[Bug libstdc++/109814] [13/14 Regression] freestanding using cmath does not compile anymore

2023-05-12 Thread lestofante88 at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109814 --- Comment #9 from lesto fante --- To be fair I have no idea what would be the impact of removing freestanding, from a quick test does not seems to have a realistic impact. I guess what happen here is that the freestanding option should

[Bug tree-optimization/109829] Optimizing __builtin_signbit(x) ? -x : x or abs for FP

2023-05-12 Thread jakub at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109829 --- Comment #5 from Jakub Jelinek --- Or canonicalize COND_EXPRs such that only ne and not eq appears in condition (and similarly to other comparisons)?

[Bug tree-optimization/109829] Optimizing __builtin_signbit(x) ? -x : x or abs for FP

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

[Bug bootstrap/106472] No rule to make target '../libbacktrace/libbacktrace.la', needed by 'libgo.la'.

2023-05-12 Thread matoro_gcc_bugzilla at matoro dot tk via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106472 --- Comment #26 from matoro --- We also had somebody report on IRC that they observed this on powerpc (not sure what tuple), again with -j1. It does not seem to show up with -j2, so likely -j1 is necessary to trigger.

[Bug bootstrap/106472] No rule to make target '../libbacktrace/libbacktrace.la', needed by 'libgo.la'.

2023-05-12 Thread matoro_gcc_bugzilla at matoro dot tk via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106472 --- Comment #25 from matoro --- Created attachment 55067 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=55067=edit build.log for gcc-13.1 This bug is still present on 13.1.0, attaching log

Re: More C type errors by default for GCC 14

2023-05-12 Thread Joseph Myers
On Fri, 12 May 2023, Florian Weimer wrote: > This sone seems to be a good candidate for additional errors, though: > > warned_here = pedwarn > (loc, warn_return_type >= 0 ? OPT_Wreturn_type : 0, > "% with no value, in function returning non-void"); > > It's a clear type volation that

[Bug fortran/109788] [14 Regression] gcc/hwint.h:293:61: runtime error: shift exponent 64 is too large for 64-bit type 'long unsigned int since r14-377-gc92b8be9b52b7e

2023-05-12 Thread anlauf at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109788 --- Comment #12 from anlauf at gcc dot gnu.org --- (In reply to Jakub Jelinek from comment #11) > If you e.g. put breakpoint on the spot I changed and stopon the testcase > with -Os when t == global_trees[TI_VOID_LIST_NODE] you can then see in >

[Bug c/109826] Incompatible pointer types in ?: not covered by -Wincompatible-pointer-types

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109826 --- Comment #2 from Andrew Pinski --- The warning is not even controlled by an option either so only -Werror turns it into an error.

[Bug tree-optimization/109829] Optimizing __builtin_signbit(x) ? -x : x or abs for FP

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109829 --- Comment #3 from Andrew Pinski --- Should add this one too: (simplify (cond (ne (SIGNBIT @0) zero_p@1) @0 (neg @0)) (neg (abs @0)))

[Bug c++/106943] GCC building clang/llvm with LTO flags causes ICE in clang

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106943 --- Comment #27 from Andrew Pinski --- (In reply to Alexander Monakov from comment #26) > Would that help? GCC raises its own stack limit to 64MB: > > gcc.cc: stack_limit_increase (64 * 1024 * 1024); > toplev.cc: stack_limit_increase (64 *

[Bug c++/106943] GCC building clang/llvm with LTO flags causes ICE in clang

2023-05-12 Thread amonakov at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106943 --- Comment #26 from Alexander Monakov --- Would that help? GCC raises its own stack limit to 64MB: gcc.cc: stack_limit_increase (64 * 1024 * 1024); toplev.cc: stack_limit_increase (64 * 1024 * 1024);

[Bug ada/106168] Errors with empty array aggregate.

2023-05-12 Thread p.p11 at orange dot fr via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106168 Pascal Pignard changed: What|Removed |Added Status|RESOLVED|REOPENED Resolution|FIXED

[Bug tree-optimization/109829] Optimizing __builtin_signbit(x) ? -x : x or abs for FP

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109829 Andrew Pinski changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned

Re: [PATCH] ipa: Self-DCE of uses of removed call LHSs (PR 108007)

2023-05-12 Thread Bernhard Reutner-Fischer via Gcc-patches
On 12 May 2023 14:45:14 CEST, Martin Jambor wrote: >gcc/ChangeLog: > >2023-05-11 Martin Jambor > > PR ipa/108007 > * cgraph.h (cgraph_edge): Add a parameter to > redirect_call_stmt_to_callee. > * ipa-param-manipulation.h (ipa_param_adjustments): Added a >

[Bug c++/106943] GCC building clang/llvm with LTO flags causes ICE in clang

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106943 --- Comment #25 from Andrew Pinski --- (In reply to Alexander Monakov from comment #24) > Appreciate the advice. So far I've managed to reduce the number of LTO > inputs down to two files, RegisterBankInfo.cpp.o plus APInt.cpp.o. I also > built

[Bug c/109828] [13/14 Regression] C2x:static compound literal (with flexible array) in initializer leads to invalid size and ICE

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109828 --- Comment #5 from Andrew Pinski --- (In reply to Yann Droneaud from comment #4) > I'm still playing with this, for example https://godbolt.org/z/dfjr8veh5, > and I've noticed the size of the compound_initializer is incorrect too: > Maybe it's

Re: [PATCH] Machine_Mode: Extend machine_mode from 8 to 16 bits

2023-05-12 Thread Bernhard Reutner-Fischer via Gcc-patches
On 12 May 2023 08:49:53 CEST, Richard Biener via Gcc-patches wrote: >> gcc/ChangeLog: >> >> * combine.cc (struct reg_stat_type): Extended machine mode to 16 bits. >> * cse.cc (struct qty_table_elem): Ditto. >> (struct table_elt): Ditto. >> (struct set): Ditto. >> *

[Bug c/109828] [13/14 Regression] C2x:static compound literal (with flexible array) in initializer leads to invalid size and ICE

2023-05-12 Thread yann at droneaud dot fr via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109828 --- Comment #4 from Yann Droneaud --- I'm still playing with this, for example https://godbolt.org/z/dfjr8veh5, and I've noticed the size of the compound_initializer is incorrect too: struct s { char i; char c[]; }; const struct s

[Bug c++/106943] GCC building clang/llvm with LTO flags causes ICE in clang

2023-05-12 Thread amonakov at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106943 --- Comment #24 from Alexander Monakov --- Appreciate the advice. So far I've managed to reduce the number of LTO inputs down to two files, RegisterBankInfo.cpp.o plus APInt.cpp.o. I also built gcc-12.3 with lineinfo and have a better

[Bug c/109828] [13/14 Regression] C2x:static compound literal (with flexible array) in initializer leads to invalid size and ICE

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109828 Andrew Pinski changed: What|Removed |Added Ever confirmed|0 |1 Summary|C2x:static

[Bug c/23087] Misleading warning, "... differ in signedness" with the character types

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=23087 Andrew Pinski changed: What|Removed |Added Status|REOPENED|NEW Summary|Misleading

[Bug c/23087] Misleading warning, "... differ in signedness"

2023-05-12 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=23087 Andrew Pinski changed: What|Removed |Added CC||ideasman42 at gmail dot com --- Comment

  1   2   3   4   5   >