[Bug fortran/49565] character(kind=4) is emitted as DW_ATE_unsigned, not DW_ATE_unsigned_char

2024-03-20 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=49565 --- Comment #13 from Tom Tromey --- (In reply to anlauf from comment #12) > After reading this ancient thread, I don't see anything left to do. Closing. GCC still emits <1>: Abbrev Number: 1 (DW_TAG_base_type) DW_AT_byte_size : 4

[Bug libcc1/113977] debug info for alignment of structure is unspecified

2024-02-29 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=113977 --- Comment #13 from Tom Tromey --- This is fixed on trunk now. I think that means it'll be in GCC 14... ? Which maybe I shouldn't have done according to the current status. Anyway, I'm not sure any more how gcc manages bugs, so I don't know if

[Bug rtl-optimization/38534] gcc 4.2.1 and above: No need to save called-saved registers in 'noreturn' function

2024-02-27 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=38534 --- Comment #39 from Tom Tromey --- (In reply to Lukas Grätz from comment #36) > > #2 0x004011d2 in baz (a=a@entry=42, b=b@entry=43, c=c@entry=44, > > d=, > > e=, f= > reading variable: value has been optimized out>, g=48, h=49)

[Bug libcc1/113977] debug info for alignment of structure is unspecified

2024-02-21 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=113977 Tom Tromey changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |tromey at gcc dot gnu.org

[Bug libcc1/113977] debug info for alignment of structure is unspecified

2024-02-20 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=113977 Tom Tromey changed: What|Removed |Added CC||tromey at gcc dot gnu.org --- Comment #9

[Bug debug/8188] DW_AT_containing_type incorrectly emitted

2024-01-28 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=8188 --- Comment #6 from Tom Tromey --- I wanted to mention -- I don't particularly care if this attribute goes away or not (assuming it indeed doesn't negatively affect gdb), but I do dispute the idea that DWARF proscribes which attributes may or may

[Bug debug/8188] DW_AT_containing_type incorrectly emitted

2024-01-28 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=8188 --- Comment #5 from Tom Tromey --- The uses in gdb seem to all be for the old v2 C++ ABI. Removing them might break that code, but OTOH that code is untested, probably already broken, and anyway long since obsolete. Note that Rust+LLVM use this

[Bug debug/99178] Emit .debug_names

2024-01-10 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99178 --- Comment #7 from Tom Tromey --- (In reply to David Blaikie from comment #6) > Ideally that'd be detected by looking at the abbreviation table, rather than > the augmentation string - if parent info is necessary for a usage of the > table,

[Bug debug/99178] Emit .debug_names

2024-01-09 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99178 --- Comment #5 from Tom Tromey --- (In reply to David Blaikie from comment #4) I don't remember filing this bug. At the time maybe I thought it would be worthwhile to have "end to end" .debug_names generation, that is, to try to have the index

[Bug other/9346] make uninstall does not remove all files

2023-09-15 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=9346 Tom Tromey changed: What|Removed |Added CC||tromey at gcc dot gnu.org --- Comment #8

[Bug libffi/67801] error in libffi documentation

2023-07-05 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67801 --- Comment #4 from Tom Tromey --- This was fixed by commit 92456a4e5658e138e2cea79e390e3306b07685b0 Author: H.J. Lu Date: Tue Aug 31 07:14:47 2021 -0700 libffi: Sync with libffi 3.4.2 Merged commit:

[Bug debug/44126] wrong location description for DW_AT_vtable_elem_location

2023-04-02 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=44126 --- Comment #7 from Tom Tromey --- I happened to be looking in this area and I see that gcc still generates the old, incorrect form.

[Bug fortran/49475] [OOP][debugging] Add DWARF info for Fortran's OOP features (extension, member functions)

2023-03-07 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=49475 --- Comment #4 from Tom Tromey --- Note that ifort implemented this and gdb supports that now. See https://sourceware.org/bugzilla/show_bug.cgi?id=22497 for some info.

