https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94043
Kewen Lin changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94043
--- Comment #18 from CVS Commits ---
The master branch has been updated by Kewen Lin :
https://gcc.gnu.org/g:bd0f22a8d5caea8905f38ff1fafce31c1b7d33ad
commit r10-7491-gbd0f22a8d5caea8905f38ff1fafce31c1b7d33ad
Author: Kewen Lin
Date: Tue Mar
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94043
--- Comment #17 from Richard Biener ---
(In reply to Kewen Lin from comment #16)
> Created attachment 48125 [details]
> untested patch
LGTM
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94043
Kewen Lin changed:
What|Removed |Added
Attachment #48122|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94043
--- Comment #15 from rguenther at suse dot de ---
On Thu, 26 Mar 2020, linkw at gcc dot gnu.org wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94043
>
> --- Comment #14 from Kewen Lin ---
> (In reply to Richard Biener from comment #13)
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94043
--- Comment #14 from Kewen Lin ---
(In reply to Richard Biener from comment #13)
>
> + /* Find all SSA NAMEs in stmts which is defined in current loop,
> create
> +PHIs for them, and replace them with phi results accordingly. */
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94043
--- Comment #13 from Richard Biener ---
(In reply to Kewen Lin from comment #12)
> Created attachment 48122 [details]
> ppc64le tested patch
>
> Thanks Richi!
>
> A patch draft attached to ensure on the right track, also
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94043
--- Comment #12 from Kewen Lin ---
Created attachment 48122
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=48122=edit
ppc64le tested patch
Thanks Richi!
A patch draft attached to ensure on the right track, also
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94043
--- Comment #11 from rguenther at suse dot de ---
On March 24, 2020 3:59:35 AM GMT+01:00, "linkw at gcc dot gnu.org"
wrote:
>https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94043
>
>--- Comment #10 from Kewen Lin ---
>(In reply to Richard Biener
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94043
--- Comment #10 from Kewen Lin ---
(In reply to Richard Biener from comment #9)
> OK, so it's indeed vectorizable_live_operation not paying attention to
> loop-closed SSA form.
>
> What it should do before building the lane extract is create a
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94043
--- Comment #9 from Richard Biener ---
OK, so it's indeed vectorizable_live_operation not paying attention to
loop-closed SSA form.
What it should do before building the lane extract is create a _new_
loop-closed PHI node for the vectorized def
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94043
--- Comment #8 from Kewen Lin ---
> It's most likely either SCEV or expand_simple_operations looking throuhg
> the single-arg PHI (which we should avoid for LC PHI nodes)
Thanks Richi, I found the loop-closed PHI form was broken after we
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94043
--- Comment #7 from rguenther at suse dot de ---
On Fri, 20 Mar 2020, linkw at gcc dot gnu.org wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94043
>
> --- Comment #6 from Kewen Lin ---
> (In reply to rguent...@suse.de from comment #5)
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94043
--- Comment #6 from Kewen Lin ---
(In reply to rguent...@suse.de from comment #5)
> On Fri, 20 Mar 2020, linkw at gcc dot gnu.org wrote:
>
> > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94043
> >
> > --- Comment #4 from Kewen Lin ---
> >
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94043
--- Comment #5 from rguenther at suse dot de ---
On Fri, 20 Mar 2020, linkw at gcc dot gnu.org wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94043
>
> --- Comment #4 from Kewen Lin ---
> This was just exposed from my commit, it can
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94043
--- Comment #4 from Kewen Lin ---
This was just exposed from my commit, it can also be reproduced without my
commit but with -fno-vect-cost-model.
Some loops we have for this case:
;; Loop 1
;; header 3, latch 10
;; depth 1, outer 0
;;
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94043
Kewen Lin changed:
What|Removed |Added
Assignee|unassigned at gcc dot gnu.org |linkw at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94043
Martin Liška changed:
What|Removed |Added
Last reconfirmed||2020-03-17
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94043
Jakub Jelinek changed:
What|Removed |Added
Target Milestone|9.3 |9.4
--- Comment #1 from Jakub Jelinek
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94043
Richard Biener changed:
What|Removed |Added
CC||rguenth at gcc dot gnu.org
Target
20 matches
Mail list logo