This is an archive of the discontinued LLVM Phabricator instance.

[mlir][tosa] Add tosa.argmax to linalg lowering
ClosedPublic

Authored by rsuderman on Mar 22 2021, 8:14 PM.

Details

Summary

Tosa's argmax lowering is representable as a linalg.indexed_generic
operation. Include the lowering to this type for both integer and
floating point types.

Diff Detail

Event Timeline

rsuderman created this revision.Mar 22 2021, 8:14 PM
rsuderman requested review of this revision.Mar 22 2021, 8:14 PM
silvas added inline comments.Mar 23 2021, 10:41 AM
mlir/lib/Conversion/TosaToLinalg/TosaToLinalg.cpp
868

comment giving an overview of the approach.

rsuderman updated this revision to Diff 332795.Mar 23 2021, 2:49 PM

Added header comment about lowering behavior. Cleaned up some redundant code.

rsuderman marked an inline comment as done.Mar 23 2021, 2:49 PM
silvas accepted this revision.Mar 23 2021, 3:44 PM
This revision is now accepted and ready to land.Mar 23 2021, 3:44 PM
This revision was landed with ongoing or failed builds.Mar 23 2021, 4:07 PM
This revision was automatically updated to reflect the committed changes.