clayborg requested changes to this revision. clayborg added a comment. This revision now requires changes to proceed.
Seems like it would be better to mimic how @expectedFailureAndroid works. See inlined comments. ================ Comment at: packages/Python/lldbsuite/test/functionalities/watchpoint/step_over_watchpoint/TestStepOverWatchpoint.py:20 @@ -19,2 +19,3 @@ @expectedFailureAndroid(archs=['arm', 'aarch64']) # Watchpoints not supported + @expectedFailureArmLinux("llvm.org/pr26031") @expectedFailureWindows("llvm.org/pr24446") ---------------- Seems like it would be better to do things like @expectedFailureAndroid that is just above: ``` @expectedFailureLinux(archs=['arm'], bug="llvm.org/pr26031") ``` I would like to avoid having new variants of the @expectedFailure that contain the architecture in the name. http://reviews.llvm.org/D15893 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits