[PATCH] D85911: libclc: spirv: Add various functions

Daniel Stone via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 2 23:28:36 PDT 2020


daniels added a comment.

In D85911#2253686 <https://reviews.llvm.org/D85911#2253686>, @jenatali wrote:

> In D85911#2253680 <https://reviews.llvm.org/D85911#2253680>, @jvesely wrote:
>
>> I've thought about this. I think having a section on expected uses of all targets in README would be the best alternative. Feel free to go ahead with this patch as is. README.TXT is hopelessly out of date anyway so updating it should be done in a separate patch anyway.
>
> I believe neither @daniels nor I have permission to go ahead on our own. If you're good with this, I think we'll need you to merge it for us.

Right @jvesely, I don't have any push access either. If either you or @tstellar could land this that would be great please. Thanks for the review!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85911



More information about the llvm-commits mailing list