[Bug c/114526] ISO C does not prohibit extensions: fix misconception.

2024-04-02 Thread kkylheku at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114526 --- Comment #19 from Kaz Kylheku --- (In reply to Harald van Dijk from comment #18) > (In reply to Kaz Kylheku from comment #17) > > The standrad does not define the conversion at the *type* level. > > ... > > The program is strictly conforming

[Bug c/114526] ISO C does not prohibit extensions: fix misconception.

2024-04-02 Thread kkylheku at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114526 --- Comment #17 from Kaz Kylheku --- (In reply to Harald van Dijk from comment #14) > (In reply to Joseph S. Myers from comment #11) > > I think that simply failing to say whether a value of type X may be > > converted to type Y is clearly

[Bug c/114526] ISO C does not prohibit extensions: fix misconception.

2024-04-02 Thread kkylheku at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114526 --- Comment #13 from Kaz Kylheku --- (In reply to Joseph S. Myers from comment #11) > I think that simply failing to say whether a value of type X may be > converted to type Y is clearly enough for it at least to be unspecified > whether or

[Bug c/114526] ISO C does not prohibit extensions: fix misconception.

2024-04-02 Thread kkylheku at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114526 --- Comment #12 from Kaz Kylheku --- (In reply to Harald van Dijk from comment #10) > Sorry, sent my earlier comment too soon. > > (In reply to Joseph S. Myers from comment #8) > > I believe conversions between function and object pointers are

[Bug c/114526] ISO C does not prohibit extensions: fix misconception.

2024-03-28 Thread kkylheku at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114526 --- Comment #7 from Kaz Kylheku --- Also, it would be useful for the documentation to list all the -W-* options that are implied by -Wpedantic. The function/object pointer conversion diagnostics, unfortunately, are tied to -Wpedantic itself,

[Bug c/83584] "ISO C forbids conversion of object pointer to function pointer type" -- no, not really

2024-03-28 Thread kkylheku at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83584 Kaz Kylheku changed: What|Removed |Added CC||kkylheku at gmail dot com --- Comment #22

[Bug c/114526] ISO C does not prohibit extensions: fix misconception.

2024-03-28 Thread kkylheku at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114526 --- Comment #3 from Kaz Kylheku --- (In reply to Andrew Pinski from comment #2) > Actually it is a required diagnostic. See PR 11234 for explanation on how. > This was changed a little over 20 years ago explictly to reject this because > it is

[Bug c/114526] New: ISO C does not prohibit extensions: fix misconception.

2024-03-28 Thread kkylheku at gmail dot com via Gcc-bugs
Component: c Assignee: unassigned at gcc dot gnu.org Reporter: kkylheku at gmail dot com Target Milestone: --- There seems to be a historic misconception in the development of GCC that the C standard prohibits extensions. The -Wpedantic/-pedantic options are unfortunately

[Bug c++/36587] Feature: add warning for constructor call with discarded return.

2024-03-18 Thread kkylheku at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=36587 --- Comment #19 from Kaz Kylheku --- (In reply to Jonathan Wakely from comment #18) > Was there an earlier submission? No there wasn't; that's my mistake in my comment.

[Bug c++/36587] Feature: add warning for constructor call with discarded return.

2024-03-17 Thread kkylheku at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=36587 --- Comment #16 from Kaz Kylheku --- (In reply to Andrew Pinski from comment #14) > C++17 adds nodiscard attribute which can be placed on class/struct types, > functions, constructors and others and then you get a warning if you ignore > the

[Bug c++/36587] Feature: add warning for constructor call with discarded return.

2024-03-17 Thread kkylheku at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=36587 --- Comment #15 from Kaz Kylheku --- (In reply to Manuel López-Ibáñez from comment #13) > (In reply to Kaz Kylheku from comment #11) > > I deployed that change to large team of developers, and the toolchain with > > that change went to customers

[Bug preprocessor/44317] ,##__VA_ARGS__ comma not eaten with -std=c++0x

2022-04-21 Thread kkylheku at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=44317 Kaz Kylheku changed: What|Removed |Added CC||kkylheku at gmail dot com --- Comment #12

