[llvm] r295268 - Remove useless local variable.

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 15 17:41:04 PST 2017


Author: ruiu
Date: Wed Feb 15 19:41:04 2017
New Revision: 295268

URL: http://llvm.org/viewvc/llvm-project?rev=295268&view=rev
Log:
Remove useless local variable.

Modified:
    llvm/trunk/lib/MC/WinCOFFObjectWriter.cpp

Modified: llvm/trunk/lib/MC/WinCOFFObjectWriter.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/MC/WinCOFFObjectWriter.cpp?rev=295268&r1=295267&r2=295268&view=diff
==============================================================================
--- llvm/trunk/lib/MC/WinCOFFObjectWriter.cpp (original)
+++ llvm/trunk/lib/MC/WinCOFFObjectWriter.cpp Wed Feb 15 19:41:04 2017
@@ -777,15 +777,13 @@ static std::time_t getTime() {
 
 void WinCOFFObjectWriter::writeObject(MCAssembler &Asm,
                                       const MCAsmLayout &Layout) {
-  size_t SectionsSize = Sections.size();
-  if (SectionsSize > static_cast<size_t>(INT32_MAX))
+  if (Sections.size() > INT32_MAX)
     report_fatal_error(
         "PE COFF object files can't have more than 2147483647 sections");
 
-  // Assign symbol and section indexes and offsets.
-  int32_t NumberOfSections = static_cast<int32_t>(SectionsSize);
-
-  UseBigObj = NumberOfSections > COFF::MaxNumberOfSections16;
+  UseBigObj = Sections.size() > COFF::MaxNumberOfSections16;
+  Header.NumberOfSections = Sections.size();
+  Header.NumberOfSymbols = 0;
 
   // Assign section numbers.
   size_t Number = 1;
@@ -796,9 +794,6 @@ void WinCOFFObjectWriter::writeObject(MC
     ++Number;
   }
 
-  Header.NumberOfSections = NumberOfSections;
-  Header.NumberOfSymbols = 0;
-
   for (const std::string &Name : Asm.getFileNames()) {
     // round up to calculate the number of auxiliary symbols required
     unsigned SymbolSize = UseBigObj ? COFF::Symbol32Size : COFF::Symbol16Size;




More information about the llvm-commits mailing list