[Bug c++/108811] New: add enum annotation for switch statements

2023-02-15 Thread tromey at gcc dot gnu.org via Gcc-bugs
Component: c++ Assignee: unassigned at gcc dot gnu.org Reporter: tromey at gcc dot gnu.org Target Milestone: --- In gdb we don't generally want -Wswitch-enum, because there are many switches where it's not appropriate. However, for a subset of switch statements, it is nice to have

[Bug c++/94845] DWARF function name doesn't match demangled name in base type template parameters

2022-10-21 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94845 --- Comment #10 from Tom Tromey --- See also bug #49130 and bug #49537, which we filed when gdb hit these same problems.

[Bug c++/105798] New: Add new -Wshadow for data member

2022-05-31 Thread tromey at gcc dot gnu.org via Gcc-bugs
++ Assignee: unassigned at gcc dot gnu.org Reporter: tromey at gcc dot gnu.org Target Milestone: --- While refactoring gdb -- changing a function to a method -- I accidentally introduced a self-assign, because the function used local variables that had the same name as the class

[Bug debug/100446] GDB has problems reading GCC's debugging info level -g3

2022-04-19 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100446 --- Comment #8 from Tom Tromey --- This behavior can also be affected by the choice of linker, see bug #91239.

[Bug debug/87432] LTO produced debug info makes gdb slow

2022-04-19 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87432 Tom Tromey changed: What|Removed |Added CC||tromey at gcc dot gnu.org --- Comment #2

[Bug debug/91239] gcc generates invalid .debug_macro sections (according to lld folks)

2022-04-19 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91239 --- Comment #3 from Tom Tromey --- Created attachment 52836 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=52836=edit test program I thought I'd upload the sources. You can just untar. Compile with "gcc -g3 -O0 r.cc z.cc -o z" If you

[Bug debug/91239] gcc generates invalid .debug_macro sections (according to lld folks)

2022-04-19 Thread tromey at gcc dot gnu.org via Gcc-bugs
||tromey at gcc dot gnu.org --- Comment #2 from Tom Tromey --- I separately discovered this problem when debugging an apparent gdb slowdown, which I tracked down to a pathological .debug_macro section -- but only when I switched to using 'mold' to link. > So how does a testcase look l

[Bug libcc1/67590] libcc1 cannot find objdump when cross build native

2022-01-09 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67590 Tom Tromey changed: What|Removed |Added CC||tromey at gcc dot gnu.org --- Comment #2

[Bug bootstrap/65763] tm.h: No such file or directory

2022-01-09 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65763 Tom Tromey changed: What|Removed |Added CC||townsend at astro dot wisc.edu --- Comment

[Bug libcc1/64320] Missing config.h during findcomp.cc compilation when srcdir=objdir

2022-01-09 Thread tromey at gcc dot gnu.org via Gcc-bugs
||tromey at gcc dot gnu.org Resolution|--- |DUPLICATE --- Comment #4 from Tom Tromey --- The patch here, that was reported as fixing the problem, was fixed in bug #65763. *** This bug has been marked as a duplicate of bug 65763 ***

[Bug libcc1/63792] libcc1 doesn't built i386 multilib

2022-01-09 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63792 Tom Tromey changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug libcc1/65817] libcc1: ICE: SEGV: c_incomplete_type_error()

2022-01-09 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65817 Tom Tromey changed: What|Removed |Added Resolution|--- |FIXED Status|UNCONFIRMED

[Bug libcc1/67128] Makefile.in, libcc1 and --enable-shared

2022-01-09 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67128 --- Comment #8 from Tom Tromey --- *** Bug 96240 has been marked as a duplicate of this bug. ***

[Bug bootstrap/96240] Error in building gcc-11 with --disable-shared

2022-01-09 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96240 Tom Tromey changed: What|Removed |Added Resolution|--- |DUPLICATE Status|UNCONFIRMED

