https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88834
rsandifo at gcc dot gnu.org changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolutio
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88834
--- Comment #22 from rsandifo at gcc dot gnu.org
---
(In reply to kugan from comment #21)
> (In reply to Christophe Lyon from comment #20)
> > Hi Kugan,
> >
> > The new test fails with -mabi=ilp32:
> > FAIL: gcc.target/aarch64/pr88834.c scan-as
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88834
--- Comment #21 from kugan at gcc dot gnu.org ---
(In reply to Christophe Lyon from comment #20)
> Hi Kugan,
>
> The new test fails with -mabi=ilp32:
> FAIL: gcc.target/aarch64/pr88834.c scan-assembler-times \\tld2w\\t{z[0-9]+.s
> - z[0-9]+.s}, p
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88834
--- Comment #20 from Christophe Lyon ---
Hi Kugan,
The new test fails with -mabi=ilp32:
FAIL: gcc.target/aarch64/pr88834.c scan-assembler-times \\tld2w\\t{z[0-9]+.s -
z[0-9]+.s}, p[0-7]/z, \\[x[0-9]+, x[0-9]+, lsl 2\\]\\n 2
FAIL: gcc.target/aarc
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88834
--- Comment #19 from kugan at gcc dot gnu.org ---
Author: kugan
Date: Thu Jun 13 03:18:54 2019
New Revision: 272232
URL: https://gcc.gnu.org/viewcvs?rev=272232&root=gcc&view=rev
Log:
gcc/ChangeLog:
2019-06-13 Kugan Vivekanandarajah
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88834
--- Comment #18 from rsandifo at gcc dot gnu.org
---
(In reply to kugan from comment #12)
> (In reply to rsand...@gcc.gnu.org from comment #10)
> > (In reply to kugan from comment #9)
> > > Created attachment 46040 [details]
> > > patch
> >
> >
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88834
--- Comment #17 from kugan at gcc dot gnu.org ---
(In reply to Wilco from comment #16)
> (In reply to kugan from comment #15)
> > (In reply to Wilco from comment #11)
> > > There is also something odd with the way the loop iterates, this doesn't
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88834
--- Comment #16 from Wilco ---
(In reply to kugan from comment #15)
> (In reply to Wilco from comment #11)
> > There is also something odd with the way the loop iterates, this doesn't
> > look right:
> >
> > whilelo p0.s, x3, x4
> >
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88834
--- Comment #15 from kugan at gcc dot gnu.org ---
(In reply to Wilco from comment #11)
> There is also something odd with the way the loop iterates, this doesn't
> look right:
>
> whilelo p0.s, x3, x4
> incwx3
> ptest
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88834
--- Comment #14 from kugan at gcc dot gnu.org ---
Created attachment 46104
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=46104&action=edit
testcase
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88834
kugan at gcc dot gnu.org changed:
What|Removed |Added
Attachment #46040|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88834
--- Comment #12 from kugan at gcc dot gnu.org ---
(In reply to rsand...@gcc.gnu.org from comment #10)
> (In reply to kugan from comment #9)
> > Created attachment 46040 [details]
> > patch
>
> Wasn't sure whether this patch was WIP or the final v
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88834
Wilco changed:
What|Removed |Added
CC||wilco at gcc dot gnu.org
--- Comment #11 from Wi
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88834
--- Comment #10 from rsandifo at gcc dot gnu.org
---
(In reply to kugan from comment #9)
> Created attachment 46040 [details]
> patch
Wasn't sure whether this patch was WIP or the final version
for review, but we need to do something more gener
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88834
kugan at gcc dot gnu.org changed:
What|Removed |Added
Attachment #45686|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88834
--- Comment #8 from kugan at gcc dot gnu.org ---
(In reply to rsand...@gcc.gnu.org from comment #7)
> Thanks for looking at this.
>
> (In reply to kugan from comment #6)
> > cmp w3, 0
> > ble .L1
> > sub w3, w3, #1
> >
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88834
--- Comment #7 from rsandifo at gcc dot gnu.org
---
Thanks for looking at this.
(In reply to kugan from comment #6)
> cmp w3, 0
> ble .L1
> sub w3, w3, #1
> mov x4, 0
> cntwx5
> ptrue p1
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88834
--- Comment #6 from kugan at gcc dot gnu.org ---
>
> Note the difference in mode for aarch64_classify_address. Not sure if this
> is because of the way my patch changes ivopt.
Yes, it ws my mistake in iv-use. with attached patch, I now get
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88834
kugan at gcc dot gnu.org changed:
What|Removed |Added
Attachment #45661|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88834
--- Comment #4 from kugan at gcc dot gnu.org ---
Created attachment 45661
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=45661&action=edit
ivopt patch v1
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88834
--- Comment #3 from kugan at gcc dot gnu.org ---
I added iv-use for MASKED_LOAD_LANE and the result is
cmp w3, 0
ble .L1
sub w5, w3, #1
mov x4, 0
lsr w5, w5, 1
add w5, w5, 1
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88834
kugan at gcc dot gnu.org changed:
What|Removed |Added
CC||kugan at gcc dot gnu.org
--- C
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88834
ktkachov at gcc dot gnu.org changed:
What|Removed |Added
Keywords||missed-optimization
23 matches
Mail list logo