[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