Submitted.

It works great. Thanks!

Recover the missing r.

Thanks for the review. BTW, how can I test the changes on the cmake and bazel files? I usually run "ninja check-mlir" for testing.

• Expose hasOnlyScalarElementwiseOp as a global function.
• Specify a dependency to FuncDialect in the CMake and Bazel files.

Make the commit title more explicit.

rebase to fix a build error for bazel

update the python pass manager test

I reverted the commit because there is a test failure for the python pass manager. Looking into it.

The debian failure is caused by "OSError: [Errno 28] No space left on device".

Update to use a json object instead of an array

The build failed because there is no space available.

It would be nice if you don't mind reverting.

Hi, just for heads-up, this change caused many failures in IREE tests including compilation errors and accuracy errors.

Thanks Mahesh. Much cleaner!

address the comment

LGTM. Thanks!

Adding more comments.

Adding minor comments. LGTM.

Adding more comments.

I left a few comments mostly related to naming and comments. I will review the rest later.

Rebased onto origin/main

Thanks Tobias for the info. I believe if the padding is added as an optimization for vectorization. We need to make this transformation happens before it.
If we see more cases that need a fine control, we can add a control function to this pass. Thanks!

Revert an unnecessary change on a comment.

Thanks for the review! Addressed all the TODO items.

Address the comments

Replace getAsValues with getValueOrCreateConstantIndexOp.

The plan is changed because

1. I made a big code change to support matmul cases, and
2. Mahesh pointed me to the tiling implementation, which works for any Linalg operations. I need to take a look at the code and understand how it works for all without any restrictions on the linalg operation, and
3. Apply the same or similar approach to my implementation.

Address Mahesh's comments

merge and rebase

check projected permutation for inputs

LGTM. Thanks

Fix the bazel build error and rewrite the commit message.

Reopening: The push to origin/main caused a build break due to a cyclic header inclusion between Tensor.h and Transforms.h. The issue didn't occur in my local machine, but it broke the testing system.

Merged to origin/main.

Addressed Uday's comments.

Thanks Uday for the comments. Please take another look.

Address Mahesh's comments

The code is updated with the control function approach. Please take a look.

Add control function support

Merged into the LLVM main.

Introduce resizeSplat instead of using reshape since reshape is not suppposed
to change the number of elements.

There is already reshape(), but it didn't support a shape with a different number of element even for a splat type.
I relaxed the assert condition to allow a type with a different number of elements.

Support a different shape for reshape for a splat type.

Use the attribute operand instead of matching.

Right. It does not handle the splat case yet.

I have added two more unit tests.

add two more unit tests

Fix a missing change for IterTy and ElemTy

rebase onto main

Resolve the patch error