Page MenuHomePhabricator

[MLIR] Nested regions test for Buffer Assginment

Authored by dfki-ehna on May 4 2020, 8:12 AM.



Due to the extension of Liveness, Buffer Assignment can now work on nested regions. This PR provides a test case to show that existing functionally of BA works properly.

Diff Detail

Event Timeline

dfki-ehna created this revision.May 4 2020, 8:12 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 4 2020, 8:12 AM
mehdi_amini added inline comments.May 4 2020, 10:21 AM

Can you expand the doc on what this test is doing? Either here or in line with the checks themselves.


Nit: can you indent it to represent the expected nesting? It'll make it more readable.

dfki-ehna updated this revision to Diff 262091.May 5 2020, 6:12 AM

Add description and indent.

dfki-ehna marked 2 inline comments as done.May 5 2020, 6:49 AM
dfki-ehna added reviewers: mehdi_amini, herhut, pifon2a.
dfki-ehna added a project: Restricted Project.
herhut accepted this revision.May 7 2020, 5:14 AM

Thanks, I am happy to see this land. Just some nits.


Could you reflow the comment to match 80 chars?


Could you add where the BB3 starts? That makes it easier to understand what is going on here.

This revision is now accepted and ready to land.May 7 2020, 5:14 AM
dfki-ehna marked 2 inline comments as done.May 13 2020, 6:03 AM
This revision was automatically updated to reflect the committed changes.