This is an archive of the discontinued LLVM Phabricator instance.

[mlir][tosa] Call TosaInferShapes pass in addTosaToLinalgPasses
ClosedPublic

Authored by AviadCo on Apr 22 2023, 10:04 AM.

Details

Summary

This pass is useful to legalize rankless and dynamic shapes towards static using operands' shapes & types.

Diff Detail

Event Timeline

AviadCo created this revision.Apr 22 2023, 10:04 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 22 2023, 10:04 AM
AviadCo requested review of this revision.Apr 22 2023, 10:04 AM

Is the ordering after decomposition on purpose?

Is the ordering after decomposition on purpose?

No, do you think it should be before Decomposition to have more affect?

jpienaar accepted this revision.May 4 2023, 7:24 PM

I haven't tested, that just stood out to me. Test, if it has no effect, then leave as is (i could see argument both ways and I think it should matter, or rather where it matters it is covering some gaps).

This revision is now accepted and ready to land.May 4 2023, 7:24 PM

I haven't tested, that just stood out to me. Test, if it has no effect, then leave as is (i could see argument both ways and I think it should matter, or rather where it matters it is covering some gaps).

Well, currently no one uses this funciton directly in this repository, I am using it from another repository and I am not pretty sure the ordering has an affect, I will land it as is for now.