[PATCH] D78144: [mlir] Fix assert on signed integer type in EDSC

[Diego Caballero via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rG2f8b4545f496: [mlir] Fix assert on signed integer type in EDSC (authored by adstraw, committed by dcaballe).

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78144

Files:
  mlir/lib/Dialect/Affine/EDSC/Builders.cpp


Index: mlir/lib/Dialect/Affine/EDSC/Builders.cpp
===================================================================
--- mlir/lib/Dialect/Affine/EDSC/Builders.cpp
+++ mlir/lib/Dialect/Affine/EDSC/Builders.cpp
@@ -156,7 +156,7 @@
     return createBinaryHandle<FOp>(lhs, rhs);
   } else if (thisType.isa<VectorType>() || thisType.isa<TensorType>()) {
     auto aggregateType = thisType.cast<ShapedType>();
-    if (aggregateType.getElementType().isSignedInteger())
+    if (aggregateType.getElementType().isSignlessInteger())
       return createBinaryHandle<IOp>(lhs, rhs);
     else if (aggregateType.getElementType().isa<FloatType>())
       return createBinaryHandle<FOp>(lhs, rhs);
@@ -225,7 +225,7 @@
   (void)lhsType;
   (void)rhsType;
   assert(lhsType == rhsType && "cannot mix types in operators");
-  assert((lhsType.isa<IndexType>() || lhsType.isSignedInteger()) &&
+  assert((lhsType.isa<IndexType>() || lhsType.isSignlessInteger()) &&
          "only integer comparisons are supported");
 
   auto op = ScopedContext::getBuilder().create<CmpIOp>(


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D78144.257551.patch
Type: text/x-patch
Size: 1060 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200414/6c4b943c/attachment.bin>