From 02af8c22df523d7cda4399058e0a0945d54f4972 Mon Sep 17 00:00:00 2001 From: Uday Bondhugula Date: Tue, 5 Mar 2019 15:05:34 -0800 Subject: Change Pass:getFunction() to return pointer instead of ref - NFC - change this for consistency - everything else similar takes/returns a Function pointer - the FuncBuilder ctor, Block/Value/Instruction::getFunction(), etc. - saves a whole bunch of &s everywhere PiperOrigin-RevId: 236928761 --- mlir/lib/Transforms/MemRefDataFlowOpt.cpp | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) (limited to 'mlir/lib/Transforms/MemRefDataFlowOpt.cpp') diff --git a/mlir/lib/Transforms/MemRefDataFlowOpt.cpp b/mlir/lib/Transforms/MemRefDataFlowOpt.cpp index 4fc544074c0..f48f90923ce 100644 --- a/mlir/lib/Transforms/MemRefDataFlowOpt.cpp +++ b/mlir/lib/Transforms/MemRefDataFlowOpt.cpp @@ -211,8 +211,8 @@ void MemRefDataFlowOpt::forwardStoreToLoad(OpPointer loadOp) { void MemRefDataFlowOpt::runOnFunction() { // Only supports single block functions at the moment. - Function &f = getFunction(); - if (f.getBlocks().size() != 1) { + Function *f = getFunction(); + if (f->getBlocks().size() != 1) { markAllAnalysesPreserved(); return; } @@ -224,7 +224,8 @@ void MemRefDataFlowOpt::runOnFunction() { memrefsToErase.clear(); // Walk all load's and perform load/store forwarding. - f.walk([&](OpPointer loadOp) { forwardStoreToLoad(loadOp); }); + f->walk( + [&](OpPointer loadOp) { forwardStoreToLoad(loadOp); }); // Erase all load op's whose results were replaced with store fwd'ed ones. for (auto *loadOp : loadOpsToErase) { -- cgit v1.2.3