[Bug libcc1/67128] Makefile.in, libcc1 and --enable-shared

2022-01-09 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67128 Tom Tromey changed: What|Removed |Added CC||skunk at iskunk dot org --- Comment #7

[Bug libcc1/66955] Bootstrap error: libcc1 compiled as shared library despite --disable-shared

2022-01-09 Thread tromey at gcc dot gnu.org via Gcc-bugs
|--- |DUPLICATE CC||tromey at gcc dot gnu.org --- Comment #6 from Tom Tromey --- This is a dup. I think libcc1 has to be built shared. So if you want --disable-shared, also use --disable-libcc1. Maybe libcc1 should disable itself -- something

[Bug libstdc++/96240] Error in building gcc-11 with --disable-shared

2022-01-09 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96240 Tom Tromey changed: What|Removed |Added Resolution|DUPLICATE |--- Status|RESOLVED

[Bug libcc1/67128] Makefile.in, libcc1 and --enable-shared

2022-01-09 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67128 Tom Tromey changed: What|Removed |Added CC||570070308 at qq dot com --- Comment #6

[Bug libcc1/96240] Error in building gcc-11 with --disable-shared

2022-01-09 Thread tromey at gcc dot gnu.org via Gcc-bugs
||tromey at gcc dot gnu.org Status|UNCONFIRMED |RESOLVED --- Comment #1 from Tom Tromey --- I think this is a dup. libcc1 has to be built shared. Maybe it should automatically disable itself, I don't know; but otherwise you can --disable-libcc1 if you

[Bug libcc1/67128] Makefile.in, libcc1 and --enable-shared

2022-01-09 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67128 Tom Tromey changed: What|Removed |Added CC||tromey at gcc dot gnu.org --- Comment #5

[Bug c/94669] libcc1: 4 * minor performance problem

2022-01-05 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94669 --- Comment #8 from Tom Tromey --- (In reply to David Binderman from comment #7) > Could this bug be marked as fixed, then ? Yes, but I don't really know the GCC rules about closing reports any more, so someone else probably ought to handle it.

[Bug c++/79531] bad location when trying to define undeclared method

2021-08-27 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79531 --- Comment #3 from Tom Tromey --- (In reply to Andrew Pinski from comment #2) > Which seems ok, unless I am missing something. Looks good to me too, IMO you could close this bug.

[Bug debug/100446] GDB has problems reading GCC's debugging info level -g3

2021-05-10 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100446 Tom Tromey changed: What|Removed |Added CC||tromey at gcc dot gnu.org --- Comment #6

[Bug preprocessor/100435] oddity in hash table use in libcpp

2021-05-06 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100435 --- Comment #2 from Tom Tromey --- (In reply to Richard Biener from comment #1) > I think it's just an omission and indeed a bug. I can write a patch easily enough, but I don't have a good way to test it.

[Bug preprocessor/100435] New: oddity in hash table use in libcpp

2021-05-05 Thread tromey at gcc dot gnu.org via Gcc-bugs
: preprocessor Assignee: unassigned at gcc dot gnu.org Reporter: tromey at gcc dot gnu.org Target Milestone: --- I noticed that the libcpp hash tables in libcpp/files.c use htab_hash_string, but compare filenames with filename_cmp. This by itself is not a bug, in the sense that it can't

[Bug c++/94845] DWARF function name doesn't match demangled name in base type template parameters

2021-04-22 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94845 --- Comment #8 from Tom Tromey --- (In reply to rob...@ocallahan.org from comment #7) > So gdb reads DW_AT_name "func", parses it, reserializes it to > "func", and uses that? Yeah. (Actually it's even worse than that, because at least one

[Bug c++/94845] DWARF function name doesn't match demangled name in base type template parameters

2021-04-22 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94845 Tom Tromey changed: What|Removed |Added CC||tromey at gcc dot gnu.org --- Comment #6

[Bug debug/99178] New: Emit .debug_names

