Page MenuHomePhabricator
Differential D90226

[flang] update ODS syntax to use OpBuidlerDAG instead of OpBuilder
ClosedPublic
Actions

Authored by ftynse on Oct 27 2020, 5:00 AM.

Details

Reviewers
schweitz
Commits
rGcc2a2bb5ce51: [flang] update ODS syntax to use OpBuidlerDAG instead of OpBuilder
Summary

Upstream MLIR has deprecated OpBuilder in ODS.

Diff Detail

Event Timeline

ftynse created this revision.Oct 27 2020, 5:00 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 27 2020, 5:00 AM
Herald added subscribers: llvm-commits, rriddle. · View Herald Transcript
ftynse requested review of this revision.Oct 27 2020, 5:00 AM
Herald added a subscriber: stephenneuendorffer. · View Herald TranscriptOct 27 2020, 5:00 AM
Harbormaster completed remote builds in B76536: Diff 300956.Oct 27 2020, 5:18 AM
ftynse added a comment.Nov 4 2020, 4:42 AM

Ping.

This updates the code to avoid using a deprecated API that will be removed soon.

PeteSteinfeld added a project: Restricted Project.Nov 4 2020, 9:36 AM
Herald added a subscriber: jdoerfert. · View Herald TranscriptNov 4 2020, 9:36 AM
schweitz accepted this revision.Nov 4 2020, 9:47 AM

Looks good. Thanks for doing this.

This revision is now accepted and ready to land.Nov 4 2020, 9:47 AM
Closed by commit rGcc2a2bb5ce51: [flang] update ODS syntax to use OpBuidlerDAG instead of OpBuilder (authored by ftynse). · Explain WhyNov 5 2020, 12:28 AM
This revision was automatically updated to reflect the committed changes.
ftynse added a commit: rGcc2a2bb5ce51: [flang] update ODS syntax to use OpBuidlerDAG instead of OpBuilder.

Revision Contents
Changeset List

PathSize
flang/
include/
flang/
Optimizer/
Dialect/
276 lines

Diff 303051

View Options

flang/include/flang/Optimizer/Dialect/FIROps.td

Loading...