[PATCH] D41113: [LLVMgold] Don't set resolutions for undefined symbols to 'Prevailing'

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 13 10:53:22 PST 2017


pcc added a comment.

> I now wonder if we can simply ignore empty symbol name with this

Doesn't seem like a good idea to me. It is an error for a linker to mark more than one copy of a symbol as prevailing, and the assert would allow us to check for that property in most cases.


https://reviews.llvm.org/D41113





More information about the llvm-commits mailing list