ychen added a comment.

In D128750#3721441 <https://reviews.llvm.org/D128750#3721441>, @royjacobson 
wrote:

> Some documentation/test nits, and one question, but otherwise LGTM.
> Could you fix the merge conflict? It would be nice to see pre-commit CI 
> results.
>
> Given the complexity of the technical details here, as long as it doesn't 
> unreasonably delay this, please wait for other people to also review it.

Thanks for the review. Sure, I'll wait for more feedback.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128750

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to