[lld] [lld][ELF] Skip finalizeAddressDependentContent if assignAddresses produces errors. (PR #75581)

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Sat Dec 16 10:21:57 PST 2023


https://github.com/MaskRay requested changes to this pull request.

This is a minor known problem about repeated diagnostics. We don't have too tests, so I added one while refactoring `PROVIDED` tests in 215c5656449c8e817a3759d989f27ba39d711cbd 2 days ago.

This patch just removes one repeated diagnostic. We have another error in `script->processSymbolAssignments();`. Adding `if (errorCount()) return;` there can remove the 2 repeats but `-Map` output would be less precise.

https://github.com/llvm/llvm-project/pull/75581


More information about the llvm-commits mailing list