From fc5f10c5c596466cb9352548869025e936f31c87 Mon Sep 17 00:00:00 2001 From: Stella Laurenzo Date: Fri, 19 Jun 2020 10:38:51 -0700 Subject: [PATCH] Bump revision and fix issues. * llvm revision = 4836188ad9b3334b0c1e055d45ccaa54ed797e4b * iree revision = 091482e8fdf599d6cb5c701d5b3ccb27fc66c014 --- README.md | 4 ++-- lib/E2E/LowerRankedShapes.cpp | 4 ++-- lib/E2E/LowerToLLVM.cpp | 4 ++-- lib/E2E/LowerToMemRefABI.cpp | 6 +++--- 4 files changed, 9 insertions(+), 9 deletions(-) diff --git a/README.md b/README.md index 31af46ef9..56ecbfa8a 100644 --- a/README.md +++ b/README.md @@ -27,7 +27,7 @@ export LDFLAGS=-fuse-ld=$(which ld.lld-$LLVM_VERSION) export LLVM_SRC_DIR=/path/to/llvm-project # Check out last known good commit. -(cd $LLVM_SRC_DIR && git checkout 52cae05e087b3d4fd02849fc37c387c720055ffb) +(cd $LLVM_SRC_DIR && git checkout 4836188ad9b3334b0c1e055d45ccaa54ed797e4b) ./tools/install_mlir.sh ./tools/cmake_configure.sh @@ -70,7 +70,7 @@ Notes: * Python sources are symlinked to the output directory at configure time. Adding sources will require a reconfigure. Editing should not. * It is a very common issue to have both python 2.7 (aka. "python") and python - 3.x (aka. "python3") on a system at a time (and we can only hope that one + 3.x (aka. "python3") on a system at a time (and we can only hope that one day this ends). Since the native library at development time binds to a specific version, if you try to run with a different python, you will get an error about the "native" module not being found. diff --git a/lib/E2E/LowerRankedShapes.cpp b/lib/E2E/LowerRankedShapes.cpp index e1e05b915..08450496e 100644 --- a/lib/E2E/LowerRankedShapes.cpp +++ b/lib/E2E/LowerRankedShapes.cpp @@ -26,7 +26,7 @@ public: LogicalResult matchAndRewrite(shape::BroadcastOp op, ArrayRef operands, ConversionPatternRewriter &rewriter) const override { - shape::BroadcastOp::OperandAdaptor adaptor(operands); + shape::BroadcastOp::Adaptor adaptor(operands); auto lhs = adaptor.lhs().getDefiningOp(); auto rhs = adaptor.rhs().getDefiningOp(); if (!lhs || !rhs) @@ -88,7 +88,7 @@ public: LogicalResult matchAndRewrite(tcp::GetExtentOp op, ArrayRef operands, ConversionPatternRewriter &rewriter) const override { - tcp::GetExtentOp::OperandAdaptor adaptor(operands); + tcp::GetExtentOp::Adaptor adaptor(operands); auto fromExtents = adaptor.shape().getDefiningOp(); if (!fromExtents) return rewriter.notifyMatchFailure(op, "not a from_extents op"); diff --git a/lib/E2E/LowerToLLVM.cpp b/lib/E2E/LowerToLLVM.cpp index 815fe4c45..7605a2604 100644 --- a/lib/E2E/LowerToLLVM.cpp +++ b/lib/E2E/LowerToLLVM.cpp @@ -28,7 +28,7 @@ public: LogicalResult matchAndRewrite(tcp::AbortIfOp op, ArrayRef operands, ConversionPatternRewriter &rewriter) const override { - tcp::AbortIfOp::OperandAdaptor adaptor(operands); + tcp::AbortIfOp::Adaptor adaptor(operands); rewriter.replaceOpWithNewOp(op, abortIfFunc, adaptor.pred()); return success(); } @@ -67,7 +67,7 @@ class LowerToLLVM : public LowerToLLVMBase { populateStdToLLVMConversionPatterns(converter, patterns); patterns.insert(abortIfFunc); - if (failed(applyFullConversion(module, target, patterns, &converter))) { + if (failed(applyFullConversion(module, target, patterns))) { return signalPassFailure(); } } diff --git a/lib/E2E/LowerToMemRefABI.cpp b/lib/E2E/LowerToMemRefABI.cpp index df0164298..c33f8666f 100644 --- a/lib/E2E/LowerToMemRefABI.cpp +++ b/lib/E2E/LowerToMemRefABI.cpp @@ -28,7 +28,7 @@ public: LogicalResult matchAndRewrite(TensorStoreOp op, ArrayRef operands, ConversionPatternRewriter &rewriter) const override { - TensorStoreOp::OperandAdaptor adaptor(operands); + TensorStoreOp::Adaptor adaptor(operands); // The tensor has been converted to an unranked memref. We need to cast // it to the original memref type and copy it to the destination. // @@ -52,7 +52,7 @@ public: LogicalResult matchAndRewrite(TensorLoadOp op, ArrayRef operands, ConversionPatternRewriter &rewriter) const override { - TensorLoadOp::OperandAdaptor adaptor(operands); + TensorLoadOp::Adaptor adaptor(operands); auto type = UnrankedMemRefType::get(op.getType().getElementType(), 0); // TODO: This won't work. The LLVM unranked memref calling convention // doesn't allow returning an unranked memref becuase it lowers it to @@ -93,7 +93,7 @@ public: LogicalResult matchAndRewrite(shape::ShapeOfOp op, ArrayRef operands, ConversionPatternRewriter &rewriter) const override { - shape::ShapeOfOp::OperandAdaptor adaptor(operands); + shape::ShapeOfOp::Adaptor adaptor(operands); auto tensorType = op.arg().getType().cast(); auto rankedMemRefType = MemRefType::get(tensorType.getShape(), tensorType.getElementType());