<div dir="ltr">At some point I really would love to reduce the number of decorators.  It's starting to get ridiculous :)<div><br></div><div>Seems like we only need one decorator that takes everything as optional arguments</div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Sep 22, 2015 at 10:42 AM Tamas Berghammer <<a href="mailto:tberghammer@google.com">tberghammer@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">tberghammer added a comment.<br>
<br>
In <a href="http://reviews.llvm.org/D13028#250822" rel="noreferrer" target="_blank">http://reviews.llvm.org/D13028#250822</a>, @clayborg wrote:<br>
<br>
> Looks good. For the failing test cases, just check this stuff in and we should take care or marking any needed tests and expected fail. Is there a way to mark a test such that the "dwarf in .o files" will expected fail if we need to do that? If there currently isn't, we need to add the ability to say that normal dwarf (dSYM file for MacOSX or dwarf in object file for all other platforms), DWO, and dwarf in .o files (apple) can be marked as expected fail, skip, etc...<br>
<br>
<br>
I added an [expectedFailure,expectedFlakey,skipIf][Dwarf,Dsym] (will add Dwo whit the rest of the dwo specific logic) decorators and we can also specify the debug info for expectedFailureAll. If we need more decorators then we can add them the first time we have to use them (the debug info fomrat is stored in self.debug_info as a string)<br>
<br>
<br>
<a href="http://reviews.llvm.org/D13028" rel="noreferrer" target="_blank">http://reviews.llvm.org/D13028</a><br>
<br>
<br>
<br>
</blockquote></div>