From 9191efdfc18eb4f6b84b9fc3a8979797fe0dafd5 Mon Sep 17 00:00:00 2001 From: Sean Silva Date: Mon, 18 May 2020 12:51:49 -0700 Subject: [PATCH] Fix typo in LowerLinalgLoopDimOps The legality condition was reversed. It's unclear to me why this didn't cause "failed to legalize". --- lib/E2E/E2E.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/E2E/E2E.cpp b/lib/E2E/E2E.cpp index 086f0721b..102024764 100644 --- a/lib/E2E/E2E.cpp +++ b/lib/E2E/E2E.cpp @@ -222,7 +222,7 @@ class LowerLinalgLoopDimOps // TODO: We only need this because we use `dim` ops for the memref // ABI. Once we layer that out into our own runtime types, we can // remove this. - return op.getOperand().getDefiningOp(); + return !op.getOperand().getDefiningOp(); }); target.addLegalOp(); if (failed(applyPartialConversion(func, target, patterns))) {