jingham added a comment.

In D112222#3141355 <https://reviews.llvm.org/D112222#3141355>, @puremourning 
wrote:

> Sorry, but is anything further required from me on this patch ?

You addressed an issue Shafik asked you to, so then he should make sure he's 
happy with the change and mark the patch accepted.  OTOH we're all busy so it's 
easy to let this sort of thing drop.  On your end, you should wait a polite 
interval then issue a gentle ping in his direction...  A week is an okay 
interval, I think, so now would be appropriate.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112222

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

Reply via email to