Add support for vector types and vector constants.

src/IceDefs.h

 //===- subzero/src/IceDefs.h - Common Subzero declaraions -------*- C++ -*-===//
 //
 //                        The Subzero Code Generator
 //
 // This file is distributed under the University of Illinois Open Source
 // License. See LICENSE.TXT for details.
 //
 //===----------------------------------------------------------------------===//
 //
 // This file declares various useful types and classes that have
@@ skipping 11 matching lines @@
 #include <cstdio>     // snprintf
 #include <functional> // std::less
 #include <list>
 #include <map>
 #include <set>
 #include <string>
 #include <vector>
 
 #include "llvm/ADT/BitVector.h"
 #include "llvm/ADT/SmallBitVector.h"
+#include "llvm/ADT/SmallVector.h"
 #include "llvm/ADT/STLExtras.h"
 #include "llvm/Support/Casting.h"
 #include "llvm/Support/Compiler.h" // LLVM_STATIC_ASSERT
 #include "llvm/Support/raw_ostream.h"
 #include "llvm/Support/Timer.h"
 
 // Roll our own static_assert<> in the absence of C++11.  TODO: change
 // to static_assert<> with C++11.
 template <bool> struct staticAssert;
 template <> struct staticAssert<true> {}; // only true is defined
@@ skipping 16 matching lines @@
 
 // TODO: Switch over to LLVM's ADT container classes.
 // http://llvm.org/docs/ProgrammersManual.html#picking-the-right-data-structure-for-a-task
 typedef std::string IceString;
 typedef std::list<Inst *> InstList;
 typedef std::list<InstPhi *> PhiList;
 typedef std::vector<Variable *> VarList;
 typedef std::vector<CfgNode *> NodeList;
 typedef std::vector<Constant *> ConstantList;
 
+const unsigned VECT128_BYTES = 16;
+// TODO(stichnot): Switch Vect128 to std::array after C++11
+typedef llvm::SmallVector<char, VECT128_BYTES> Vect128;

[JF, 2014/06/30 17:48:50]

You should use uint8_t here. I'd also name the class Vect128Bit or something
similar, it gets confusing to have numbers without a unit otherwise.

[wala, 2014/06/30 22:13:24]

Unless vector constants are supported, this type is unnecessary.

I'll keep your naming suggestion in mind for the future.

+typedef llvm::SmallVector<bool, VECT128_BYTES> BitVect;

[JF, 2014/06/30 17:48:50]

Can you add a comment that explains this type?

[wala, 2014/06/30 22:13:24]

Unless vector constants are supported, this type is unnecessary.

+
 // SizeT is for holding small-ish limits like number of source
 // operands in an instruction.  It is used instead of size_t (which
 // may be 64-bits wide) when we want to save space.
 typedef uint32_t SizeT;
 
 // InstNumberT is for holding an instruction number.  Instruction
 // numbers are used for representing Variable live ranges.
 typedef int32_t InstNumberT;
 
 enum LivenessMode {
@@ skipping 62 matching lines @@
 
 private:
   const llvm::TimeRecord Start;
   Timer(const Timer &) LLVM_DELETED_FUNCTION;
   Timer &operator=(const Timer &) LLVM_DELETED_FUNCTION;
 };
 
 } // end of namespace Ice
 
 #endif // SUBZERO_SRC_ICEDEFS_H