[PATCH] D14382: [ELF2] - Basic implementation of -r/--relocatable

[Rafael Espíndola via llvm-commits]

> ----------------
> rafael wrote:
>> Can you get here? If so, add a test, if not, use an assert.
> I just assumed that users who will try linker probably will use the release version. So if them meet that error situation, there is no way they see the assert, it will not be debug build for them.
> error() is universal way to prompt here. I would use it or fatal() here.

If it cannot be reached this is dead code and should be an assert.
Never add an error or fatal that cannot actually happen.

Cheers,
Rafael