This is an archive of the discontinued LLVM Phabricator instance.

[mlir] Delete MemRefType::setMemorySpace(unsigned)
ClosedPublic

Authored by tpopp on Aug 23 2022, 5:50 AM.

Diff Detail

Event Timeline

tpopp created this revision.Aug 23 2022, 5:50 AM
tpopp requested review of this revision.Aug 23 2022, 5:50 AM
tpopp edited the summary of this revision. (Show Details)Aug 23 2022, 6:00 AM
rriddle requested changes to this revision.Aug 23 2022, 12:09 PM
rriddle added inline comments.
mlir/include/mlir/IR/BuiltinTypes.h
196

You also have to delete the definition (in BuiltinTypes.cpp)

This revision now requires changes to proceed.Aug 23 2022, 12:09 PM
tpopp updated this revision to Diff 455542.Aug 25 2022, 4:58 AM

Remove definition of method after only declaration was previously removed.

tpopp marked an inline comment as done.Aug 25 2022, 4:58 AM
tpopp added inline comments.
mlir/include/mlir/IR/BuiltinTypes.h
196

:facepalm: Thanks for catching that!

rriddle accepted this revision.Aug 28 2022, 10:33 PM
This revision is now accepted and ready to land.Aug 28 2022, 10:33 PM
This revision was landed with ongoing or failed builds.Aug 29 2022, 3:33 AM
This revision was automatically updated to reflect the committed changes.