maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

[mlir] Rename 'setInsertionPointAfter' to avoid ambiguity 

Rename 'setInsertionPointAfter(Value)' API to avoid ambiguity with
'setInsertionPointAfter(Operation *)' for SingleResult operations which
implicitly convert to Value (see D86756).

Differential Revision: https://reviews.llvm.org/D87155
This commit is contained in:
Diego Caballero 2020-09-04 11:44:32 -07:00
parent c3fd2a50ba
commit 609f5e050c
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
mlir/include/mlir/IR/Builders.h
View File

@ -333,7 +333,7 @@ public:
/// defining operation. This will cause subsequent insertions to go right /// defining operation. This will cause subsequent insertions to go right
/// after it. Otherwise, value is a BlockArgumen. Sets the insertion point to /// after it. Otherwise, value is a BlockArgumen. Sets the insertion point to
/// the start of its block. /// the start of its block.
void setInsertionPointAfter(Value val) { void setInsertionPointAfterValue(Value val) {
if (Operation *op = val.getDefiningOp()) { if (Operation *op = val.getDefiningOp()) {
setInsertionPointAfter(op); setInsertionPointAfter(op);
} else { } else {

2
mlir/lib/Dialect/Affine/Transforms/SuperVectorize.cpp
View File

@ -945,7 +945,7 @@ static bool isUniformDefinition(Value value,
/// vectorization strategy in 'state'. /// vectorization strategy in 'state'.
static Value vectorizeUniform(Value value, VectorizationState *state) { static Value vectorizeUniform(Value value, VectorizationState *state) {
OpBuilder builder(value.getContext()); OpBuilder builder(value.getContext());
builder.setInsertionPointAfter(value); builder.setInsertionPointAfterValue(value);
auto vectorTy = getVectorType(value.getType(), state->strategy); auto vectorTy = getVectorType(value.getType(), state->strategy);
auto bcast = builder.create<BroadcastOp>(value.getLoc(), vectorTy, value); auto bcast = builder.create<BroadcastOp>(value.getLoc(), vectorTy, value);