https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87665
--- Comment #9 from Jonathan Wakely ---
Correction, on both x86_64 and ppc64le it requires all of:
-fstrict-aliasing -ftree-loop-vectorize -ftree-sra -ftree-ch -ftree-forwprop
-ftree-copy-prop
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87665
Ville Voutilainen changed:
What|Removed |Added
CC||ville.voutilainen at gmail dot
com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87665
--- Comment #7 from Jonathan Wakely ---
On x86_64 it fails with -O1 -fstrict-aliasing -ftree-loop-vectorize
It still fails if every -fno-xxx option for -O1 is added except
-fno-tree-copy-prop so it seems to require:
-fstrict-aliasing
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87665
--- Comment #6 from Jonathan Wakely ---
Since r254694 only changed all -std modes to match -std=c++17 I bisected again
using -std=c++17 to find the change that really caused it, and it was indeed
r246301 which first removed that null check.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87665
--- Comment #5 from Jonathan Wakely ---
Confirmed that -fcheck-null fixes the miscompilation.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87665
Jonathan Wakely changed:
What|Removed |Added
CC||ville at gcc dot gnu.org
Target
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87665
Richard Biener changed:
What|Removed |Added
Target Milestone|--- |8.3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87665
--- Comment #3 from Jan Christoph Uhde ---
ยป g++-head -v
Using built-in specs.
COLLECT_GCC=g++-head
COLLECT_LTO_WRAPPER=/bal/gcc_trunk_install/bin/../libexec/gcc/x86_64-pc-linux-gnu/9.0.0/lto-wrapper
Target: x86_64-pc-linux-gnu
Configured with:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87665
--- Comment #2 from Jonathan Wakely ---
(In reply to Jonathan Wakely from comment #1)
> The behaviour changed with r255268
Which doesn't make sense as the bug seems to be target-independent.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87665
Jonathan Wakely changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Known to work|
10 matches
Mail list logo