[Bug c++/65866] Wrong warning when using list-initialization: operation on 'b' may be undefined [-Wsequence-point]

2024-07-13 Thread matthijsvanduin at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65866 --- Comment #8 from Matthijs van Duin --- (In reply to Yuxuan Shui from comment #7) > IIUC now wrong codegen has all been fixed? so the only thing left should be > the diagnostic bug? It seems so yes, the combined testcase in Bug 70796 comment

[Bug c++/70796] [DR 1030] Initialization order with braced-init-lists still broken

2021-11-03 Thread matthijsvanduin at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70796 Matthijs van Duin changed: What|Removed |Added CC||matthijsvanduin at gmail dot com

[Bug c++/65866] Wrong warning when using list-initialization: operation on 'b' may be undefined [-Wsequence-point]

2021-11-03 Thread matthijsvanduin at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65866 Matthijs van Duin changed: What|Removed |Added CC||matthijsvanduin at gmail dot com

[Bug c++/70792] Incorrect sequence point warning with uniform initializer syntax

2021-11-03 Thread matthijsvanduin at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70792 --- Comment #12 from Matthijs van Duin --- (In reply to Jonathan Wakely from comment #11) > (In reply to Matthijs van Duin from comment #10) > > I'm assuming this means a new bug should be opened > > about the wrong code generation? > > Yes

[Bug c++/70792] Incorrect sequence point warning with uniform initializer syntax

2021-11-03 Thread matthijsvanduin at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70792 --- Comment #10 from Matthijs van Duin --- (In reply to Jonathan Wakely from comment #9) > (In reply to Matthijs van Duin from comment #4) > > Also bug 65866 exists for the warning itself. > > I think this is an exact dup. > > *** This bug has

[Bug c++/89695] unexpected copying of trivially copyable prvalue arguments

2021-10-28 Thread matthijsvanduin at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89695 --- Comment #5 from Matthijs van Duin --- So for reference, the Itanium C++ ABI says: "A type which is trivial for the purposes of the ABI will be passed and returned according to the rules of the base C ABI, e.g. in registers; often this has

[Bug c++/102942] Implicit copy constructor not elided

2021-10-28 Thread matthijsvanduin at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102942 Matthijs van Duin changed: What|Removed |Added CC||matthijsvanduin at gmail dot com

[Bug tree-optimization/83234] Aggressive loop optim warning for loop iteration that cannot happen

2021-07-30 Thread matthijsvanduin at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83234 Matthijs van Duin changed: What|Removed |Added CC||matthijsvanduin at gmail dot com

[Bug tree-optimization/100801] Aggressive loop optimizations cause incorrect warning

2021-07-30 Thread matthijsvanduin at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100801 Matthijs van Duin changed: What|Removed |Added CC||matthijsvanduin at gmail dot com

[Bug target/92075] New: extracting element from NEON float-vector moves to/from integer register