2021-02-19 Thread tromey at gcc dot gnu.org via Gcc-bugs
: unassigned at gcc dot gnu.org Reporter: tromey at gcc dot gnu.org Target Milestone: --- DWARF 5 includes a new index section, .debug_names. GCC should emit this with -gdwarf-5

[Bug plugins/65817] libcc1: ICE: SEGV: c_incomplete_type_error()

2021-01-24 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65817 Tom Tromey changed: What|Removed |Added CC||tromey at gcc dot gnu.org --- Comment #3

[Bug other/63792] libcc1 doesn't built i386 multilib

2021-01-22 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63792 Tom Tromey changed: What|Removed |Added CC||tromey at gcc dot gnu.org --- Comment #1

[Bug c/94669] libcc1: 4 * minor performance problem

2021-01-22 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94669 Tom Tromey changed: What|Removed |Added CC||tromey at gcc dot gnu.org

[Bug c/47781] warnings from custom printf format specifiers

2020-12-14 Thread tromey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=47781 --- Comment #24 from Tom Tromey --- (In reply to David Crocker from comment #23) > I need this feature too. Instead of waiting several more years for an > all-singing all-dancing solution, PLEASE can we have a simple solution that > allows me to

[Bug fortran/95509] [11 regression] gfortran.dg/spellcheck-operator.f90 fails after r11-875

2020-06-05 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95509 Tom Tromey changed: What|Removed |Added Resolution|--- |FIXED Status|NEW

[Bug fortran/95509] [11 regression] gfortran.dg/spellcheck-operator.f90 fails after r11-875

2020-06-05 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95509 --- Comment #3 from Tom Tromey --- https://gcc.gnu.org/pipermail/gcc-patches/2020-June/547388.html

[Bug fortran/95509] [11 regression] gfortran.dg/spellcheck-operator.f90 fails after r11-875

2020-06-04 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95509 Tom Tromey changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |tromey at gcc dot gnu.org

[Bug c/95379] Don't warn about the universal zero initializer for a structure with the 'designated_init' attribute.

2020-05-28 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95379 --- Comment #7 from Tom Tromey --- (In reply to Asher Gordon from comment #6) > (In reply to Tom Tromey from comment #5) > > Since this warning is intended to work like sparse, introducing > > a divergence here would seem to make the feature

[Bug c/95379] Don't warn about the universal zero initializer for a structure with the 'designated_init' attribute.

2020-05-28 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95379 Tom Tromey changed: What|Removed |Added CC||tromey at gcc dot gnu.org --- Comment #5

[Bug debug/83935] DWARF for a variant part is incorrect

2020-03-12 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83935 Tom Tromey changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug debug/93988] invalid DWARF emitted for complex integer

2020-03-02 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93988 --- Comment #2 from Tom Tromey --- (In reply to Richard Biener from comment #1) > I wonder if there is (or should be) sth like DW_ATE_unsupported ... using > DW_ATE_lo_user is indeed unfortunate but not wrong per-se. Adding > a

[Bug debug/93988] New: invalid DWARF emitted for complex integer

2020-03-01 Thread tromey at gcc dot gnu.org
: debug Assignee: unassigned at gcc dot gnu.org Reporter: tromey at gcc dot gnu.org Target Milestone: --- Consider this test case: _Complex int x = 23i; Compile with -g and examine the resulting DWARF: <1><31>: Abbrev Number: 3 (DW_TAG_base_type) <32>

[Bug c++/93458] ICE using coroutines

2020-01-28 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93458 --- Comment #4 from Tom Tromey --- > BTW, did you include ? > > (FAOD: it would still be broken if you did, but ISTM we might at some point > add a hint that if the traits can't be found, you probably forgot that). The code was exactly as

[Bug c/93472] New: Document extended forms of constant expression

2020-01-27 Thread tromey at gcc dot gnu.org
: c Assignee: unassigned at gcc dot gnu.org Reporter: tromey at gcc dot gnu.org Target Milestone: --- GCC accepts extended forms of constant expression. An example that came up recently was: const int a = 5; const int b = a; IIUC the standard permits the compiler to accept

