[PATCH] D23989: Fix -Wunused-but-set-variable warning.

Haojian Wu via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 29 05:34:51 PDT 2016


This revision was automatically updated to reflect the committed changes.
Closed by commit rL279964: Fix -Wunused-but-set-variable warning. (authored by hokein).

Changed prior to commit:
  https://reviews.llvm.org/D23989?vs=69554&id=69559#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D23989

Files:
  llvm/trunk/lib/Target/X86/X86ISelLowering.cpp

Index: llvm/trunk/lib/Target/X86/X86ISelLowering.cpp
===================================================================
--- llvm/trunk/lib/Target/X86/X86ISelLowering.cpp
+++ llvm/trunk/lib/Target/X86/X86ISelLowering.cpp
@@ -14617,25 +14617,21 @@
 
   MVT LogicVT;
   MVT EltVT;
-  unsigned NumElts;
 
   if (VT.isVector()) {
     LogicVT = VT;
     EltVT = VT.getVectorElementType();
-    NumElts = VT.getVectorNumElements();
   } else if (IsF128) {
     // SSE instructions are used for optimized f128 logical operations.
     LogicVT = MVT::f128;
     EltVT = VT;
-    NumElts = 1;
   } else {
     // There are no scalar bitwise logical SSE/AVX instructions, so we
     // generate a 16-byte vector constant and logic op even for the scalar case.
     // Using a 16-byte mask allows folding the load of the mask with
     // the logic op, so it can save (~4 bytes) on code size.
     LogicVT = (VT == MVT::f64) ? MVT::v2f64 : MVT::v4f32;
     EltVT = VT;
-    NumElts = (VT == MVT::f64) ? 2 : 4;
   }
 
   unsigned EltBits = EltVT.getSizeInBits();


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D23989.69559.patch
Type: text/x-patch
Size: 1052 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160829/2bb085f1/attachment.bin>


More information about the cfe-commits mailing list