James Greenhalgh wrote:
On Tue, Jul 28, 2015 at 12:25:40PM +0100, Alan Lawrence wrote:
I'd have preferred the unrelated changes here as separate patches. If you
pull them out, they are OK to commit independent of this patch.
Done (r226352 and r226353).
Ah ok, I see what is going on here... None of these are actually used for
the 128-bit vector modes, so missing entries or incorrect entries for
V8HFmode don't matter.
Hah, yes, you are right. Can I propose the attached patch as a first step, and
then I'll repost what's left from my original plus the next patch (still
currently fixing after rebasing over Charles' patches)...
bootstrapped + check-gcc on aarch64-none-linux-gnu.
gcc/ChangeLog:
* config/aarch64/iterators.md (VRL2, VRL3, VRL4): Remove values for
128-bit vector modes.
diff --git a/gcc/config/aarch64/iterators.md b/gcc/config/aarch64/iterators.md
index b19d3d743a733303371341d9f323f47ed83618c3..37cb6938151ff8ce0b7e1bff18e575cf052c6a77 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")])
