Yes, this patch LGTM.<div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jul 11, 2012 at 1:15 PM, Stepan Dyatkovskiy <span dir="ltr"><<a href="mailto:STPWORLD@narod.ru" target="_blank">STPWORLD@narod.ru</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">>From llvm-diff doc:<br>
"If llvm-diff finds no differences between the modules, it will exit with 0 and produce no output. Otherwise it will exit with a non-zero value."<br>
So it llvm-diff is better choice then. Fixed patch is attached.<br>
-Stepan.<br>
<br>
12.07.2012, 00:07, "Jim Grosbach" <<a href="mailto:grosbach@apple.com">grosbach@apple.com</a>>:<br>
<div class="HOEnZb"><div class="h5">> llvm-diff compares bitcode. Perhaps it would work here? Note: I haven't tried it for this; I'm hypothesizing.<br>
><br>
> -Jim<br>
><br>
> On Jul 11, 2012, at 1:06 PM, Stepan Dyatkovskiy <<a href="mailto:STPWORLD@narod.ru">STPWORLD@narod.ru</a>> wrote:<br>
><br>
>>  How to test that BitcodeReader=>Writer=Reader doesn't distorts the data then?<br>
>>  -Stepan.<br>
>><br>
>>  12.07.2012, 00:03, "Jim Grosbach" <<a href="mailto:grosbach@apple.com">grosbach@apple.com</a>>:<br>
>>>  On Jul 11, 2012, at 1:00 PM, Benjamin Kramer <<a href="mailto:benny.kra@gmail.com">benny.kra@gmail.com</a>> wrote:<br>
>>>>   On 11.07.2012, at 21:55, Stepan Dyatkovskiy <<a href="mailto:stpworld@narod.ru">stpworld@narod.ru</a>> wrote:<br>
>>>>>   Index: test/Bitcode/2012-05-07-SwitchInstRangesSupport.ll<br>
>>>>>   ===================================================================<br>
>>>>>   --- test/Bitcode/2012-05-07-SwitchInstRangesSupport.ll (revision 160066)<br>
>>>>>   +++ test/Bitcode/2012-05-07-SwitchInstRangesSupport.ll (working copy)<br>
>>>>>   @@ -6,7 +6,7 @@<br>
>>>>>   ; RUN: llvm-dis %t.bc -o - | tail -n +2 > %t.ll<br>
>>>>>   ; RUN: llvm-as %t.ll -o %t2.bc<br>
>>>>>   ; RUN: llvm-dis %t2.bc -o - | tail -n +2 > %t2.ll<br>
>>>>>   -; RUN: diff %t.ll %t2.ll | not grep .*<br>
>>>>>   +; RUN: diff %t.ll %t2.ll | not grep ^.*$<br>
>>>>   diff returns 1 (error) when there is a difference so you can drop the "| not grep ^.*$" part here.<br>
>>>  Not all platforms have diff(1). Windows, e.g. Not sure how much we care, though.<br>
>>><br>
>>>  -j</div></div><br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>