[Lldb-commits] [PATCH] D91612: [lldb] Fix a couple of remote llgs tests

2020-11-18 Thread David Spickett via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG50f12ade2de1: [lldb] Fix a couple of remote llgs tests (authored by DavidSpickett). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D91612/new/ https://reviews

[Lldb-commits] [PATCH] D91612: [lldb] Fix a couple of remote llgs tests

2020-11-17 Thread Pavel Labath via Phabricator via lldb-commits
labath accepted this revision. labath added a comment. In D91612#2399408 , @DavidSpickett wrote: > I presume these tests aren't commonly run or have been masked by the expected > failures. Yes, that seems to be the case, though I don't actually see a re

[Lldb-commits] [PATCH] D91612: [lldb] Fix a couple of remote llgs tests

2020-11-17 Thread David Spickett via Phabricator via lldb-commits
DavidSpickett added a reviewer: labath. DavidSpickett added a comment. For reference the argument was removed in: https://reviews.llvm.org/D90313 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D91612/new/ https://reviews.llvm.org/D91612

[Lldb-commits] [PATCH] D91612: [lldb] Fix a couple of remote llgs tests

2020-11-17 Thread David Spickett via Phabricator via lldb-commits
DavidSpickett added a reviewer: JDevlieghere. DavidSpickett added a comment. I presume these tests aren't commonly run or have been masked by the expected failures. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D91612/new/ https://reviews.llvm.org/

[Lldb-commits] [PATCH] D91612: [lldb] Fix a couple of remote llgs tests

2020-11-17 Thread David Spickett via Phabricator via lldb-commits
DavidSpickett created this revision. Herald added a project: LLDB. Herald added a subscriber: lldb-commits. DavidSpickett requested review of this revision. Herald added a subscriber: JDevlieghere. init_llgs_test no longer takes an argument but these two were not updated. Also fix some mistakes i