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

[Peter Collingbourne via Phabricator via llvm-commits]

pcc added a comment.

This seems fine to me with the assert.

> There's even a probable current issue - if the prevailing def in the asm is marked dead, anything reachable from it might be marked dead as well (i.e. if the only use is in that def).

Module asm definitions do not refer to anything at the summary level, so it does not matter if we mark them as dead. Any symbols that are referenced from module inline asm are GC roots because of https://reviews.llvm.org/D32544.


https://reviews.llvm.org/D41113