On 19 Aug 15:01, Richard Henderson wrote:
> >  ;; All vector modes including V?TImode, used in move patterns.
> >  (define_mode_iterator V16
> > -  [(V32QI "TARGET_AVX") V16QI
> > -   (V16HI "TARGET_AVX") V8HI
> > -   (V8SI "TARGET_AVX") V4SI
> > -   (V4DI "TARGET_AVX") V2DI
> > +  [(V64QI "TARGET_AVX512F") (V32QI "TARGET_AVX") V16QI
> > +   (V32HI "TARGET_AVX512F") (V16HI "TARGET_AVX") V8HI
> > +   (V16SI "TARGET_AVX512F") (V8SI "TARGET_AVX") V4SI
> > +   (V8DI "TARGET_AVX512F")  (V4DI "TARGET_AVX") V2DI
> >     (V2TI "TARGET_AVX") V1TI
> > -   (V8SF "TARGET_AVX") V4SF
> > -   (V4DF "TARGET_AVX") V2DF])
> > +   (V16SF "TARGET_AVX512F") (V8SF "TARGET_AVX") V4SF
> > +   (V8DF "TARGET_AVX512F")  (V4DF "TARGET_AVX") V2DF])
> 
> Let's rename this VMOVE, and apply only that change as a separate patch.

Hello,
I think this is kinda obvious.

I've renamed V16 iterator into VMOVE.

ChangeLog entry:

2013-08-20  Kirill Yukhin  <kirill.yuk...@intel.com>

        * config/i386/sse.md (V16): Rename to...
        (VMOVE): this.
        (mov<mode>): Update iterator name.
        (*mov<mode>_internal): Ditto.
        (push<mode>1): Ditto.
        (movmisalign<mode>): Ditto.

Bootstrap passing.

I'll check it in into MT as obvious if nobody objects in 20 hrs.

Thanks, K

---
 gcc/config/i386/sse.md | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/gcc/config/i386/sse.md b/gcc/config/i386/sse.md
index 4397498..5c07dd7 100644
--- a/gcc/config/i386/sse.md
+++ b/gcc/config/i386/sse.md
@@ -96,7 +96,7 @@
 ])
 
 ;; All vector modes including V?TImode, used in move patterns.
-(define_mode_iterator V16
+(define_mode_iterator VMOVE
   [(V32QI "TARGET_AVX") V16QI
    (V16HI "TARGET_AVX") V8HI
    (V8SI "TARGET_AVX") V4SI
@@ -435,8 +435,8 @@
 ;; This is essential for maintaining stable calling conventions.
 
 (define_expand "mov<mode>"
-  [(set (match_operand:V16 0 "nonimmediate_operand")
-       (match_operand:V16 1 "nonimmediate_operand"))]
+  [(set (match_operand:VMOVE 0 "nonimmediate_operand")
+       (match_operand:VMOVE 1 "nonimmediate_operand"))]
   "TARGET_SSE"
 {
   ix86_expand_vector_move (<MODE>mode, operands);
@@ -444,8 +444,8 @@
 })
 
 (define_insn "*mov<mode>_internal"
-  [(set (match_operand:V16 0 "nonimmediate_operand"               "=x,x ,m")
-       (match_operand:V16 1 "nonimmediate_or_sse_const_operand"  "C ,xm,x"))]
+  [(set (match_operand:VMOVE 0 "nonimmediate_operand"               "=x,x ,m")
+       (match_operand:VMOVE 1 "nonimmediate_or_sse_const_operand"  "C ,xm,x"))]
   "TARGET_SSE
    && (register_operand (operands[0], <MODE>mode)
        || register_operand (operands[1], <MODE>mode))"
@@ -586,7 +586,7 @@
 })
 
 (define_expand "push<mode>1"
-  [(match_operand:V16 0 "register_operand")]
+  [(match_operand:VMOVE 0 "register_operand")]
   "TARGET_SSE"
 {
   ix86_expand_push (<MODE>mode, operands[0]);
@@ -594,8 +594,8 @@
 })
 
 (define_expand "movmisalign<mode>"
-  [(set (match_operand:V16 0 "nonimmediate_operand")
-       (match_operand:V16 1 "nonimmediate_operand"))]
+  [(set (match_operand:VMOVE 0 "nonimmediate_operand")
+       (match_operand:VMOVE 1 "nonimmediate_operand"))]
   "TARGET_SSE"
 {
   ix86_expand_vector_move_misalign (<MODE>mode, operands);
-- 
1.7.11.7

Reply via email to