https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77485
Bug 77485 depends on bug 33562, which changed state.
Bug 33562 Summary: [6 Regression] aggregate DSE disabled
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=33562
What|Removed |Added
--
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77485
--- Comment #16 from Jeffrey A. Law ---
Author: law
Date: Mon Jan 16 23:43:05 2017
New Revision: 244509
URL: https://gcc.gnu.org/viewcvs?rev=244509&root=gcc&view=rev
Log:
2017-01-16 Jeff Law
PR tree-optimization/79090
PR tree
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77485
--- Comment #15 from Jeffrey A. Law ---
Author: law
Date: Sat Jan 14 06:16:23 2017
New Revision: 244461
URL: https://gcc.gnu.org/viewcvs?rev=244461&root=gcc&view=rev
Log:
PR tree-optimization/33562
PR tree-optimization/61912
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77485
Bug 77485 depends on bug 33562, which changed state.
Bug 33562 Summary: [5/6 Regression] aggregate DSE disabled
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=33562
What|Removed |Added
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77485
Bug 77485 depends on bug 33562, which changed state.
Bug 33562 Summary: [5/6 Regression] aggregate DSE disabled
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=33562
What|Removed |Added
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77485
--- Comment #14 from Jeffrey A. Law ---
Author: law
Date: Fri Jan 13 15:46:22 2017
New Revision: 23
URL: https://gcc.gnu.org/viewcvs?rev=23&root=gcc&view=rev
Log:
PR tree-optimization/61912
PR tree-optimization/77485
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77485
--- Comment #12 from Jeffrey A. Law ---
Author: law
Date: Fri Jan 13 15:37:09 2017
New Revision: 21
URL: https://gcc.gnu.org/viewcvs?rev=21&root=gcc&view=rev
Log:
PR tree-optimization/33562
PR tree-optimization/61912
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77485
--- Comment #13 from Jeffrey A. Law ---
Author: law
Date: Fri Jan 13 15:42:08 2017
New Revision: 22
URL: https://gcc.gnu.org/viewcvs?rev=22&root=gcc&view=rev
Log:
PR tree-optimization/33562
PR tree-optimization/61912
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77485
Jeffrey A. Law changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77485
--- Comment #10 from Jeffrey A. Law ---
[0.00%]:
MEM[(char[170] *)& + 30B] = {};
.buf[0] = 48;
[ ... ]
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77485
--- Comment #9 from Jeffrey A. Law ---
That ought to be significantly easier and cleaner. I really didn't like the
transformation into memset. I'll give that a whirl tomorrow.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77485
--- Comment #8 from rguenther at suse dot de ---
On Wed, 14 Dec 2016, law at redhat dot com wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77485
>
> --- Comment #7 from Jeffrey A. Law ---
> There was a bit of cruft and a missed ADDR_EXP
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77485
--- Comment #7 from Jeffrey A. Law ---
There was a bit of cruft and a missed ADDR_EXPR in that last fragment of gimple
code.
;; basic block 2, loop depth 0, count 0, freq 0, maybe hot
;;prev block 0, next block 1, flags: (NEW, REACHABLE, V
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77485
--- Comment #6 from Jeffrey A. Law ---
So DSE involving CONSTRUCTOR nodes is a bit of a pain.
The way CONSTRUCTOR nodes are used past gimplification is awkward for DSE.
Specifically we have a CONSTRUCTOR node with no ELTS. The semantics of tha
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77485
--- Comment #5 from rguenther at suse dot de ---
On Thu, 8 Dec 2016, law at redhat dot com wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77485
>
> --- Comment #4 from Jeffrey A. Law ---
> So my patches for 33562 will detect the partial
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77485
--- Comment #4 from Jeffrey A. Law ---
So my patches for 33562 will detect the partial dead store in "foo", but I
never wrote the bits to narrow partial dead stores.
The difficulty in optimizing this particular case will be rewriting the
CONSTRU
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77485
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #3 fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77485
--- Comment #2 from petschy at gmail dot com ---
I agree that the generic case can become quite complicated: if after the
memset, the individual values are written with gaps between them, or multiple
contiguous chunks with gaps between them, it's
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77485
Richard Biener changed:
What|Removed |Added
Keywords||missed-optimization
Status|
19 matches
Mail list logo