https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66726
Andrew Pinski changed:
What|Removed |Added
See Also||https://gcc.gnu.org/bugzill
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66726
--- Comment #21 from Jorn Wolfgang Rennecke ---
Author: amylaar
Date: Mon Jul 1 21:48:55 2019
New Revision: 272911
URL: https://gcc.gnu.org/viewcvs?rev=272911&root=gcc&view=rev
Log:
PR middle-end/66726
* tree-ssa-phiopt.c (facto
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66726
Jeffrey A. Law changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66726
--- Comment #19 from kugan at gcc dot gnu.org ---
Author: kugan
Date: Sun Jul 24 12:47:29 2016
New Revision: 238695
URL: https://gcc.gnu.org/viewcvs?rev=238695&root=gcc&view=rev
Log:
gcc/ChangeLog:
2016-07-24 Kugan Vivekanandarajah
P
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66726
Eric Botcazou changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66726
--- Comment #18 from kugan at gcc dot gnu.org ---
Reverted r233362 as it caused PR69786 and PR69781. I will test for these and
post a revised patch for next stage1.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66726
--- Comment #17 from kugan at gcc dot gnu.org ---
Author: kugan
Date: Fri Feb 12 06:40:55 2016
New Revision: 233368
URL: https://gcc.gnu.org/viewcvs?rev=233368&root=gcc&view=rev
Log:
2016-02-12 Kugan Vivekanandarajah
revert:
2
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66726
--- Comment #16 from kugan at gcc dot gnu.org ---
Author: kugan
Date: Fri Feb 12 00:24:22 2016
New Revision: 233362
URL: https://gcc.gnu.org/viewcvs?rev=233362&root=gcc&view=rev
Log:
gcc/ChangeLog:
2016-02-12 Kugan Vivekanandarajah
P
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66726
--- Comment #15 from Jakub Jelinek ---
Author: jakub
Date: Mon Dec 7 21:05:51 2015
New Revision: 231384
URL: https://gcc.gnu.org/viewcvs?rev=231384&root=gcc&view=rev
Log:
PR middle-end/66726
* g++.dg/tree-ssa/pr66726.c: Renamed
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66726
--- Comment #14 from kugan at gcc dot gnu.org ---
Sorry I missed this. I posted patch which was OKed but in further testing I
found an issue. I am testing a modified patch and will post for review soon.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66726
--- Comment #13 from Andreas Schwab ---
../../gcc/tree-ssa-reassoc.c: In function 'void
maybe_optimize_range_tests(gimple*)':
../../gcc/tree-ssa-reassoc.c:3193:8: error: 'operand_entry_t' was not declared
in this scope
operand_entry_t oe
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66726
--- Comment #12 from kugan at gcc dot gnu.org ---
Created attachment 35976
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=35976&action=edit
patch for tree-ssa-reassoc
Here is a prototype patch (to fix comment 9) that makes tree-ssa-reassoc
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66726
--- Comment #11 from kugan at gcc dot gnu.org ---
Thanks for reporting. This test case is valid for targets that has branch cost
greater than 1.
One way to handle this is by disabling this for convections involving bool that
are part of branch (?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66726
--- Comment #10 from Jeffrey A. Law ---
Sinking the cast changes the form of the range tests into one that
tree-ssa-reassoc isn't prepared to handle. Sadly the form presented with the
cast sunk is *simpler* than the original.
I'm testing a bit
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66726
--- Comment #9 from Andreas Schwab ---
On m68k:
FAIL: gcc.dg/pr46309.c scan-tree-dump-times reassoc1 "Optimizing range tests
a_[0-9]*.D. -.1, 1. and -.3, 3.[\n\r]* into" 1
FAIL: gcc.dg/pr46309.c scan-tree-dump-times reassoc1 "Optimizing range te
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66726
--- Comment #8 from kugan at gcc dot gnu.org ---
Author: kugan
Date: Sun Jul 12 11:22:42 2015
New Revision: 225722
URL: https://gcc.gnu.org/viewcvs?rev=225722&root=gcc&view=rev
Log:
gcc/testsuite/ChangeLog:
2015-07-12 Kugan Vivekanandarajah
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66726
--- Comment #7 from Jeffrey A. Law ---
The other thing to keep in mind, sinking of this nature ought to be applicable
to other unary ops and cases where we have multiple PHI args that are
SSA_NAMEs.It shouldn't be structurally limited to just
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66726
--- Comment #6 from Jeffrey A. Law ---
WRT c#2. PRE will try to optimize away a runtime redundant expression. In the
cases I'm looking at, there is only a single runtime evaluation. But that
evaluation can be sunk from a set of predecessors i
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66726
--- Comment #5 from kugan at gcc dot gnu.org ---
(In reply to Andrew Pinski from comment #4)
> (In reply to kugan from comment #3)
> > > really you should handle more
> > > than two arguments to phis.
> > I am not sure how we can handle phi stmt w
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66726
--- Comment #4 from Andrew Pinski ---
(In reply to kugan from comment #3)
> > really you should handle more
> > than two arguments to phis.
> I am not sure how we can handle phi stmt with more than two arguments here.
> Any hints please?
Yes the
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66726
--- Comment #3 from kugan at gcc dot gnu.org ---
> really you should handle more
> than two arguments to phis.
I am not sure how we can handle phi stmt with more than two arguments here. Any
hints please?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66726
--- Comment #2 from Andrew Pinski ---
Comment on attachment 35888
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=35888
untested prototype patch
I thought pre does this already and that seems like a better place than
phi-opt. Oh wait pre do
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66726
--- Comment #1 from kugan at gcc dot gnu.org ---
Created attachment 35888
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=35888&action=edit
untested prototype patch
Hi Jeff,
Here is a patch (without debug dumps and not tesetd fully). Is thi
23 matches
Mail list logo
