https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100866
HaoChen Gui changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100866
--- Comment #16 from CVS Commits ---
The master branch has been updated by HaoChen Gui :
https://gcc.gnu.org/g:9d68cba5eb20442f8075b8f92d1b20a00022852f
commit r13-4423-g9d68cba5eb20442f8075b8f92d1b20a00022852f
Author: Haochen Gui
Date: Wed
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100866
--- Comment #15 from CVS Commits ---
The master branch has been updated by HaoChen Gui :
https://gcc.gnu.org/g:eaba55ffef961c28f6a15d845a4d6b77b8a8bab1
commit r13-3603-geaba55ffef961c28f6a15d845a4d6b77b8a8bab1
Author: Xionghu Luo
Date: Wed
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100866
--- Comment #14 from Segher Boessenkool ---
(In reply to luoxhu from comment #13)
> It is not visible in combine due to the constant data is in *.LC0 and
combine can see things in the constant pool in various ways though (just
like many other
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100866
--- Comment #13 from luoxhu at gcc dot gnu.org ---
It is not visible in combine due to the constant data is in *.LC0 and
UNSPEC_VPERM. Will shelf this and switch to other high priority issues.
pr100866.c.277r.combine:
(note 4 0 20 2 [bb 2]
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100866
--- Comment #12 from Segher Boessenkool ---
(In reply to Bill Schmidt from comment #11)
> Segher, does this fit naturally in combine?
This is just constant folding, combine won't have much to do with it.
It is always better (namely, lower
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100866
--- Comment #11 from Bill Schmidt ---
Segher, does this fit naturally in combine?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100866
--- Comment #10 from Bill Schmidt ---
Right, it would be a good optimization. We've stopped focusing much on P8
optimization work at this point simply because of lack of resources.
The needed transform is to recognize load-xxlnor-vperm as a
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100866
--- Comment #9 from Jens Seifert ---
I know that if I would use vec_perm builtin as an end user, that you then need
to fulfill to the LE specification, but you can always optimize the code as you
like as long as it creates correct results
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100866
--- Comment #8 from luoxhu at gcc dot gnu.org ---
(In reply to Jens Seifert from comment #7)
> Regarding vec_revb for vector unsigned int. I agree that
> revb:
> .LFB0:
> .cfi_startproc
> vspltish %v1,8
> vspltisw %v0,-16
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100866
--- Comment #7 from Jens Seifert ---
Regarding vec_revb for vector unsigned int. I agree that
revb:
.LFB0:
.cfi_startproc
vspltish %v1,8
vspltisw %v0,-16
vrlh %v2,%v2,%v1
vrlw %v2,%v2,%v0
blr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100866
--- Comment #6 from luoxhu at gcc dot gnu.org ---
For V4SI, it is also better to use vector splat and vector rotate operations.
revb:
.LFB0:
.cfi_startproc
vspltish %v1,8
vspltisw %v0,-16
vrlh %v2,%v2,%v1
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100866
--- Comment #5 from luoxhu at gcc dot gnu.org ---
(In reply to Segher Boessenkool from comment #4)
> This PR is specifically about the vec_revb builtin. But yes, we should
> look at what is generated for all other code (having only the builtin
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100866
--- Comment #4 from Segher Boessenkool ---
This PR is specifically about the vec_revb builtin. But yes, we should
look at what is generated for all other code (having only the builtin
generate good code is suboptimal for a generic thing like
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100866
--- Comment #3 from luoxhu at gcc dot gnu.org ---
diff --git a/gcc/config/rs6000/altivec.md b/gcc/config/rs6000/altivec.md
index 097a127be07..35b3f1a0e1a 100644
--- a/gcc/config/rs6000/altivec.md
+++ b/gcc/config/rs6000/altivec.md
@@ -1932,7
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100866
luoxhu at gcc dot gnu.org changed:
What|Removed |Added
CC||luoxhu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100866
Segher Boessenkool changed:
What|Removed |Added
Target|powerpc-*-*-* |powerpc*
17 matches
Mail list logo
