[clang] [llvm] [HLSL][Matrix] Support row and column indexing modes for MatrixSubscriptExpr (PR #171564)

Justin Bogner via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 19 12:12:42 PST 2025


================
@@ -238,19 +238,36 @@ class MatrixBuilder {
     else
       B.CreateAssumption(Cmp);
   }
-
-  /// Compute the index to access the element at (\p RowIdx, \p ColumnIdx) from
-  /// a matrix with \p NumRows embedded in a vector.
-  Value *CreateIndex(Value *RowIdx, Value *ColumnIdx, unsigned NumRows,
+  Value *createIndex(Value *RowIdx, Value *ColumnIdx, unsigned NumRows,
----------------
bogner wrote:

Please choose consistency with the existing code over clang-tidy's rules. While it's annoying that this class doesn't follow the LLVM conventions inconsistency is more painful to deal with.

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


More information about the llvm-commits mailing list