[PATCH] D106005: [Docs] Define matrix initialisation in MatrixTypes documentation

Saurabh Jha via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 20 11:16:33 PDT 2021


SaurabhJha added inline comments.


================
Comment at: clang/docs/MatrixTypes.rst:271
+---------------------
+An empty value of a matrix type M can be initialised using this syntax:
+
----------------
SaurabhJha wrote:
> fhahn wrote:
> > is there a reason we need to allow 'empty' initialisation? What does it mean? 
> By empty, I meant this `{}` initialisation which was in the test case. I got the term wrong but seems like we do want to support `{}` initialisation, right?
I meant this test case https://github.com/llvm/llvm-project/blob/3323a628ec821b8b75d3b60bf1510931f97d3883/clang/test/CodeGenCXX/matrix-type-builtins.cpp#L78


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106005



More information about the cfe-commits mailing list