diff options
| author | River Riddle <riverriddle@google.com> | 2019-09-20 11:36:49 -0700 |
|---|---|---|
| committer | A. Unique TensorFlower <gardener@tensorflow.org> | 2019-09-20 11:37:21 -0700 |
| commit | 2797517ecf89df597a49d670b633cae3de6c4c5a (patch) | |
| tree | 2da2a7381d0242e353b433d30eaa971903c225b1 /mlir/lib/Dialect/LoopOps | |
| parent | d8fda38ceac490ef47a4b8bf1a692030dd2d92e8 (diff) | |
| download | bcm5719-llvm-2797517ecf89df597a49d670b633cae3de6c4c5a.tar.gz bcm5719-llvm-2797517ecf89df597a49d670b633cae3de6c4c5a.zip | |
NFC: Pass OpAsmParser by reference instead of by pointer.
MLIR follows the LLVM style of pass-by-reference.
PiperOrigin-RevId: 270315612
Diffstat (limited to 'mlir/lib/Dialect/LoopOps')
| -rw-r--r-- | mlir/lib/Dialect/LoopOps/LoopOps.cpp | 42 |
1 files changed, 21 insertions, 21 deletions
diff --git a/mlir/lib/Dialect/LoopOps/LoopOps.cpp b/mlir/lib/Dialect/LoopOps/LoopOps.cpp index 4d99cac3a04..74da12fbace 100644 --- a/mlir/lib/Dialect/LoopOps/LoopOps.cpp +++ b/mlir/lib/Dialect/LoopOps/LoopOps.cpp @@ -82,32 +82,32 @@ static void print(OpAsmPrinter *p, ForOp op) { p->printOptionalAttrDict(op.getAttrs()); } -static ParseResult parseForOp(OpAsmParser *parser, OperationState *result) { - auto &builder = parser->getBuilder(); +static ParseResult parseForOp(OpAsmParser &parser, OperationState *result) { + auto &builder = parser.getBuilder(); OpAsmParser::OperandType inductionVariable, lb, ub, step; // Parse the induction variable followed by '='. - if (parser->parseRegionArgument(inductionVariable) || parser->parseEqual()) + if (parser.parseRegionArgument(inductionVariable) || parser.parseEqual()) return failure(); // Parse loop bounds. Type indexType = builder.getIndexType(); - if (parser->parseOperand(lb) || - parser->resolveOperand(lb, indexType, result->operands) || - parser->parseKeyword("to") || parser->parseOperand(ub) || - parser->resolveOperand(ub, indexType, result->operands) || - parser->parseKeyword("step") || parser->parseOperand(step) || - parser->resolveOperand(step, indexType, result->operands)) + if (parser.parseOperand(lb) || + parser.resolveOperand(lb, indexType, result->operands) || + parser.parseKeyword("to") || parser.parseOperand(ub) || + parser.resolveOperand(ub, indexType, result->operands) || + parser.parseKeyword("step") || parser.parseOperand(step) || + parser.resolveOperand(step, indexType, result->operands)) return failure(); // Parse the body region. Region *body = result->addRegion(); - if (parser->parseRegion(*body, inductionVariable, indexType)) + if (parser.parseRegion(*body, inductionVariable, indexType)) return failure(); ForOp::ensureTerminator(*body, builder, result->location); // Parse the optional attribute list. - if (parser->parseOptionalAttributeDict(result->attributes)) + if (parser.parseOptionalAttributeDict(result->attributes)) return failure(); return success(); @@ -150,33 +150,33 @@ static LogicalResult verify(IfOp op) { return success(); } -static ParseResult parseIfOp(OpAsmParser *parser, OperationState *result) { +static ParseResult parseIfOp(OpAsmParser &parser, OperationState *result) { // Create the regions for 'then'. result->regions.reserve(2); Region *thenRegion = result->addRegion(); Region *elseRegion = result->addRegion(); - auto &builder = parser->getBuilder(); + auto &builder = parser.getBuilder(); OpAsmParser::OperandType cond; Type i1Type = builder.getIntegerType(1); - if (parser->parseOperand(cond) || - parser->resolveOperand(cond, i1Type, result->operands)) + if (parser.parseOperand(cond) || + parser.resolveOperand(cond, i1Type, result->operands)) return failure(); // Parse the 'then' region. - if (parser->parseRegion(*thenRegion, {}, {})) + if (parser.parseRegion(*thenRegion, {}, {})) return failure(); - IfOp::ensureTerminator(*thenRegion, parser->getBuilder(), result->location); + IfOp::ensureTerminator(*thenRegion, parser.getBuilder(), result->location); // If we find an 'else' keyword then parse the 'else' region. - if (!parser->parseOptionalKeyword("else")) { - if (parser->parseRegion(*elseRegion, {}, {})) + if (!parser.parseOptionalKeyword("else")) { + if (parser.parseRegion(*elseRegion, {}, {})) return failure(); - IfOp::ensureTerminator(*elseRegion, parser->getBuilder(), result->location); + IfOp::ensureTerminator(*elseRegion, parser.getBuilder(), result->location); } // Parse the optional attribute list. - if (parser->parseOptionalAttributeDict(result->attributes)) + if (parser.parseOptionalAttributeDict(result->attributes)) return failure(); return success(); |

