[ARM] Refactor NEON binary ops in assembler.

src/arm/constants-arm.h

 // Copyright 2011 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef V8_ARM_CONSTANTS_ARM_H_
 #define V8_ARM_CONSTANTS_ARM_H_
 
 #include <stdint.h>
 
 #include "src/base/logging.h"
@@ skipping 309 matching lines @@
 
 // Coprocessor load/store operand size.
 enum LFlag {
   Long  = 1 << 22,  // Long load/store coprocessor.
   Short = 0 << 22   // Short load/store coprocessor.
 };
 
 
 // NEON data type
 enum NeonDataType {
-  NeonS8 = 0x1,             // U = 0, imm3 = 0b001
-  NeonS16 = 0x2,            // U = 0, imm3 = 0b010
-  NeonS32 = 0x4,            // U = 0, imm3 = 0b100
-  NeonU8 = 1 << 24 | 0x1,   // U = 1, imm3 = 0b001
-  NeonU16 = 1 << 24 | 0x2,  // U = 1, imm3 = 0b010
-  NeonU32 = 1 << 24 | 0x4,  // U = 1, imm3 = 0b100
-  NeonDataTypeSizeMask = 0x7,
-  NeonDataTypeUMask = 1 << 24
+  NeonS8 = 0,
+  NeonS16 = 1,
+  NeonS32 = 2,
+  // Gap to make it easier to extract U and size.
+  NeonU8 = 4,
+  NeonU16 = 5,
+  NeonU32 = 6
 };
 
+inline int NeonU(NeonDataType dt) { return static_cast<int>(dt) >> 2; }
+inline int NeonSz(NeonDataType dt) { return static_cast<int>(dt) & 0x3; }
+
 enum NeonListType {
   nlt_1 = 0x7,
   nlt_2 = 0xA,
   nlt_3 = 0x6,
   nlt_4 = 0x2
 };
 
 enum NeonSize {
   Neon8 = 0x0,
   Neon16 = 0x1,
@@ skipping 372 matching lines @@
 
  private:
   static const char* names_[kNumVFPRegisters];
 };
 
 
 }  // namespace internal
 }  // namespace v8
 
 #endif  // V8_ARM_CONSTANTS_ARM_H_