[Lldb-commits] [PATCH] D16936: Remove expectedFailureWindows decorator

Zachary Turner via lldb-commits lldb-commits at lists.llvm.org
Tue Feb 9 09:51:32 PST 2016

zturner added a comment.

In http://reviews.llvm.org/D16936#347141, @labath wrote:

> I think this is getting way too complicated. I haven't seen any test
>  which needs such complicated combinations of skip conditions (and I
>  hope I never see one).

I agree that you don't need arbitrary complexity, but I think there are some things we would greatly benefit from.  For example, have you ever seen this?


That's 3 decorators, when you could write it as 1:

  @expectedFailure(or(oslist="windows", compiler="gcc", hostoslist="linux"))

The rest of the functionality I mentioned you practically get for free if you support this.



More information about the lldb-commits mailing list