[Bug tree-optimization/84969] [8 Regression] Wrong code with -ftree-loop-distribute-patterns
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84969 --- Comment #6 from CVS Commits --- The master branch has been updated by Roger Sayle : https://gcc.gnu.org/g:a717376e99fb33ba3b06bd8122e884f4b63a60c9 commit r12-7607-ga717376e99fb33ba3b06bd8122e884f4b63a60c9 Author: Roger Sayle Date: Thu Mar 10 23:49:15 2022 + PR c++/84964: Middle-end patch to expand_call for ICE after sorry. This patch resolves PR c++/84969 which is an ICE in the middle-end after emitting a "sorry, unimplemented" message, and is a regression from earlier releases of GCC. This issue is that after encountering a function call requiring an unreasonable amount of stack space, the code continues and falls foul of an assert checking that stack pointer has been correctly updated. The fix is to (locally) consider aborted function calls as "no return", which skips this downstream sanity check. 2022-03-10 Roger Sayle gcc/ChangeLog PR c++/84964 * calls.cc (expand_call): Ignore stack adjustments after sorry. gcc/testsuite/ChangeLog PR c++/84964 * g++.dg/other/pr84964.C: New test case.
[Bug tree-optimization/84969] [8 Regression] Wrong code with -ftree-loop-distribute-patterns
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84969 amker at gcc dot gnu.org changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|--- |FIXED --- Comment #5 from amker at gcc dot gnu.org --- (In reply to Jakub Jelinek from comment #4) > So fixed? Yes, I think so.
[Bug tree-optimization/84969] [8 Regression] Wrong code with -ftree-loop-distribute-patterns
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84969 Jakub Jelinek changed: What|Removed |Added CC||jakub at gcc dot gnu.org --- Comment #4 from Jakub Jelinek --- So fixed?
[Bug tree-optimization/84969] [8 Regression] Wrong code with -ftree-loop-distribute-patterns
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84969 --- Comment #3 from amker at gcc dot gnu.org --- Author: amker Date: Wed Mar 21 10:42:34 2018 New Revision: 258710 URL: https://gcc.gnu.org/viewcvs?rev=258710=gcc=rev Log: PR tree-optimization/84969 * tree-loop-distribution.c (fuse_memset_builtins): Don't reorder builtin memset partitions if they set different rhs values. gcc/testsuite * gcc.dg/tree-ssa/pr84969.c: New test. Added: trunk/gcc/testsuite/gcc.dg/tree-ssa/pr84969.c Modified: trunk/gcc/ChangeLog trunk/gcc/testsuite/ChangeLog trunk/gcc/tree-loop-distribution.c
[Bug tree-optimization/84969] [8 Regression] Wrong code with -ftree-loop-distribute-patterns
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84969 --- Comment #2 from amker at gcc dot gnu.org --- I see, there are two issues here. - wrong code issue: fuse_memset_builtins breaks dependence between two intersect memset call; - efficiency issue: The loop should not be distributed into two memset calls by some model. I will fix the first one now and check if the second should be fixed at this point or not.
[Bug tree-optimization/84969] [8 Regression] Wrong code with -ftree-loop-distribute-patterns
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84969 Martin Liška changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed|2018-03-19 00:00:00 |2018-03-20 Ever confirmed|0 |1
[Bug tree-optimization/84969] [8 Regression] Wrong code with -ftree-loop-distribute-patterns
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84969 amker at gcc dot gnu.org changed: What|Removed |Added Priority|P3 |P1
[Bug tree-optimization/84969] [8 Regression] Wrong code with -ftree-loop-distribute-patterns
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84969 amker at gcc dot gnu.org changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |amker at gcc dot gnu.org --- Comment #1 from amker at gcc dot gnu.org --- Sorry for the breakage, I will investigate it.
[Bug tree-optimization/84969] [8 Regression] Wrong code with -ftree-loop-distribute-patterns
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84969 Martin Liška changed: What|Removed |Added Last reconfirmed||2018-3-19 Known to work||7.3.0 Target Milestone|--- |8.0 Known to fail||8.0