2019-10-12 Thread matthijsvanduin at gmail dot com
: normal Priority: P3 Component: target Assignee: unassigned at gcc dot gnu.org Reporter: matthijsvanduin at gmail dot com Target Milestone: --- On ARM, when extracting an element from a float32x2_t expression, i.e.: float32x2_t v = (...); float v0 = v[0

[Bug c++/89695] unexpected copying of trivially copyable prvalue arguments

2019-10-03 Thread matthijsvanduin at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89695 --- Comment #4 from Matthijs van Duin --- (In reply to Jonathan Wakely from comment #3) > I believe this is required by the ABI for trivially copyable types. I don't see how that's possible, the callee can't tell the difference.

[Bug c++/89695] unexpected copying of trivially copyable prvalue arguments

2019-03-13 Thread matthijsvanduin at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89695 Matthijs van Duin changed: What|Removed |Added Summary|inappropriate copying of|unexpected copying of

[Bug c++/89695] New: inappropriate copying of trivially copyable prvalue arguments

2019-03-12 Thread matthijsvanduin at gmail dot com
Priority: P3 Component: c++ Assignee: unassigned at gcc dot gnu.org Reporter: matthijsvanduin at gmail dot com Target Milestone: --- When a function that has a non-reference parameter of a trivially copyable class-type is invoked with a prvalue expression as argument

[Bug c++/86521] [8/9 Regression] GCC 8 selects incorrect overload of ref-qualified conversion operator template

2019-03-11 Thread matthijsvanduin at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86521 Matthijs van Duin changed: What|Removed |Added CC||matthijsvanduin at gmail dot com

[Bug c++/51253] [C++11][DR 1030] Evaluation order (sequenced-before relation) among initializer-clauses in braced-init-list

2019-01-25 Thread matthijsvanduin at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=51253 --- Comment #25 from Matthijs van Duin --- I wasn't referring to the warnings though but incorrect code generation. Since is exhibited by pretty trivial test cases (testsuite/g++.dg/cpp0x/initlist86.C confirms that { i++, i++ } works but the

[Bug c++/70792] Incorrect sequence point warning with uniform initializer syntax

2019-01-25 Thread matthijsvanduin at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70792 --- Comment #8 from Matthijs van Duin --- (In reply to Matthijs van Duin from comment #4) > return std::pair{ ++i, ++i }.first; My bad! This isn't an exhibit of the bug. I simply forgot that std::pair is not really a struct, and this

[Bug c++/83028] Incorrect -Wsequence-point warning in correct C++17 code with new evaluation order rules

2018-10-28 Thread matthijsvanduin at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83028 Matthijs van Duin changed: What|Removed |Added CC||matthijsvanduin at gmail dot com

[Bug c++/70792] Incorrect sequence point warning with uniform initializer syntax

2018-10-01 Thread matthijsvanduin at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70792 --- Comment #7 from Matthijs van Duin --- Ah! I should have checked the actual tests of 51253 before calling it a regression, apologies. I just kinda assumed that these simple cases would be covered by the work done back then.

[Bug c++/51253] [C++11][DR 1030] Evaluation order (sequenced-before relation) among initializer-clauses in braced-init-list

2018-09-30 Thread matthijsvanduin at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=51253 Matthijs van Duin changed: What|Removed |Added CC||matthijsvanduin at gmail dot com

[Bug c++/70792] Incorrect sequence point warning with uniform initializer syntax

2018-09-30 Thread matthijsvanduin at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70792 Matthijs van Duin changed: What|Removed |Added CC||matthijsvanduin at gmail dot com

[Bug target/82989] [7/8 regression] Inexplicable use of NEON for 64-bit math

2018-02-19 Thread matthijsvanduin at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82989 --- Comment #15 from Matthijs van Duin --- (In reply to Wilco from comment #14) > Yes on older cores it can be a bad idea to allow accidental use of Neon > instructions. The simplest workaround is to switch off Neon, just use > -mfpu=vfp. Sure,

[Bug target/82989] [7/8 regression] Inexplicable use of NEON for 64-bit math

2018-02-19 Thread matthijsvanduin at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82989 --- Comment #13 from Matthijs van Duin --- In case it's of interest, I did a quick benchmark of my testcase executed in a loop on a cortex-a8: Without neon: 12 instructions/iteration 14 cycles/iteration With neon: 14

[Bug target/82989] [7/8 regression ] Inexplicable use of NEON for 64-bit math

2018-02-15 Thread matthijsvanduin at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82989 --- Comment #9 from Matthijs van Duin --- I can still reproduce the same test case with: arm-linux-gnueabihf-gcc-8 (Debian 8-20180207-2) 8.0.1 20180207 (experimental) [trunk revision 257435] -mfloat-abi=hard is implicit for

[Bug target/49526] extra move instruction for smmul

2017-11-17 Thread matthijsvanduin at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=49526 Matthijs van Duin changed: What|Removed |Added CC||matthijsvanduin at gmail dot com

[Bug target/82989] New: Inexplicable use of NEON for 64-bit math

2017-11-14 Thread matthijsvanduin at gmail dot com
Assignee: unassigned at gcc dot gnu.org Reporter: matthijsvanduin at gmail dot com Target Milestone: --- The following function: void foo( uint64_t *a ) { *a += *a >> 32; } compiled with arm-linux-gnueabihf-gcc -mcpu=cortex-a8 -mfpu=neon -O2 pr

[Bug c++/80061] error on constexpr function with an unevaluated throw

2017-07-21 Thread matthijsvanduin at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80061 --- Comment #2 from Matthijs van Duin --- > void foo( bool ok ) { ^constexpr

[Bug c++/80061] error on constexpr function with an unevaluated throw

2017-07-21 Thread matthijsvanduin at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80061 Matthijs van Duin changed: What|Removed |Added CC||matthijsvanduin at gmail dot com

[Bug c++/67371] Never executed "throw" in constexpr function fails to compile

2017-07-21 Thread matthijsvanduin at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67371 Matthijs van Duin changed: What|Removed |Added CC||matthijsvanduin at gmail dot com

[Bug c++/59832] [c++11] ICE in reshape_init_class with initializer list

2016-12-15 Thread matthijsvanduin at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59832 Matthijs van Duin changed: What|Removed |Added CC||matthijsvanduin at gmail dot com

[Bug middle-end/50481] builtin to reverse the bit order

2016-12-08 Thread matthijsvanduin at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=50481 Matthijs van Duin changed: What|Removed |Added CC||matthijsvanduin at gmail dot com

[Bug c++/67625] New: some constexpr expressions rejected as enumerator value

2015-09-18 Thread matthijsvanduin at gmail dot com
Priority: P3 Component: c++ Assignee: unassigned at gcc dot gnu.org Reporter: matthijsvanduin at gmail dot com Target Milestone: --- GCC 5.2.1 rejects the following code (with -std=gnu++11 or later): constexpr uint16_t bswap16( uint16_t x ) { return