IntrinsicEmitter: Change IntrinsicsToAttributesMap from uint8_t[] to uint16_t[]
authorFangrui Song <i@maskray.me>
Thu, 28 Jan 2021 04:34:35 +0000 (20:34 -0800)
committerFangrui Song <i@maskray.me>
Thu, 28 Jan 2021 04:34:35 +0000 (20:34 -0800)
We need at least 252 UniqAttributes now, which will soon overflow.
Actually with downstream backends we can easily use up the last few values.
So bump to uint16_t.

llvm/utils/TableGen/IntrinsicEmitter.cpp

index 4be0d90..978d24c 100644 (file)
@@ -638,13 +638,13 @@ void IntrinsicEmitter::EmitAttributes(const CodeGenIntrinsicTable &Ints,
       std::max(maxArgAttrs, unsigned(intrinsic.ArgumentAttributes.size()));
     unsigned &N = UniqAttributes[&intrinsic];
     if (N) continue;
-    assert(AttrNum < 256 && "Too many unique attributes for table!");
     N = ++AttrNum;
+    assert(N < 65536 && "Too many unique attributes for table!");
   }
 
   // Emit an array of AttributeList.  Most intrinsics will have at least one
   // entry, for the function itself (index ~1), which is usually nounwind.
-  OS << "  static const uint8_t IntrinsicsToAttributesMap[] = {\n";
+  OS << "  static const uint16_t IntrinsicsToAttributesMap[] = {\n";
 
   for (unsigned i = 0, e = Ints.size(); i != e; ++i) {
     const CodeGenIntrinsic &intrinsic = Ints[i];