[Bug c++/93458] ICE using coroutines

2020-01-27 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93458 Tom Tromey changed: What|Removed |Added Keywords||ice-on-invalid-code --- Comment #1 from

[Bug c++/93458] New: ICE using coroutines

2020-01-27 Thread tromey at gcc dot gnu.org
: unassigned at gcc dot gnu.org Reporter: tromey at gcc dot gnu.org Target Milestone: --- I'm using git master gcc from today. I tried a simple coroutine program: int func(int *x) { for (int i = 0; i < 23; ++i) co_yield x[i]; } Compiling causes gcc to ICE: murgatroyd. ./inst

[Bug c/57612] add builtin to assert that expression does not have side effects

2020-01-17 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=57612 --- Comment #4 from Tom Tromey --- (In reply to H. Peter Anvin from comment #2) > I would like to second this request, however, I would like to request that > it issues a warning rather than an error. It can always be promoted to an > error via

[Bug other/61257] configure should check if sys/sdt.h is usable, not just checking the existance of the header

2019-08-12 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61257 --- Comment #6 from Tom Tromey --- (In reply to Eric Gallager from comment #4) > (In reply to Sergei Trofimovich from comment #3) > > (In reply to Sergei Trofimovich from comment #2) > > > Having explicit flags like --enable-systemtap /

[Bug debug/91411] New: Extraneous size & location attributes for members in DWARF

2019-08-09 Thread tromey at gcc dot gnu.org
rmal Priority: P3 Component: debug Assignee: unassigned at gcc dot gnu.org Reporter: tromey at gcc dot gnu.org Target Milestone: --- Consider this test case: struct x { int a : 5; int b : 2; }; struct x x; Compile with -g -c and then examine the DWARF. For x::

[Bug jit/91330] JIT "dir" entry for info is incomplete

2019-08-07 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91330 --- Comment #1 from Tom Tromey --- This is pretty easy to fix in gcc/jit/docs/conf.py: diff --git a/gcc/jit/docs/conf.py b/gcc/jit/docs/conf.py index 3e630db47ef..1224bdcc07d 100644 --- a/gcc/jit/docs/conf.py +++ b/gcc/jit/docs/conf.py @@

[Bug jit/91330] New: JIT "dir" entry for info is incomplete

2019-08-02 Thread tromey at gcc dot gnu.org
onent: jit Assignee: dmalcolm at gcc dot gnu.org Reporter: tromey at gcc dot gnu.org Target Milestone: --- Looking at the "dir" entry for the JIT, I see: murgatroyd. grep jit install/share/info/dir * libgccjit: (libgccjit.info). One line description of project. I

[Bug c++/90526] New: Missing DW_AT_const_value for constexpr field

2019-05-17 Thread tromey at gcc dot gnu.org
: c++ Assignee: unassigned at gcc dot gnu.org Reporter: tromey at gcc dot gnu.org Target Milestone: --- This test case comes from https://sourceware.org/bugzilla/show_bug.cgi?id=20020 Consider: template struct foo { static constexpr bool is_always_lock_free = true; }; int

[Bug tree-optimization/90427] New: missing "sign flipping" optimization

2019-05-10 Thread tromey at gcc dot gnu.org
nent: tree-optimization Assignee: unassigned at gcc dot gnu.org Reporter: tromey at gcc dot gnu.org Target Milestone: --- This test case comes from this blog post: https://nfrechette.github.io/2019/05/08/sign_flip_optimization/ (which also says that clang 8 performs this op

[Bug debug/83935] DWARF for a variant part is incorrect

2019-05-01 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83935 --- Comment #10 from Tom Tromey --- I have been looking at this again recently, for Ada, and now I think perhaps the approach that GCC takes should be preferred. At first I was thinking maybe the compiler could linearize the members of the

