chying added a comment. In http://reviews.llvm.org/D14673#289408, @zturner wrote:
> I guess this is ok. I don't like raising the `None` check outside the > function, but I don't see another way that isn't more convoluted. > > However, I think you still need to fix the grouping of the parentheses. the > `compiler is None` should not override the version check. I thought a call of expectedFailureAll(..compiler=None, compiler_version=['>=', '3.7']...) doesn't make much sense at the first place, so maybe just ignore compiler check in such cases? Do you suggest to check current compiler(no matter clang or gcc or others) version against given version? http://reviews.llvm.org/D14673 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
