[PATCH] D146914: Migrate `IIT_Info` into `Intrinsics.td`

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 30 11:39:30 PDT 2023


arsenm added inline comments.


================
Comment at: llvm/utils/TableGen/IntrinsicEmitter.cpp:193
+    auto Number = Rec->getValueAsInt("Number");
+    assert(0 <= Number && Number < 256 && "IIT_Info.Number should be uint8_t");
+    assert(RecsByNumber[Number].empty() && "Duplicate IIT_Info.Number");
----------------
assert(isUInt<8>(Number))


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D146914/new/

https://reviews.llvm.org/D146914



More information about the llvm-commits mailing list