[Bug fortran/90329] Incompatibility between gfortran and C lapack calls

2019-06-22 Thread kkylheku at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90329 --- Comment #48 from Kaz Kylheku --- (In reply to Thomas Koenig from comment #47) > I see two problems with this suggestion, one minor and one major. > > First, there may well be a value > 1 on the stack for a regular > call, see comment #15.

[Bug fortran/90329] Incompatibility between gfortran and C lapack calls

2019-06-22 Thread kkylheku at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90329 --- Comment #46 from Kaz Kylheku --- C pseudocode in light of previous comment: double abused_fortran_fn(double x, double y, char str[1], int len) { if (len != 1) return abused_fortran_fn(x, y, str, 1); /* full call, not

[Bug fortran/90329] Incompatibility between gfortran and C lapack calls

2019-06-22 Thread kkylheku at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90329 Kaz Kylheku changed: What|Removed |Added CC||kkylheku at gmail dot com --- Comment #45

[Bug inline-asm/87733] local register variable not honored with earlyclobber

2019-03-15 Thread kkylheku at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87733 Kaz Kylheku changed: What|Removed |Added CC||kkylheku at gmail dot com --- Comment #9

[Bug c++/36587] Feature: add warning for constructor call with discarded return.

2015-07-24 Thread kkylheku at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=36587 --- Comment #11 from Kaz Kylheku kkylheku at gmail dot com --- (In reply to Manuel López-Ibáñez from comment #10) (In reply to Kaz Kylheku from comment #1) Created attachment 15798 [details] Implements -Wunused-objects warning for C

[Bug c++/36587] Feature: add warning for constructor call with discarded return.

2009-12-11 Thread kkylheku at gmail dot com
--- Comment #6 from kkylheku at gmail dot com 2009-12-11 11:57 --- (In reply to comment #5) (In reply to comment #4) But I'm not convinced that doing this is always a mistake. Since we don't care about the object, we must care about the constructor side effect. I seem

[Bug c++/36587] Feature: add warning for constructor call with discarded return.

2009-12-10 Thread kkylheku at gmail dot com
--- Comment #4 from kkylheku at gmail dot com 2009-12-11 02:34 --- (In reply to comment #3) This would have prevented bugs I've dealt with where critical sections where not protected: { lock_guard (mutex); // mutex NOT locked here! } But I'm not convinced that doing

[Bug preprocessor/38990] preprocessing different in g++ -E and regular compiling.

2009-01-28 Thread kkylheku at gmail dot com
--- Comment #2 from kkylheku at gmail dot com 2009-01-28 16:30 --- (In reply to comment #1) Confirmed. Thanks. By the way, I started looking at patching this. My suspicions were confirmed that this is a case of pasting together invalid tokens. The compiler sees the tokens individually

[Bug c++/38990] New: preprocessing different in g++ -E and regular compiling.

2009-01-27 Thread kkylheku at gmail dot com
Priority: P3 Component: c++ AssignedTo: unassigned at gcc dot gnu dot org ReportedBy: kkylheku at gmail dot com http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38990

[Bug c++/36587] New: Feature: add warning for constructor call with discarded return.

2008-06-20 Thread kkylheku at gmail dot com
Component: c++ AssignedTo: unassigned at gcc dot gnu dot org ReportedBy: kkylheku at gmail dot com http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36587

[Bug c++/36587] Feature: add warning for constructor call with discarded return.

2008-06-20 Thread kkylheku at gmail dot com
--- Comment #1 from kkylheku at gmail dot com 2008-06-20 19:23 --- Created an attachment (id=15798) -- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15798action=view) Implements -Wunused-objects warning for C++. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36587

[Bug c++/36587] Feature: add warning for constructor call with discarded return.

2008-06-20 Thread kkylheku at gmail dot com
--- Comment #2 from kkylheku at gmail dot com 2008-06-20 20:26 --- I should add that this is different from -Wunused-value, because I want this warning emitted even if the constructor (or its corresponding destructor) have side effects. -- http://gcc.gnu.org/bugzilla/show_bug.cgi

[Bug driver/35300] References to original ${prefix} paths in relocated toolchain and /lib and /usr/lib search paths appear in cross toolchain.

