<div dir="ltr">Fixed in r183601.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jun 7, 2013 at 5:46 PM, JF Bastien <span dir="ltr"><<a href="mailto:jfb@google.com" target="_blank">jfb@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote"><div><div class="h5"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<div dir="ltr">This is generating a warning for me:<div><br></div><div><div>/Users/sean/pg/llvm/touse/llvm/lib/Target/ARM/ARMFastISel.cpp:2664:12: warning: unused variable 'DestBits' [-Wunused-variable]</div><div>

  unsigned DestBits = DestVT.getSizeInBits();</div>
<div>           ^</div><div>1 warning generated.</div></div></div></blockquote><div><br></div></div></div><div>Sorry about that. The following diff should fix the issue (it's only used in the below asserts). OK to commit?</div>

<div><br></div><div><br></div><div><div>diff --git a/lib/Target/ARM/ARMFastISel.cpp b/lib/Target/ARM/ARMFastISel.cpp</div><div>index 16383bf..62bc782 100644</div><div>--- a/lib/Target/ARM/ARMFastISel.cpp</div>
<div>+++ b/lib/Target/ARM/ARMFastISel.cpp</div><div>@@ -2662,6 +2662,7 @@ unsigned ARMFastISel::ARMEmitIntExt(MVT SrcVT, unsigned SrcReg, MVT DestVT,</div><div> </div><div>   unsigned SrcBits = SrcVT.getSizeInBits();</div>

<div>   unsigned DestBits = DestVT.getSizeInBits();</div><div>+  (void) DestBits;</div><div class="im"><div>   assert((SrcBits < DestBits) && "can only extend to larger types");</div></div><div class="im">
<div>   assert((DestBits == 32 || DestBits == 16 || DestBits == 8) &&</div>
</div><div>          "other sizes unimplemented");</div><div><br></div></div></div></div></div>
</blockquote></div><br></div>