[Bug c++/89854] New: diagnostic differences between misuse of "." and "->"

2019-03-27 Thread tromey at gcc dot gnu.org
Priority: P3 Component: c++ Assignee: unassigned at gcc dot gnu.org Reporter: tromey at gcc dot gnu.org Target Milestone: --- I'm using the system gcc on Fedora 29: gcc (GCC) 8.2.1 20180801 (Red Hat 8.2.1-2) Consider this source: struct s { int f; }; int x(stru

[Bug c++/89818] New: possibly invalid name mangling

2019-03-25 Thread tromey at gcc dot gnu.org
++ Assignee: unassigned at gcc dot gnu.org Reporter: tromey at gcc dot gnu.org Target Milestone: --- Apologies for the vagueness of this bug. I ran across a pull request that mentions that gcc will sometimes emit an erroneous 'sr' mangling: https://github.com/gimli-rs/cpp_demangle/pull

[Bug c++/80635] std::optional and bogus -Wmaybe-uninitialized warning

2019-01-15 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80635 --- Comment #14 from Tom Tromey --- (In reply to Jonathan Wakely from comment #8) > Something like __builtin_unreachable() to say "trust me" would be nice, but > I can't think how to do it. How about an attribute that can be attached to the

[Bug c/64862] printf attribute should accept other string types

2019-01-11 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64862 --- Comment #10 from Tom Tromey --- Also I think all the test suite changes never really worked.

[Bug c/64862] printf attribute should accept other string types

2019-01-11 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64862 --- Comment #9 from Tom Tromey --- Created attachment 45413 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=45413=edit ancient patch

[Bug c/64862] printf attribute should accept other string types

2019-01-11 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64862 --- Comment #8 from Tom Tromey --- Sorry about the extreme delay on this. I think my patch has long since bit-rotted, but I can attach it for reference. I believe my assignment situation got sorted out so this should be fine to read and/or copy

[Bug c++/88520] New: improve warning when `struct` is required