2008-02-23 Thread kkylheku at gmail dot com
--- Comment #7 from kkylheku at gmail dot com 2008-02-23 08:03 --- Both my patches apply even if Carlos' patches are removed. The crti.o problem remains. What's happening is that xgcc actually searches for the crti.o module, and then passes the full path to crti.o down to collect2

[Bug driver/35300] References to original ${prefix} paths in relocated toolchain and /lib and /usr/lib search paths appear in cross toolchain.

2008-02-23 Thread kkylheku at gmail dot com
--- Comment #8 from kkylheku at gmail dot com 2008-02-23 08:54 --- I ended up doing the symlink trick because quite a bit of the sysroot material is needed to build everything, like libstdc++. It worked like a charm. And so now I have a compiler with search paths only in its own tree

[Bug driver/35300] New: References to original ${prefix} paths in relocated toolchain and /lib and /usr/lib search paths appear in cross toolchain.

2008-02-22 Thread kkylheku at gmail dot com
in cross toolchain. Product: gcc Version: 4.1.1 Status: UNCONFIRMED Severity: normal Priority: P3 Component: driver AssignedTo: unassigned at gcc dot gnu dot org ReportedBy: kkylheku at gmail dot com GCC build

[Bug driver/35300] References to original ${prefix} paths in relocated toolchain and /lib and /usr/lib search paths appear in cross toolchain.

2008-02-22 Thread kkylheku at gmail dot com
--- Comment #1 from kkylheku at gmail dot com 2008-02-23 01:35 --- Created an attachment (id=15210) -- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15210action=view) Patch to gcc/prefix.c for path relocation. At some point gcc/gcc.c calls the function set_std_prefix in gcc/prefix.c

[Bug driver/35300] References to original ${prefix} paths in relocated toolchain and /lib and /usr/lib search paths appear in cross toolchain.

2008-02-22 Thread kkylheku at gmail dot com
--- Comment #2 from kkylheku at gmail dot com 2008-02-23 01:40 --- Created an attachment (id=15211) -- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15211action=view) Hack to not have /lib and /usr/ paths in cross-compiler. I don't know whether this path is right in general

[Bug driver/35300] References to original ${prefix} paths in relocated toolchain and /lib and /usr/lib search paths appear in cross toolchain.

2008-02-22 Thread kkylheku at gmail dot com
--- Comment #3 from kkylheku at gmail dot com 2008-02-23 02:18 --- Oops, I spoke to soon. The no-prefix-search path breaks my final gcc build, with shared libraries and glibc. When making libgcc_s.so, it can't find the crti.o module. What's happening is two problems. Firstly

[Bug driver/35300] References to original ${prefix} paths in relocated toolchain and /lib and /usr/lib search paths appear in cross toolchain.

2008-02-22 Thread kkylheku at gmail dot com
--- Comment #5 from kkylheku at gmail dot com 2008-02-23 04:58 --- (In reply to comment #4) [crti.o is] found through multilib os-directory suffixes. Actually, I'm even more confused, because the breakage I'm seeing is simply arising from mips64-linux-ld being called on a bunch

[Bug driver/35300] References to original ${prefix} paths in relocated toolchain and /lib and /usr/lib search paths appear in cross toolchain.

2008-02-22 Thread kkylheku at gmail dot com
--- Comment #6 from kkylheku at gmail dot com 2008-02-23 05:06 --- (In reply to comment #5) (In reply to comment #4) [crti.o is] found through multilib os-directory suffixes. Actually, I'm even more confused, because the breakage I'm seeing is simply arising from mips64-linux-ld

[Bug rtl-optimization/34456] Flaw in branch delay optimization (patch included).

2007-12-13 Thread kkylheku at gmail dot com
--- Comment #1 from kkylheku at gmail dot com 2007-12-14 02:51 --- Created an attachment (id=14749) -- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=14749action=view) Patch for caller used registers in mark_set_resources. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34456

[Bug rtl-optimization/34456] New: Flaw in branch delay optimization (patch included).

2007-12-13 Thread kkylheku at gmail dot com
: unassigned at gcc dot gnu dot org ReportedBy: kkylheku at gmail dot com http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34456