[Mlir-commits] [mlir] [mlir][spirv] Add basic support for SPV_EXT_replicated_composites (PR #147067)

[Mohammadreza Ameri Mahabadian]

mahabadm wrote:

> Thank you for submitting this PR. Few general and specific comments from me:
> 
>     1. We probably should also add tests to: https://github.com/llvm/llvm-project/blob/main/mlir/test/Dialect/SPIRV/IR/availability.mlir
> 
>     2. I wonder whether it would make sense to add some tests to https://github.com/llvm/llvm-project/tree/main/mlir/test/Dialect/SPIRV/IR including some negative tests?
> 
>     3. Regarding the use of braces; I feel like in some places they could be dropped. See: https://llvm.org/docs/CodingStandards.html#don-t-use-braces-on-simple-single-statement-bodies-of-if-else-loop-statements But I leave the decision up to you.

Many thanks for your comments @IgWod-IMG !
I have now added several negative tests and also a test in availability.mlir. These were great suggestions resulting in improving the patch even further.

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