diff mbox

[AArch64] Remove unused VRL2/3/4 iterator values

Message ID 55BA3D6E.8050101@arm.com
State New
Headers show

Commit Message

Alan Lawrence July 30, 2015, 3:06 p.m. UTC 
James Greenhalgh wrote:
>>
>>  (define_mode_attr VRL2 [(V8QI "V32QI") (V4HI "V16HI")
>>  			(V2SI "V8SI")  (V2SF "V8SF")
>> -			(DI   "V4DI")  (DF   "V4DF")
>> -			(V16QI "V32QI") (V8HI "V16HI")
>> -			(V4SI "V8SI")  (V4SF "V8SF")
>> -			(V2DI "V4DI")  (V2DF "V4DF")])
>> +			(DI   "V4DI")  (DF   "V4DF")]
> 
> These all look to be missing a closing ')'.
> 
> Thanks,
> James
> 

Erm, yes. Clearly that is not the version of the patch I tested....it should 
have been the attached. Sorry!

(bootstrapped + check-gcc on aarch64-none-linux-gnu)

Comments

James Greenhalgh July 30, 2015, 3:13 p.m. UTC | #1 
On Thu, Jul 30, 2015 at 04:06:22PM +0100, Alan Lawrence wrote:
> James Greenhalgh wrote:
> >>
> >>  (define_mode_attr VRL2 [(V8QI "V32QI") (V4HI "V16HI")
> >>  			(V2SI "V8SI")  (V2SF "V8SF")
> >> -			(DI   "V4DI")  (DF   "V4DF")
> >> -			(V16QI "V32QI") (V8HI "V16HI")
> >> -			(V4SI "V8SI")  (V4SF "V8SF")
> >> -			(V2DI "V4DI")  (V2DF "V4DF")])
> >> +			(DI   "V4DI")  (DF   "V4DF")]
> > 
> > These all look to be missing a closing ')'.
> > 
> > Thanks,
> > James
> > 
> 
> Erm, yes. Clearly that is not the version of the patch I tested....it should 
> have been the attached. Sorry!
> 
> (bootstrapped + check-gcc on aarch64-none-linux-gnu)

Yup, this one is OK.

Thanks,
James
diff mbox

Patch

diff --git a/gcc/config/aarch64/iterators.md b/gcc/config/aarch64/iterators.md
index b19d3d743a733303371341d9f323f47ed83618c3..eaaac09b928d3312051e52a7ee6540fde813b373 100644
--- a/gcc/config/aarch64/iterators.md
+++ b/gcc/config/aarch64/iterators.md
@@ -540,24 +540,15 @@ 
 
 (define_mode_attr VRL2 [(V8QI "V32QI") (V4HI "V16HI")
 			(V2SI "V8SI")  (V2SF "V8SF")
-			(DI   "V4DI")  (DF   "V4DF")
-			(V16QI "V32QI") (V8HI "V16HI")
-			(V4SI "V8SI")  (V4SF "V8SF")
-			(V2DI "V4DI")  (V2DF "V4DF")])
+			(DI   "V4DI")  (DF   "V4DF")])
 
 (define_mode_attr VRL3 [(V8QI "V48QI") (V4HI "V24HI")
 			(V2SI "V12SI")  (V2SF "V12SF")
-			(DI   "V6DI")  (DF   "V6DF")
-			(V16QI "V48QI") (V8HI "V24HI")
-			(V4SI "V12SI")  (V4SF "V12SF")
-			(V2DI "V6DI")  (V2DF "V6DF")])
+			(DI   "V6DI")  (DF   "V6DF")])
 
 (define_mode_attr VRL4 [(V8QI "V64QI") (V4HI "V32HI")
 			(V2SI "V16SI")  (V2SF "V16SF")
-			(DI   "V8DI")  (DF   "V8DF")
-			(V16QI "V64QI") (V8HI "V32HI")
-			(V4SI "V16SI")  (V4SF "V16SF")
-			(V2DI "V8DI")  (V2DF "V8DF")])
+			(DI   "V8DI")  (DF   "V8DF")])
 
 (define_mode_attr VSTRUCT_DREG [(OI "TI") (CI "EI") (XI "OI")])