This is an archive of the discontinued LLVM Phabricator instance.

[MLIR] Fix memref region compute for 0-d memref accesses
ClosedPublic

Authored by bondhugula on Jun 13 2020, 12:30 PM.

Details

Summary

Fix memref region compute for 0-d memref accesses in certain cases (when
there are loops surrounding such 0-d accesses).

Diff Detail

Event Timeline

bondhugula created this revision.Jun 13 2020, 12:30 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 13 2020, 12:30 PM
ftynse accepted this revision.Jun 15 2020, 2:07 AM
ftynse added inline comments.
mlir/lib/Analysis/Utils.cpp
214

Nit: this function is a misnomer, it returns loops not induction variables

This revision is now accepted and ready to land.Jun 15 2020, 2:07 AM
andydavis1 accepted this revision.Jun 15 2020, 7:24 AM
dcaballe accepted this revision.Jun 15 2020, 8:59 AM
bondhugula marked 2 inline comments as done.Jun 15 2020, 12:13 PM
bondhugula added inline comments.
mlir/lib/Analysis/Utils.cpp
214

Thanks for noticing - will fix this in an NFC commit soon.

This revision was automatically updated to reflect the committed changes.
bondhugula marked an inline comment as done.