amurzeau added a comment.

In D144431#4146524 <https://reviews.llvm.org/D144431#4146524>, @carlosgalvezp 
wrote:

> Alright, sounds good, thanks for the clarification! Then I misunderstood the 
> change in the tests and indeed it belonged together in this patch, sorry for 
> the confusion. I have approved both patches so we can land them now. Can you 
> land the patch or would you like me to do it for you? If so please provide 
> name and email (same as your Github account) for attribution.

No worries, yes please land the patches for me as `Alexis Murzeau 
<amub...@gmail.com>` (phabricator mail and handle is the same as github).
Thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144431

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

Reply via email to