@chelini If there are no other comments, would you mind submitting the change. I do not have commit access.

Spelling fix

@ingomueller-net Would you be able to submit the change. I do not have commit access.

Remove InsertionGuard now that https://reviews.llvm.org/D154684 has landed

Re-add the InsertionGuard

test removing insertion guard

Address comments from @ingomueller-net

Fix issue where operations created after the signature conversion were generated
inside the loop body rather than after the loop.

Delete implicitly created body block

Addressed @rsuderman's feedback.

Address rsuderman's comments on test cases

@rsuderman Could you elaborate more on your last point in the review:

Convert additional concat test to use CHECK-DAG

Lock down computation of the total size of the output tensor

Address rsuderman's comments

If there are no other comments, would someone mind submitting this change, since I do not have commit access?

In D151117#4375142, @mehdi_amini wrote:

to have a working legalization to linalg.

Is there a path from there to linalg? It's not clear to me how it works?
And actually I'm wondering if this lowering could be expressed with a linalg.generic indeed?

Revert accidental change to file

Address @eric-k256's comments

@jpienaar Would you mind submitting this change then, since I do not have commit privileges?

In D150439#4342736, @jpienaar wrote:

We should not be folding if the type changes AFAIK (I think that may not be enforced ...). Is this change effectively ensuring it doesn't change types?

If there are no other comments, could someone with submission privileges please submit the patch?

@stellaraccident @GeorgeARM if this change looks acceptable, would one of you mind submitting it? I do not have commit access.

Address @stellaraccident's comments

Avoid converting data to mlir::Attributes in all cases