diff options
| author | Uday Bondhugula <bondhugula@google.com> | 2019-03-05 15:05:34 -0800 |
|---|---|---|
| committer | jpienaar <jpienaar@google.com> | 2019-03-29 16:58:35 -0700 |
| commit | 02af8c22df523d7cda4399058e0a0945d54f4972 (patch) | |
| tree | b49de902095777a37a8f109f3506147190134fc3 /mlir/lib/Transforms/MemRefDataFlowOpt.cpp | |
| parent | 73e0297d36452555cb9552827c498c8bde1a5f74 (diff) | |
| download | bcm5719-llvm-02af8c22df523d7cda4399058e0a0945d54f4972.tar.gz bcm5719-llvm-02af8c22df523d7cda4399058e0a0945d54f4972.zip | |
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
Diffstat (limited to 'mlir/lib/Transforms/MemRefDataFlowOpt.cpp')
| -rw-r--r-- | mlir/lib/Transforms/MemRefDataFlowOpt.cpp | 7 |
1 files changed, 4 insertions, 3 deletions
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> 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<LoadOp>([&](OpPointer<LoadOp> loadOp) { forwardStoreToLoad(loadOp); }); + f->walk<LoadOp>( + [&](OpPointer<LoadOp> loadOp) { forwardStoreToLoad(loadOp); }); // Erase all load op's whose results were replaced with store fwd'ed ones. for (auto *loadOp : loadOpsToErase) { |
