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

Florian Hahn via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 26 06:20:01 PDT 2022


fhahn added a comment.
Herald added a project: All.

In D106005#2904865 <https://reviews.llvm.org/D106005#2904865>, @SaurabhJha wrote:

> In D106005#2904424 <https://reviews.llvm.org/D106005#2904424>, @fhahn wrote:
>
>> In D106005#2896080 <https://reviews.llvm.org/D106005#2896080>, @SaurabhJha wrote:
>>
>>> In D106005#2895716 <https://reviews.llvm.org/D106005#2895716>, @fhahn wrote:
>>>
>>>> Thank you very much for working on this! Are you planning on implementing the new specification as well? It would probably be good to land the update to the spec in close succession to the implementation, to avoid confusing users.
>>>
>>> Yes, that's my plan. Once this is in, I will start working on the implementation right away.
>>
>> Ok cool! I think the latest version looks good (modulo making sure the new lines are limited to 80 chars per line). @rjmccall can you think of any scenarios where defining initializers with one expression and broadcasting them might cause issues?
>>
>> With respect to ordering the patches, I think it would be good to put up a patch implementing the newly added parts, commit it and then land the patch that adds it to the docs. WDYT?
>
> Yeah, sounds good. I will create a patch for implementing initialisation.

@SaurabhJha did you ever get a chance to create a patch?


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