[llvm] r183551 - ARM FastISel integer sext/zext improvements

JF Bastien jfb at google.com
Fri Jun 7 17:46:33 PDT 2013


> This is generating a warning for me:
>
> /Users/sean/pg/llvm/touse/llvm/lib/Target/ARM/ARMFastISel.cpp:2664:12:
> warning: unused variable 'DestBits' [-Wunused-variable]
>   unsigned DestBits = DestVT.getSizeInBits();
>            ^
> 1 warning generated.
>

Sorry about that. The following diff should fix the issue (it's only used
in the below asserts). OK to commit?


diff --git a/lib/Target/ARM/ARMFastISel.cpp b/lib/Target/ARM/ARMFastISel.cpp
index 16383bf..62bc782 100644
--- a/lib/Target/ARM/ARMFastISel.cpp
+++ b/lib/Target/ARM/ARMFastISel.cpp
@@ -2662,6 +2662,7 @@ unsigned ARMFastISel::ARMEmitIntExt(MVT SrcVT,
unsigned SrcReg, MVT DestVT,

   unsigned SrcBits = SrcVT.getSizeInBits();
   unsigned DestBits = DestVT.getSizeInBits();
+  (void) DestBits;
   assert((SrcBits < DestBits) && "can only extend to larger types");
   assert((DestBits == 32 || DestBits == 16 || DestBits == 8) &&
          "other sizes unimplemented");
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130607/258b7523/attachment.html>


More information about the llvm-commits mailing list