https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88788
Richard Biener changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88788
--- Comment #17 from Arseny Solokha ---
Is it fixed now?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88788
--- Comment #16 from prathamesh3492 at gcc dot gnu.org ---
Author: prathamesh3492
Date: Tue Jan 15 09:37:22 2019
New Revision: 267933
URL: https://gcc.gnu.org/viewcvs?rev=267933=gcc=rev
Log:
2019-01-15 Richard Biener
Prathamesh
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88788
--- Comment #15 from Richard Biener ---
(In reply to prathamesh3492 from comment #14)
> Created attachment 45425 [details]
> Patch
>
> Hi,
> In the attached patch, I cache results of malloc_candidate_p_1 and avoid
> traversing "back edges".
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88788
--- Comment #14 from prathamesh3492 at gcc dot gnu.org ---
Created attachment 45425
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=45425=edit
Patch
Hi,
In the attached patch, I cache results of malloc_candidate_p_1 and avoid
traversing
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88788
Richard Biener changed:
What|Removed |Added
Status|NEW |ASSIGNED
--- Comment #13 from Richard
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88788
--- Comment #12 from prathamesh3492 at gcc dot gnu.org ---
(In reply to Jakub Jelinek from comment #11)
> Look e.g. at -O2:
> void bar (int);
>
> void
> foo (int x)
> {
> int i = 0;
> if (x == 8)
> {
> x = 16;
> goto lab;
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88788
--- Comment #11 from Jakub Jelinek ---
Look e.g. at -O2:
void bar (int);
void
foo (int x)
{
int i = 0;
if (x == 8)
{
x = 16;
goto lab;
}
for (; i < 100; i++)
{
lab:
bar (x);
}
}
but pretty much any
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88788
--- Comment #10 from prathamesh3492 at gcc dot gnu.org ---
Oops, I didn't realize there could be loop within phi (phi result being an arg
too). I will try to come up with a better approach for handling nested PHI's.
In the meantime, for stage 4,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88788
--- Comment #9 from Jakub Jelinek ---
And no, you casn't rely on phis having just a few arguments, there are phis
with tens of thousands of arguments in real-world code.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88788
--- Comment #8 from Jakub Jelinek ---
I don't see how that can help.
First of all, the duplicate SSA_NAMEs don't have to appear just in the same
PHI, you can have them in multiple different PHIs too.
And, you can have loops there as well, either
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88788
prathamesh3492 at gcc dot gnu.org changed:
What|Removed |Added
Assignee|unassigned at gcc dot gnu.org |prathamesh3492
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88788
Jakub Jelinek changed:
What|Removed |Added
CC||jakub at gcc dot gnu.org
--- Comment #6
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88788
--- Comment #5 from prathamesh3492 at gcc dot gnu.org ---
(In reply to Martin Liška from comment #4)
> Created attachment 45403 [details]
> reduced test-case
Thanks!
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88788
--- Comment #4 from Martin Liška ---
Created attachment 45403
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=45403=edit
reduced test-case
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88788
--- Comment #3 from Martin Liška ---
(In reply to prathamesh3492 from comment #2)
> Sorry for the breakage, I will take a look.
>
> Regards,
> Prathamesh
Wait, I have almost reduced test-case.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88788
--- Comment #2 from prathamesh3492 at gcc dot gnu.org ---
Sorry for the breakage, I will take a look.
Regards,
Prathamesh
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88788
Martin Liška changed:
What|Removed |Added
Priority|P3 |P1
Status|UNCONFIRMED
18 matches
Mail list logo
