[Lldb-commits] [PATCH] D85388: [lldb] Fix bug in skipIfRosetta decorator

2020-08-06 Thread Adrian Prantl via Phabricator via lldb-commits
aprantl added a comment. Thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85388/new/ https://reviews.llvm.org/D85388 ___ lldb-commits mailing list lldb-commits@lists.llvm.org

[Lldb-commits] [PATCH] D85388: [lldb] Fix bug in skipIfRosetta decorator

2020-08-05 Thread Jonas Devlieghere via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG4fccdd5c85d0: [lldb] Fix bug in skipIfRosetta decorator (authored by JDevlieghere). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85388/new/

[Lldb-commits] [PATCH] D85388: [lldb] Fix bug in skipIfRosetta decorator

2020-08-05 Thread Davide Italiano via Phabricator via lldb-commits
davide accepted this revision. davide added a comment. This revision is now accepted and ready to land. yes, this makes sense. We could refine the check in future. Repository: rLLDB LLDB CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85388/new/ https://reviews.llvm.org/D85388

[Lldb-commits] [PATCH] D85388: [lldb] Fix bug in skipIfRosetta decorator

2020-08-05 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere added inline comments. Comment at: lldb/packages/Python/lldbsuite/test/decorators.py:542 -if not lldbplatformutil.getPlatform() in ['darwin', 'macosx']: -return "not on macOS" -if (platform.uname()[5] == "arm") and (self.getArchitecture()

[Lldb-commits] [PATCH] D85388: [lldb] Fix bug in skipIfRosetta decorator

2020-08-05 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere created this revision. JDevlieghere added reviewers: davide, friss. JDevlieghere added a project: LLDB. JDevlieghere requested review of this revision. Currently, the `skipIfRosetta` decorator will skip tests with the message "not on macOS" on non-macOS platforms. This triggers for