This is an archive of the discontinued LLVM Phabricator instance.

Add createDynamicDimOps to tensor dialect utils
ClosedPublic

Authored by frgossen on Jun 10 2022, 1:25 PM.

Diff Detail

Event Timeline

frgossen created this revision.Jun 10 2022, 1:25 PM
frgossen requested review of this revision.Jun 10 2022, 1:25 PM
pifon2a accepted this revision.Jun 12 2022, 2:03 AM
pifon2a added inline comments.
mlir/include/mlir/Dialect/Tensor/Utils/Utils.h
32

nit: I think the name is a big misleading. What you are doing here is you compute values for every dynamic dimensions of the shape. Maybe call it getDynamicDimValues?

mlir/lib/Dialect/Tensor/Utils/Utils.cpp
63

nit: usually this is called en.

This revision is now accepted and ready to land.Jun 12 2022, 2:03 AM
akuegel accepted this revision.Jun 12 2022, 11:47 PM
akuegel added inline comments.
mlir/include/mlir/Dialect/Tensor/Utils/Utils.h
31

nit: maybe "ranked tensor value" instead of just "tensor value"?

frgossen updated this revision to Diff 436354.Jun 13 2022, 5:25 AM

Address comments

frgossen marked 3 inline comments as done.Jun 13 2022, 5:25 AM

Thanks

This revision was landed with ongoing or failed builds.Jun 13 2022, 5:27 AM
This revision was automatically updated to reflect the committed changes.