From 4fe7b16ae3f48a36967e58ab753381cb34295962 Mon Sep 17 00:00:00 2001 From: Christian Sigg Date: Mon, 11 Jan 2021 13:25:23 +0100 Subject: [PATCH] [mlir] Remove unnecessary llvm.mlir.cast in AsyncToLLVM lowering. Reviewed By: ftynse Differential Revision: https://reviews.llvm.org/D94400 --- mlir/lib/Conversion/AsyncToLLVM/AsyncToLLVM.cpp | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/mlir/lib/Conversion/AsyncToLLVM/AsyncToLLVM.cpp b/mlir/lib/Conversion/AsyncToLLVM/AsyncToLLVM.cpp index 92b01ce..4884327 100644 --- a/mlir/lib/Conversion/AsyncToLLVM/AsyncToLLVM.cpp +++ b/mlir/lib/Conversion/AsyncToLLVM/AsyncToLLVM.cpp @@ -945,13 +945,7 @@ public: loc, LLVM::LLVMPointerType::get(llvmValueType), storage.getResult(0)); // Load from the async value storage. - auto loaded = rewriter.create(loc, castedStorage.getResult()); - - // Cast from LLVM type to the expected value type if necessary. This cast - // will become no-op after lowering to LLVM. - if (valueType == loaded.getType()) - return loaded; - return rewriter.create(loc, valueType, loaded); + return rewriter.create(loc, castedStorage.getResult()); } }; -- 2.7.4