[PATCH] D45260: COFF: Layout sections in the same order as link.exe
Hans Wennborg via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 4 11:39:39 PDT 2018
hans added inline comments.
================
Comment at: COFF/Writer.cpp:660-662
+ .Case(".rsrc", INT_MAX - 1)
+ .Case(".reloc", INT_MAX)
+ .Default(INT_MAX / 2); // Default to somewhere in the middle.
----------------
ruiu wrote:
> This is a nit but I'd perhaps use 99, 100 and 50 or something like that for these numbers instead of super large numbers.
Will do.
================
Comment at: COFF/Writer.cpp:661
+ .Case(".rsrc", INT_MAX - 1)
+ .Case(".reloc", INT_MAX)
+ .Default(INT_MAX / 2); // Default to somewhere in the middle.
----------------
pcc wrote:
> Have you confirmed that UpdateResource updates the `.reloc` address in the data directory if it is moved?
Let me check..
https://reviews.llvm.org/D45260
More information about the llvm-commits
mailing list