https://github.com/jimingham approved this pull request.

LGTM except for the necessity of the `continue` I mentioned.

If it is in fact needed, that's fine to include here (though the need seems 
bogus to me that's obviously not something this PR should fix).  Otherwise, 
maybe an assert would be better?

But the logic of the PR seems fine.

https://github.com/llvm/llvm-project/pull/204480
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to