experiments/STT-tensorflow
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

[MLIR][NFC] Adopt FuncOp/Region argument API's.

Browse Source 
- Use FuncOp::getArguments() and Region::getArguments() and friends where possible
  instead of going through the front() block.

PiperOrigin-RevId: 325352975
Change-Id: Ib3dcfed692c0e04c554120a748f82e9efe009b89
This commit is contained in:
Rahul Joshi 2020-08-06 18:31:33 -07:00 committed by TensorFlower Gardener
parent 8c0c1e1730
commit 3b47c2bdea
3 changed files with 6 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
tensorflow/compiler/mlir/hlo/lib/Dialect/mhlo/transforms/lhlo_legalize_to_gpu.cc
View File

@ -147,9 +147,9 @@ class LhloReduceToGPULaunchConverter : public OpConversionPattern<ReduceOp> {
// Now copy over the actual body of the reduction, leaving out the
// terminator.
BlockAndValueMapping mapping;
mapping.map(reduce_op.body().front().getArgument(0), accumulator);
mapping.map(reduce_op.body().front().getArgument(1), rhs);
mapping.map(reduce_op.body().front().getArgument(2), accumulator);
mapping.map(reduce_op.body().getArgument(0), accumulator);
mapping.map(reduce_op.body().getArgument(1), rhs);
mapping.map(reduce_op.body().getArgument(2), accumulator);
for (auto& nested : reduce_op.body().front().without_terminator()) {
auto clone = rewriter.clone(nested, mapping);
for (auto pair : llvm::zip(nested.getResults(), clone->getResults())) {

5
tensorflow/compiler/mlir/lite/transforms/while_loop_outline.cc
View File

@ -80,7 +80,7 @@ void WhileOutlinePass::OutlineWhile(WhileOp while_op) {
// The basic block arguments correspond to values that are loop carried, while
// all those post are loop independent. Initialize extern_values with while_op
// not loop carried operands.
auto num_loop_carried = while_op.cond().front().getNumArguments();
auto num_loop_carried = while_op.cond().getNumArguments();
auto not_carried_operands =
while_op.getOperands().drop_front(num_loop_carried);
extern_values.insert(not_carried_operands.begin(),
@ -124,8 +124,7 @@ void WhileOutlinePass::OutlineWhile(WhileOp while_op) {
// Collect new types.
SmallVector<Type, 4> types;
types.reserve(extra_operands.size() + while_op.getNumOperands());
for (BlockArgument ba : while_op.cond().front().getArguments())
types.push_back(ba.getType());
for (Type type : while_op.cond().getArgumentTypes()) types.push_back(type);
for (Value operand : extern_values) types.push_back(operand.getType());
// Create outline function from region. Optional pass extra arguments through

2
tensorflow/compiler/mlir/tensorflow/translate/import_model.cc
View File

@ -2873,7 +2873,7 @@ void AdjustBoundInputArgTypes(mlir::ModuleOp module) {
mlir::OpBuilder builder(func.getBody());
llvm::SmallVector<mlir::Type, 4> new_input_types;
for (int i = 0, e = func.getNumArguments(); i < e; i++) {
auto arg = func.front().getArgument(i);
auto arg = func.getArgument(i);
auto global_tensor = mlir::tf_saved_model::LookupBoundInputOfType<
mlir::tf_saved_model::GlobalTensorOp>(func, i, symbol_table);
if (global_tensor) {