[PATCH] D42598: Add the /order option.

[Rui Ueyama via Phabricator via llvm-commits]

ruiu added inline comments.


================
Comment at: lld/COFF/Driver.cpp:781
+  for (StringRef S : args::getLines(MB->getMemBufferRef()))
+    Config->Order[S] = INT_MIN + Config->Order.size();
+}
----------------
pcc wrote:
> According to https://msdn.microsoft.com/en-us/library/00kh39zz.aspx it looks like the names need to be mangled on 32-bit platforms.
Actually that document is wrong. I added a comment about that.


https://reviews.llvm.org/D42598