[PATCH] D34541: [COFF] Fix SECTION and SECREL relocation handling for absolute symbols

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 22 16:31:11 PDT 2017


rnk marked an inline comment as done.
rnk added inline comments.


================
Comment at: lld/COFF/Symbols.cpp:64
 
+unsigned DefinedAbsolute::OutputSectionIndex = 0;
+
----------------
ruiu wrote:
> You want to use `uint32_t`.
True. uint16_t would probably be best.


https://reviews.llvm.org/D34541





More information about the llvm-commits mailing list