[Mlir-commits] [mlir] [MLIR][SparseTensor] Loop ordering strategy infrastructure (flag) (PR #154656)

Aart Bik llvmlistbot at llvm.org
Tue Aug 26 09:41:43 PDT 2025


================
@@ -55,6 +55,15 @@ enum class SparseEmitStrategy {
   kDebugInterface, // generate only place-holder for sparse iteration
 };
 
+namespace sparse_tensor {
+
+/// Selects between different loop ordering strategies for sparse tensor 
----------------
aartbik wrote:

Please pay attention to style. Period at end of comments (unless inline).
Also, in line with the other comments, I would say (since this is a struct not a method)

// Defines a strategy for loop ordering during sparse code generation.

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


More information about the Mlir-commits mailing list