[lld] r186355 - [PECOFF][Writer] Follow up patch for r186336.

[Rui Ueyama]

Author: ruiu
Date: Mon Jul 15 17:20:10 2013
New Revision: 186355

URL: http://llvm.org/viewvc/llvm-project?rev=186355&view=rev
Log:
[PECOFF][Writer] Follow up patch for r186336.

- Make a const reference instead of copying an object
- Fix a comment

Modified:
    lld/trunk/lib/ReaderWriter/PECOFF/WriterPECOFF.cpp

Modified: lld/trunk/lib/ReaderWriter/PECOFF/WriterPECOFF.cpp
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/ReaderWriter/PECOFF/WriterPECOFF.cpp?rev=186355&r1=186354&r2=186355&view=diff
==============================================================================
--- lld/trunk/lib/ReaderWriter/PECOFF/WriterPECOFF.cpp (original)
+++ lld/trunk/lib/ReaderWriter/PECOFF/WriterPECOFF.cpp Mon Jul 15 17:20:10 2013
@@ -612,13 +612,13 @@ public:
     PageOffsetT blocks = groupByPage(relocSites);
     for (auto &i : blocks) {
       uint64_t pageAddr = i.first;
-      std::vector<uint16_t> offsetsInPage = i.second;
+      const std::vector<uint16_t> &offsetsInPage = i.second;
       appendAtom(createBaseRelocBlock(_file, pageAddr, offsetsInPage));
     }
   }
 
 private:
-  // When loaded into memory, data section should be readable and writable.
+  // When loaded into memory, reloc section should be readable and writable.
   static const uint32_t characteristics =
       llvm::COFF::IMAGE_SCN_MEM_READ |
       llvm::COFF::IMAGE_SCN_CNT_INITIALIZED_DATA;
@@ -644,7 +644,7 @@ private:
 
   // Create the content of a relocation block.
   DefinedAtom *createBaseRelocBlock(const File &file, uint64_t pageAddr,
-                                    std::vector<uint16_t> &offsets) {
+                                    const std::vector<uint16_t> &offsets) {
     uint32_t size = 8 + offsets.size() * 2;
     std::vector<uint8_t> contents(size);