Page MenuHomePhabricator

[MLIR][Shape] Allow `cstr_broadcastable` to accept extent tensors
ClosedPublic

Authored by frgossen on Jul 20 2020, 5:29 AM.

Diff Detail

Event Timeline

frgossen created this revision.Jul 20 2020, 5:29 AM
Herald added a reviewer: silvas. · View Herald Transcript
Herald added a project: Restricted Project. · View Herald Transcript
herhut accepted this revision.Jul 20 2020, 5:48 AM

I wonder whether we need to duplicate all the tests to also cover tensor types. Under the hood it would be the same code-path anyway, though.

frgossen updated this revision to Diff 279240.Jul 20 2020, 7:38 AM

Add test cases

Added two test cases to have some test that the extent tensor variant is also considered. Duplication all tests would

This revision was not accepted when it landed; it landed in state Needs Review.Jul 20 2020, 7:40 AM
This revision was automatically updated to reflect the committed changes.