[PATCH] D38513: [LLD] [COFF] Add support for GNU binutils import libraries

Zachary Turner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 20 09:24:16 PDT 2018


zturner added inline comments.


================
Comment at: COFF/Writer.cpp:378
+    std::map<std::pair<StringRef, uint32_t>, std::vector<Chunk *>> &Map) {
+  const uint32_t RDATA = IMAGE_SCN_CNT_INITIALIZED_DATA | IMAGE_SCN_MEM_READ;
+
----------------
ruiu wrote:
> nit: we omit `const` if it is obviously a const.
Do we?  It doesn't hurt anything and prevents the user from modifying the value later.  I don't see a good reason to omit it.


https://reviews.llvm.org/D38513





More information about the llvm-commits mailing list