[PATCH] D72533: [mlir] Add a signedness semantics bit to IntegerType

River Riddle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 29 23:41:00 PST 2020


rriddle added inline comments.


================
Comment at: mlir/include/mlir/IR/OpBase.td:312
 // Any integer type irrespective of its width.
 def AnyInteger : Type<CPred<"$_self.isa<IntegerType>()">, "integer">;
 
----------------
Can you go through and make sure all of the current usages of IntegerType use the signless variant? Or is this something planned for a followup?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72533





More information about the llvm-commits mailing list