Page MenuHomePhabricator

Please use GitHub pull requests for new patches. Phabricator shutdown timeline

Differential D114928

[mlir][linalg][bufferize][NFC] Use same OpBuilder throughout bufferization
ClosedPublic
Actions

Authored by springerm on Dec 1 2021, 9:50 PM.

Details

Reviewers
nicolasvasilache
aartbik
Commits
rG6db200736c51: [mlir][linalg][bufferize][NFC] Use same OpBuilder throughout bufferization
Summary

Also set insertion point right before calling bufferize. No need to put an InsertionGuard anymore.

Depends On D114927

Diff Detail

Event Timeline

springerm created this revision.Dec 1 2021, 9:50 PM
Herald added a reviewer: aartbik. · View Herald TranscriptDec 1 2021, 9:50 PM
Herald added subscribers: sdasgup3, wenzhicui, wrengr and 21 others. · View Herald Transcript
springerm requested review of this revision.Dec 1 2021, 9:50 PM
Herald added a project: Restricted Project. · View Herald TranscriptDec 1 2021, 9:50 PM
Herald added subscribers: limo1996, stephenneuendorffer. · View Herald Transcript
springerm added a child revision: D114929: [mlir][linalg][bufferize][NFC] Map only tensors in BufferizationState.Dec 1 2021, 9:51 PM
Harbormaster completed remote builds in B137061: Diff 391209.Dec 1 2021, 10:15 PM
springerm updated this revision to Diff 391235.Dec 2 2021, 12:08 AM

update

Harbormaster completed remote builds in B137081: Diff 391235.Dec 2 2021, 12:31 AM
nicolasvasilache accepted this revision.Dec 3 2021, 12:48 AM
This revision is now accepted and ready to land.Dec 3 2021, 12:48 AM
This revision was landed with ongoing or failed builds.Dec 3 2021, 5:03 PM
Closed by commit rG6db200736c51: [mlir][linalg][bufferize][NFC] Use same OpBuilder throughout bufferization (authored by springerm). · Explain Why
This revision was automatically updated to reflect the committed changes.
springerm added a commit: rG6db200736c51: [mlir][linalg][bufferize][NFC] Use same OpBuilder throughout bufferization.

Revision Contents
Changeset List

PathSize
mlir/
include/
mlir/
Dialect/
Linalg/
ComprehensiveBufferize/
16 lines
lib/
Dialect/
Linalg/
ComprehensiveBufferize/
11 lines
24 lines
13 lines
14 lines
4 lines
36 lines
18 lines

Diff 391782

View Options

mlir/include/mlir/Dialect/Linalg/ComprehensiveBufferize/BufferizableOpInterface.h

Loading...
View Options

mlir/lib/Dialect/Linalg/ComprehensiveBufferize/ArithInterfaceImpl.cpp

Loading...
View Options

mlir/lib/Dialect/Linalg/ComprehensiveBufferize/BufferizableOpInterface.cpp

Loading...
View Options

mlir/lib/Dialect/Linalg/ComprehensiveBufferize/LinalgInterfaceImpl.cpp

Loading...
View Options

mlir/lib/Dialect/Linalg/ComprehensiveBufferize/ModuleBufferization.cpp

Loading...
View Options

mlir/lib/Dialect/Linalg/ComprehensiveBufferize/SCFInterfaceImpl.cpp

Loading...
View Options

mlir/lib/Dialect/Linalg/ComprehensiveBufferize/TensorInterfaceImpl.cpp

Loading...
View Options

mlir/lib/Dialect/Linalg/ComprehensiveBufferize/VectorInterfaceImpl.cpp

Loading...