[PATCH] D63507: Teach TableGen Intrin Emitter to handle LLVMPointerType<llvm_any_ty>
Erich Keane via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 25 17:09:44 PDT 2019
This revision was automatically updated to reflect the committed changes.
Closed by commit rGad28befcf49c: Teach TableGen Intrin Emitter to handle LLVMPointerType<llvm_any_ty> (authored by erichkeane).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D63507/new/
https://reviews.llvm.org/D63507
Files:
llvm/test/TableGen/intrinsic-pointer-to-any.td
llvm/utils/TableGen/IntrinsicEmitter.cpp
Index: llvm/utils/TableGen/IntrinsicEmitter.cpp
===================================================================
--- llvm/utils/TableGen/IntrinsicEmitter.cpp
+++ llvm/utils/TableGen/IntrinsicEmitter.cpp
@@ -372,6 +372,9 @@
unsigned Tmp = 0;
switch (getValueType(R->getValueAsDef("VT"))) {
default: break;
+ case MVT::iPTR:
+ UpdateArgCodes(R->getValueAsDef("ElTy"), ArgCodes, NumInserted, Mapping);
+ break;
case MVT::iPTRAny:
++Tmp;
LLVM_FALLTHROUGH;
Index: llvm/test/TableGen/intrinsic-pointer-to-any.td
===================================================================
--- /dev/null
+++ llvm/test/TableGen/intrinsic-pointer-to-any.td
@@ -0,0 +1,53 @@
+// RUN: llvm-tblgen -gen-intrinsic-impl %s | FileCheck %s
+
+// This test is validating that it an Intrinsic with an LLVMPointerType to
+// llvm_any_ty still properly work after r363233. That patch rewrote the
+// substitution handling code in the Intrinsic Emitter, and didn't consider this
+// case, so TableGen would hit an assertion in EncodeFixedType that was checking
+// to ensure that the substitution being processed was correctly replaced.
+
+class IntrinsicProperty;
+class SDNodeProperty;
+
+class ValueType<int size, int value> {
+ string Namespace = "MVT";
+ int Size = size;
+ int Value = value;
+}
+
+def iPTR : ValueType<0 , 254>;
+def Any : ValueType<0 , 255>;
+
+class LLVMType<ValueType vt> {
+ ValueType VT = vt;
+ int isAny = 0;
+}
+
+
+class Intrinsic<list<LLVMType> ret_types> {
+ string LLVMName = "";
+ string TargetPrefix = ""; // Set to a prefix for target-specific intrinsics.
+ list<LLVMType> RetTypes = ret_types;
+ list<LLVMType> ParamTypes = [];
+ list<IntrinsicProperty> IntrProperties = [];
+ list<SDNodeProperty> Properties = [];
+ bit isTarget = 0;
+}
+
+class LLVMQualPointerType<LLVMType elty>
+ : LLVMType<iPTR>{
+ LLVMType ElTy = elty;
+ int AddrSpace = 0;
+}
+
+class LLVMPointerType<LLVMType elty>
+ : LLVMQualPointerType<elty>;
+
+let isAny = 1 in {
+ def llvm_any_ty : LLVMType<Any>;
+}
+def i8 : ValueType<8, 3>;
+def llvm_i8_ty : LLVMType<i8>;
+
+def int_has_ptr_to_any : Intrinsic<[LLVMPointerType<llvm_any_ty>, llvm_i8_ty]>;
+// CHECK: /* 0 */ 21, 14, 15, 0, 2, 0
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D63507.206553.patch
Type: text/x-patch
Size: 2235 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190626/1f854282/attachment.bin>
More information about the llvm-commits
mailing list