rupprecht added a comment.

In D70324#1755712 <https://reviews.llvm.org/D70324#1755712>, @davide wrote:

> In D70324#1755704 <https://reviews.llvm.org/D70324#1755704>, @labath wrote:
>
> > [This is probably not the right patch, as the last run on that bot still 
> > doesn't have this commit.]
>


-> D70137 <https://reviews.llvm.org/D70137> is the correct patch that actually 
adds this test -- this is testonly cleanup

>> My understanding is that all pexpect tests are failing on that bot and are 
>> skipped with `@skipIfAsan`. Probably this one needs to be skipped too. 
>> @teemperor was trying to reproduce some of those failures locally and 
>> failed... Maybe that bot is cursed...
> 
> Fair.

I looked at the failure in some detail. (I don't have hardware to repro it on 
though). The test itself runs fine but the cleanup (I guess the `self.quit()`) 
at the end seems to fail: `pexpect.exceptions.ExceptionPexpect: Could not 
terminate the child.` Is that in line with the failures that `@skipIfAsan` is 
intended for?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70324



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

Reply via email to