[PATCH] [PPC64] Add v1i128 Vector Type

hfinkel at anl.gov hfinkel at anl.gov
Sat Apr 11 19:20:56 PDT 2015


================
Comment at: include/llvm/CodeGen/MachineValueType.h:29
@@ -28,2 +28,3 @@
   /// type can be represented by an MVT.
-  class MVT {
+
+class MVT {
----------------
Don't add blank line.

================
Comment at: include/llvm/IR/Constants.h:676
@@ -675,2 +675,3 @@
   static Constant *get(LLVMContext &Context, ArrayRef<uint64_t> Elts);
+  static Constant *get(LLVMContext &Context, ArrayRef<__uint128_t> Elts);
   static Constant *get(LLVMContext &Context, ArrayRef<float> Elts);
----------------
We have don't have a standard __uint128_t type, so you can't use it.

Also, you don't define these anywhere. Do you need them?

http://reviews.llvm.org/D8564

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list