https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106022
--- Comment #18 from CVS Commits ---
The master branch has been updated by H.J. Lu :
https://gcc.gnu.org/g:f3a5e75cb66dc96efca7d494fe6060746c88acb1
commit r13-1415-gf3a5e75cb66dc96efca7d494fe6060746c88acb1
Author: H.J. Lu
Date: Fri Jun 17
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106022
--- Comment #17 from Hongtao.liu ---
(In reply to H.J. Lu from comment #16)
> Created attachment 53227 [details]
> The v4 patch
>
> Here is a different approach by supporting 2/4/8 byte constant vector stores.
It LGTM.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106022
H.J. Lu changed:
What|Removed |Added
Attachment #53186|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106022
--- Comment #15 from rguenther at suse dot de ---
On Mon, 27 Jun 2022, hjl.tools at gmail dot com wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106022
>
> --- Comment #14 from H.J. Lu ---
> (In reply to rguent...@suse.de from comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106022
--- Comment #14 from H.J. Lu ---
(In reply to rguent...@suse.de from comment #13)
> On Fri, 24 Jun 2022, hjl.tools at gmail dot com wrote:
>
> > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106022
> >
> > --- Comment #12 from H.J. Lu ---
> >
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106022
--- Comment #13 from rguenther at suse dot de ---
On Fri, 24 Jun 2022, hjl.tools at gmail dot com wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106022
>
> --- Comment #12 from H.J. Lu ---
> (In reply to Richard Biener from comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106022
--- Comment #12 from H.J. Lu ---
(In reply to Richard Biener from comment #11)
> (In reply to H.J. Lu from comment #9)
> > (In reply to Richard Biener from comment #8)
> > > (In reply to H.J. Lu from comment #6)
> > > > Created attachment 53169
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106022
--- Comment #11 from Richard Biener ---
(In reply to H.J. Lu from comment #9)
> (In reply to Richard Biener from comment #8)
> > (In reply to H.J. Lu from comment #6)
> > > Created attachment 53169 [details]
> > > A patch
> > >
> > > This
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106022
H.J. Lu changed:
What|Removed |Added
Attachment #53171|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106022
--- Comment #9 from H.J. Lu ---
(In reply to Richard Biener from comment #8)
> (In reply to H.J. Lu from comment #6)
> > Created attachment 53169 [details]
> > A patch
> >
> > This patch multiplies the vector store cost by the number of scalar
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106022
--- Comment #8 from Richard Biener ---
(In reply to H.J. Lu from comment #6)
> Created attachment 53169 [details]
> A patch
>
> This patch multiplies the vector store cost by the number of scalar elements
> in
> a word to properly compare
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106022
--- Comment #7 from H.J. Lu ---
Created attachment 53171
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=53171=edit
The v2 patch
Handle vector store.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106022
--- Comment #6 from H.J. Lu ---
Created attachment 53169
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=53169=edit
A patch
This patch multiplies the vector store cost by the number of scalar elements in
a word to properly compare scalar
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106022
--- Comment #5 from Hongtao.liu ---
(In reply to Richard Biener from comment #3)
> We already put the const_vector into the constant pool at RTL expansion time,
> possibly because the move patterns do not allow vector constants fitting into
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106022
--- Comment #4 from Richard Biener ---
(In reply to H.J. Lu from comment #1)
> SLP thinks that it needs 4 stores to store 4 bytes of integer constant.
> But it takes only 1 4-byte store.
Yes, SLP looks at the GIMPLE IL, if you disable
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106022
Richard Biener changed:
What|Removed |Added
Target Milestone|--- |12.2
Ever confirmed|0
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106022
--- Comment #2 from Hongtao.liu ---
(In reply to H.J. Lu from comment #1)
> SLP thinks that it needs 4 stores to store 4 bytes of integer constant.
> But it takes only 1 4-byte store.
I think we can simplify that at combine.
Failed to match
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106022
--- Comment #1 from H.J. Lu ---
SLP thinks that it needs 4 stores to store 4 bytes of integer constant.
But it takes only 1 4-byte store.
18 matches
Mail list logo
