[PATCH] D99037: [Matrix] Implement C-style explicit type conversions for matrix types

Saurabh Jha via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 9 08:28:29 PDT 2021


Seems like the new build is passing. Can you please commit it on my behalf
if it looks okay to you?

Thanks a lot for your help in this patch.

Saurabh

On Fri, Apr 9, 2021 at 12:37 PM Saurabh Jha via Phabricator <
reviews at reviews.llvm.org> wrote:

> SaurabhJha added a comment.
>
> In D99037#2679131 <https://reviews.llvm.org/D99037#2679131>, @fhahn wrote:
>
> > In D99037#2678848 <https://reviews.llvm.org/D99037#2678848>,
> @SaurabhJha wrote:
> >
> >> In D99037#2678779 <https://reviews.llvm.org/D99037#2678779>, @fhahn
> wrote:
> >>
> >>>
> >>
> >> Will create a new patch to address your last comments
> >
> > Can you directly update this one? I'll commit it after the update.
>
> Sure, done. I did not know I could change the patch after its accepted.
>
>
> Repository:
>   rG LLVM Github Monorepo
>
> CHANGES SINCE LAST ACTION
>   https://reviews.llvm.org/D99037/new/
>
> https://reviews.llvm.org/D99037
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20210409/0194b25e/attachment.html>


More information about the cfe-commits mailing list