https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96744
--- Comment #1 from CVS Commits ---
The master branch has been updated by H.J. Lu :
https://gcc.gnu.org/g:87c753ac241f25d222d46ba1ac66ceba89d6a200
commit r11-2812-g87c753ac241f25d222d46ba1ac66ceba89d6a200
Author: H.J. Lu
Date: Fri Aug 21 09:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96744
--- Comment #2 from Hongtao.liu ---
Created attachment 49107
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=49107&action=edit
Enable spill to mask only under m_core_AVX512
this patch will fail
cat test.c
#include
void
_mm512_2interse
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96744
--- Comment #3 from Uroš Bizjak ---
Created attachment 49112
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=49112&action=edit
Retune mask <-> general moves cost
It looks to me that mask <-> general cost is too low, so the compiler now
pref
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96744
--- Comment #4 from Uroš Bizjak ---
Created attachment 49114
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=49114&action=edit
Double-reg mask moves
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96744
--- Comment #5 from Uroš Bizjak ---
(In reply to Hongtao.liu from comment #2)
> Need to add define_insn for movp2qi/movp2hi?
Yes, this is needed to cover some corner cases. Please see attachment 49114.
However, the patch assumes that avx512vp2
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96744
--- Comment #6 from Uroš Bizjak ---
(In reply to Uroš Bizjak from comment #5)
> However, the patch assumes that avx512vp2intersect implies mavx512dq,
> otherwise there is no direct QImode move from mask register to memory
> available.
This is th
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96744
--- Comment #7 from Hongtao.liu ---
(In reply to Uroš Bizjak from comment #5)
> (In reply to Hongtao.liu from comment #2)
>
> > Need to add define_insn for movp2qi/movp2hi?
>
> Yes, this is needed to cover some corner cases. Please see attachme
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96744
Richard Biener changed:
What|Removed |Added
Ever confirmed|0 |1
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96744
--- Comment #9 from Hongtao.liu ---
(In reply to Hongtao.liu from comment #7)
> (In reply to Uroš Bizjak from comment #5)
> > (In reply to Hongtao.liu from comment #2)
> >
> > > Need to add define_insn for movp2qi/movp2hi?
> >
> > Yes, this is
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96744
--- Comment #10 from Hongtao.liu ---
(In reply to Uroš Bizjak from comment #3)
> Created attachment 49112 [details]
> Retune mask <-> general moves cost
>
> It looks to me that mask <-> general cost is too low, so the compiler now
> prefers thes
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96744
--- Comment #11 from Uroš Bizjak ---
(In reply to Hongtao.liu from comment #9)
> > > However, the patch assumes that avx512vp2intersect implies mavx512dq,
> >
> > Let me check this part.
>
> Confirmed, avx512vp2intersect implies avx512dq.
>
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96744
--- Comment #12 from CVS Commits ---
The master branch has been updated by hongtao Liu :
https://gcc.gnu.org/g:58d6eea0e0754351b399a4b85562f81326a184ad
commit r11-2917-g58d6eea0e0754351b399a4b85562f81326a184ad
Author: liuhongt
Date: Wed Aug
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96744
--- Comment #13 from CVS Commits ---
The releases/gcc-10 branch has been updated by hongtao Liu
:
https://gcc.gnu.org/g:436ee9fec2751826bc48069d4dd320207fd9dfa4
commit r10-8684-g436ee9fec2751826bc48069d4dd320207fd9dfa4
Author: liuhongt
Date:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96744
--- Comment #14 from CVS Commits ---
The master branch has been updated by hongtao Liu :
https://gcc.gnu.org/g:ecc3135a4a264b66c163ef32e815a99746b81800
commit r11-3202-gecc3135a4a264b66c163ef32e815a99746b81800
Author: liuhongt
Date: Mon Aug
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96744
--- Comment #15 from Hongtao.liu ---
Fixed in GCC11.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96744
Richard Biener changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
16 matches
Mail list logo