2018-12-16 Thread tromey at gcc dot gnu.org
++ Assignee: unassigned at gcc dot gnu.org Reporter: tromey at gcc dot gnu.org Target Milestone: --- Consider this source: int Foo; struct Foo { int a; }; extern int f(Foo x); gcc (I'm using 8.2.1 from Fedora 28) says: q.cc:8:14: warning: ‘f’ initialized and declared

[Bug c/65158] printf attribute error reporting assumes single-byte characters

2018-09-14 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65158 Tom Tromey changed: What|Removed |Added Status|WAITING |REOPENED --- Comment #2 from Tom Tromey

[Bug target/87104] missed &, == optimization makes Emacs ~0.4% slower on x86-64

2018-08-31 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87104 --- Comment #13 from Tom Tromey --- (In reply to pipcet from comment #12) > So I think the performance difference is really significant for Emacs; my > plan is to test all three versions on other programs, make sure the code > works for C

[Bug c++/87062] mis-optimized code with -O3 and std::pair

2018-08-22 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87062 --- Comment #1 from Tom Tromey --- Analysis in the comments there puts the blame on -ftree-slp-vectorize

[Bug c++/87062] New: mis-optimized code with -O3 and std::pair

2018-08-22 Thread tromey at gcc dot gnu.org
++ Assignee: unassigned at gcc dot gnu.org Reporter: tromey at gcc dot gnu.org Target Milestone: --- I'm filing this on behalf of someone who posted this bug on reddit. https://www.reddit.com/r/cpp/comments/99e1ri/interesting_gcc_optimizer_bug/ Copying text from there: Looks

[Bug jit/87003] use nonnull attribute in libgccjit.h

2018-08-22 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87003 --- Comment #2 from Tom Tromey --- I don't really know the best thing to do. I see your point about graceful failure being a useful feature, in cases where the result of some gcc-jit function is passed as an argument to another one. Maybe

[Bug jit/87005] New: gcc_jit_context_get_builtin_function not documented

2018-08-17 Thread tromey at gcc dot gnu.org
Component: jit Assignee: dmalcolm at gcc dot gnu.org Reporter: tromey at gcc dot gnu.org Target Milestone: --- The function gcc_jit_context_get_builtin_function is not documented.

[Bug jit/87004] New: no way to mark a function as noreturn

2018-08-17 Thread tromey at gcc dot gnu.org
Assignee: dmalcolm at gcc dot gnu.org Reporter: tromey at gcc dot gnu.org Target Milestone: --- Currently all blocks must be terminated either with a jump or a return. I think it should also be possible to terminate a block with a call to a noreturn function. But, there is no way

[Bug jit/87003] New: use nonnull attribute in libgccjit.h

2018-08-17 Thread tromey at gcc dot gnu.org
Assignee: dmalcolm at gcc dot gnu.org Reporter: tromey at gcc dot gnu.org Target Milestone: --- Many functions in libgccjit.h take a pointer argument, and it isn't clear which of these can be NULL and which cannot. It would be a bit helpful if the nonnull attribute were applied

[Bug jit/87002] New: allow integers larger than "long"

2018-08-17 Thread tromey at gcc dot gnu.org
onent: jit Assignee: dmalcolm at gcc dot gnu.org Reporter: tromey at gcc dot gnu.org Target Milestone: --- gcc-jit has gcc_jit_context_new_rvalue_from_int and gcc_jit_context_new_rvalue_from_long, but on some platforms long might be 32 bits, but a program could still use int64_t or lon

[Bug bootstrap/84402] [meta] GCC build system: parallelism bottleneck

2018-02-22 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84402 --- Comment #17 from Tom Tromey --- The results in comment #13 seem to be missing some compilations -- I would have expected to see more files from libcpp in there. As it is I only see directives.o and line-map.o.

[Bug debug/83935] DWARF for a variant part is incorrect

2018-02-20 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83935 --- Comment #9 from Tom Tromey --- (In reply to Pierre-Marie de Rodat from comment #8) > Understood, thank you for the notice! As we have to tweak the spec one way > or another for Ada, I suggest indeed we keep the way things are implemented >

[Bug debug/83935] DWARF for a variant part is incorrect

2018-02-19 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83935 --- Comment #7 from Tom Tromey --- For Rust I ended up following the letter of the standard, so I'm going to follow this in the gdb patches as well. That said, gdb can be adapted to work with either approach, so it's not strictly necessary to

[Bug debug/83935] DWARF for a variant part is incorrect

2018-01-22 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83935 --- Comment #3 from Tom Tromey --- (In reply to Pierre-Marie de Rodat from comment #2) > Thinking more about it, the rule that the discriminant entry must be a child > of the variant part entry sounds suspicious to me. TBH this did not make

[Bug ada/83935] New: DWARF for a variant part is incorrect

2018-01-18 Thread tromey at gcc dot gnu.org
Assignee: unassigned at gcc dot gnu.org Reporter: tromey at gcc dot gnu.org Target Milestone: --- Joel Brobecker sent me an Ada test case so that I could see a real-life example of the use of DW_TAG_variant_part (in support of some Rust stuff I'm doing elsewhere). For this test

[Bug c++/61414] enum class bitfield size-checking failure

2017-07-24 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61414 --- Comment #10 from Tom Tromey --- I ran into this again, went to file a bug, and then found that I'd already filed the bug...

[Bug c++/81431] add warning for missing initializers in constructor

2017-07-13 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81431 --- Comment #1 from Tom Tromey --- Also related is bug 55837.

[Bug c++/81431] New: add warning for missing initializers in constructor

2017-07-13 Thread tromey at gcc dot gnu.org
Component: c++ Assignee: unassigned at gcc dot gnu.org Reporter: tromey at gcc dot gnu.org Target Milestone: --- I would like gcc to emit a warning when a constructor does not initialize a POD member; and in particular I'd like this not to be tied to -Wuninitialized. Having

[Bug c++/80712] New: mention the name of the attribute being ignored

2017-05-11 Thread tromey at gcc dot gnu.org
: c++ Assignee: unassigned at gcc dot gnu.org Reporter: tromey at gcc dot gnu.org Target Milestone: --- I saw an error when building firefox with gcc 6.3.1 (fedora 25 system gcc): /home/tromey/firefox-git/gecko/js/src/frontend/EitherParser.h:253:13: error: ignoring attributes

[Bug c++/79531] New: bad location when trying to define undeclared method

2017-02-15 Thread tromey at gcc dot gnu.org
Component: c++ Assignee: unassigned at gcc dot gnu.org Reporter: tromey at gcc dot gnu.org Target Milestone: --- Consider this source: === struct base { virtual void m() = 0; }; struct derived : public base { virtual void m() override; virtual void method2() override

[Bug debug/77315] emit DW_OP_form_tls_address

2016-10-31 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77315 Tom Tromey changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug debug/77315] emit DW_OP_form_tls_address

2016-10-31 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77315 --- Comment #2 from Tom Tromey --- Author: tromey Date: Mon Oct 31 20:08:44 2016 New Revision: 241721 URL: https://gcc.gnu.org/viewcvs?rev=241721=gcc=rev Log: PR debug/77315: * dwarf2out.c (mem_loc_descriptor): Use

[Bug debug/77315] emit DW_OP_form_tls_address

2016-10-20 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77315 Tom Tromey changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |tromey at gcc dot gnu.org

[Bug c++/78014] -Wformat -vs- size_t

2016-10-17 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78014 --- Comment #3 from Tom Tromey --- (In reply to jos...@codesourcery.com from comment #2) > Likewise an expression where the user did "typedef size_t my_size_t;" and > then used my_size_t. And what about expressions resulting from arithmetic

[Bug c++/78014] New: -Wformat -vs- size_t

2016-10-17 Thread tromey at gcc dot gnu.org
: unassigned at gcc dot gnu.org Reporter: tromey at gcc dot gnu.org Target Milestone: --- Compile this test program with -Wformat: #include int main() { size_t x = 0; printf("got %lu\n", x); } I expected this to give a warning, because the correct format should be

[Bug c++/77958] New: printf format checking -vs- variadic template functions

2016-10-12 Thread tromey at gcc dot gnu.org
Priority: P3 Component: c++ Assignee: unassigned at gcc dot gnu.org Reporter: tromey at gcc dot gnu.org Target Milestone: --- Consider this test case, derived from code I found in firefox: #include #include template static void // __attribute__ ((format (printf, 1, 2

[Bug debug/77315] New: emit DW_OP_form_tls_address

2016-08-21 Thread tromey at gcc dot gnu.org
Assignee: unassigned at gcc dot gnu.org Reporter: tromey at gcc dot gnu.org Target Milestone: --- Currently gcc generally emits DW_OP_GNU_push_tls_address for the location of a thread-local. However, it seems to me that the standard DW_OP_form_tls_address is equivalent but slightly

[Bug c++/72789] New: add -Wunused-private-field

2016-08-03 Thread tromey at gcc dot gnu.org
++ Assignee: unassigned at gcc dot gnu.org Reporter: tromey at gcc dot gnu.org Target Milestone: --- I found out recently that clang has a '-Wunused-private-field' warning. E.g., I got: https://github.com/mozilla/rr/pull/1757#issuecomment-237031428 I think it would be good to add this to gcc.

[Bug c/71852] New: add warning for conditions that can never be true

2016-07-12 Thread tromey at gcc dot gnu.org
Component: c Assignee: unassigned at gcc dot gnu.org Reporter: tromey at gcc dot gnu.org Target Milestone: --- Consider this code, derived from some code in gdb's arm-tdep.c: int main(int argc, char**argv) { unsigned int aa = argc; aa = aa & 0x04; if (aa ==

  1   2   3   4   >