[IR] Pretty-printer for LogicalObjectFifoFromMemrefOp

[newling]

Makes IR easier to mentally parse -- it should be easier to figure out which columns/rows a LogicalObjectFifoFromMemrefOp is on.

%lof_3_2 = ... // column 3, row 2
%lof_3_r = ... // column 3, multiple rows or unkown row 

The lit tests have more examples

[Abhishek-Varma]

Nice addition!

I have a naming related request if it's okay - since we have tile assignment related pass/functionality, perhaps we make a difference (naming-wise) between a LOF that :-

1. has not been assigned any tile.
2. has been assigned with unknown tiles/tiles on different row and col.

[Abhishek-Varma]

I'd expect this to be %lof_2_r_0 or %lof_2_r_1 or %lof_2_r_2 instead %lof_2_r_6 - not sure where the last _6 is coming from.

[newling]

Let me explain and then improve!
Explanation: there is a %tile_0 and a %tile_1 before %lof_2_r_2, that's why it's starting at 2 not 0.
Improvement TODO: (1) tiles can have better suffixing (2) separate into 2+ lit tests.

[Abhishek-Varma]

Similarly I'd expect this as %lof_0 or %lof_1 ?

[newling]

I have a naming related request if it's okay - since we have tile assignment related pass/functionality, perhaps we make a difference (naming-wise) between a LOF that :-

1. has not been assigned any tile.
2. has been assigned with unknown tiles/tiles on different row and col.

Ok. I guess that would mean something like %lof for (1) and %lof_c_r for (2). I'll try this

[jtuyls]

This is great!