[PATCH] D42598: Add the /order option.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 26 15:37:34 PST 2018


ruiu added a comment.

As you pointed out, my last comment was wrong. The MSVC linker mangles symbols with "_" if it is on i386 and is not mangled yet.

I tried to add a warning message to warn on unused symbol names in symbol ordering file, but it seems a bit more complicated to do that than I originally thought, so I'll leave it for a follow-up patch.


https://reviews.llvm.org/D42598





More information about the llvm-commits mailing list