[Mlir-commits] [mlir] [Linalg] Update Vectorization to work with both named as well as generic conv ops (PR #176339)

[Andrzej Warzyński]

banach-space wrote:

Thank you for updating the summary!

> @Abhishek-Varma can you expand the PR description? I think you also made changes to the constructor of generator, etc. Also, please mention the test coverage as well. The idea is that people can quickly get what's happening from PR description without looking at all the changes. (It is a higher bar, since not all the PRs follow the requirement. It is a good thing to have, IMO.)

+1

PR summaries are very important. I really like this recent comment from Alex Bradbury in the discussion on our AI policy in LLVM:
* https://discourse.llvm.org/t/rfc-llvm-ai-tool-policy-human-in-the-loop/89159/21

> I strongly believe that the most straightforward and actionable advice we can give to contributors hoping to ensure they have sufficient understanding of a submission is to encourage them to go through the process of writing a clear description themselves.

Just pointing out - many folks throughout LLVM find PR descriptions useful and worth polishing :)

https://github.com/llvm/llvm-project/pull/176339