jingham added a comment.

Since you just mainly want to run this past the bots again, probably best to 
just resubmit.  You could update the patch here or not at your convenience, but 
I don't see it would serve any purpose to wait on approval.

Sorry for not noticing it was a shell test, BTW...

In D97786#2694686 <https://reviews.llvm.org/D97786#2694686>, @cmtice wrote:

> Hi Pavel,
>
> Thank you for both the detailed explanation and the updated test case (and 
> thank you Jim for your recommendations as well!).  I'm testing the updated 
> test case now.  Assuming the tests pass, is it ok for me to re-land this CL 
> using the updated test case?  and if so, do I need to upload the updated 
> patch to this code review?




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97786

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

Reply via email to