[PATCH] D104198: [Matrix] Add documentation for compound assignment and type conversion of matrix types

Saurabh Jha via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 24 06:00:07 PDT 2021


SaurabhJha added a comment.

Somehow the builds are failing even though this patch contains no code changes.



================
Comment at: clang/docs/LanguageExtensions.rst:526
 
+The matrix type extension also supports operations between a matrix and a scalar.
+
----------------
fhahn wrote:
> I'm not a native speaker, but I am not sure if `between` is the best choice here. To me, `on` would sound slightly better, but again, I'm no expert.
I think you are right. Will change.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D104198/new/

https://reviews.llvm.org/D104198



More information about the cfe-commits mailing list