<p dir="ltr">I'm probably going to just remove it all. Message about that pending. </p>
<div class="gmail_quote">On Jul 31, 2014 12:02 PM, "David Blaikie" <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Any chance of adding some missing test coverage here? That code was<br>
dead (& evidently untested) before...<br>
<br>
On Thu, Jul 31, 2014 at 11:37 AM, Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> wrote:<br>
> I believe you are correct. Fixed thusly:<br>
><br>
> dzur:~/sources/llvm> git svn dcommit<br>
> Committing to <a href="https://llvm.org/svn/llvm-project/llvm/trunk" target="_blank">https://llvm.org/svn/llvm-project/llvm/trunk</a> ...<br>
> M lib/MC/MCAnalysis/MCObjectDisassembler.cpp<br>
> Committed r214416<br>
><br>
> -eric<br>
><br>
><br>
> On Thu, Jul 31, 2014 at 10:26 AM, xliu <<a href="mailto:navy.xliu@gmail.com">navy.xliu@gmail.com</a>> wrote:<br>
>> my compiler gave me a warning in MCObjectDisassembler.cpp. it found a<br>
>> self-comparation in loop condition. I think it's a typo.<br>
>><br>
>> the suspicious code was introduced by this patch:<br>
>><br>
>> From f176482752fbea3139394e280adfb10270dd3aac Mon Sep 17 00:00:00 2001<br>
>> From: Ahmed Bougacha <<a href="mailto:ahmed.bougacha@gmail.com">ahmed.bougacha@gmail.com</a>><br>
>> Date: Wed, 21 Aug 2013 07:28:55 +0000<br>
>> Subject: MC CFG: Support disassembly at arbitrary addresses in<br>
>> MCObjectDisassembler.<br>
>><br>
>> git-svn-id: <a href="https://llvm.org/svn/llvm-project/llvm/trunk@188889" target="_blank">https://llvm.org/svn/llvm-project/llvm/trunk@188889</a><br>
>> 91177308-0d34-0410-b5e6-96231b3b80d8<br>
>><br>
>><br>
>> please review "SE != SE" below. i think it should be SI != SE.<br>
>><br>
>><br>
>> + for (AddressSetTy::const_iterator SI = BBI->SuccAddrs.begin(),<br>
>> + SE = BBI->SuccAddrs.end();<br>
>> + SE != SE; ++SI) {<br>
>><br>
>><br>
>> thanks,<br>
>> --lx<br>
>> _______________________________________________<br>
>> LLVM Developers mailing list<br>
>> <a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
> _______________________________________________<br>
> LLVM Developers mailing list<br>
> <a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
</blockquote></div>