[PATCH] D26981: Limit maximum number of errors to 1000.

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 22 10:35:02 PST 2016


ruiu added a comment.

This makes sense to the COFF linker indeed. But the ELF linker and COFF linkers are different (for example, the COFF linker doesn't support library use, so it can always exit() on too many error), I'd implement it into COFF as a separate patch rather than trying to integrate.

We'll probably make change to the relocation errors, so that they'll include filenames and offsets. It is just a negligence that we are printing out the exact same error.

I wonder in what situation you want to make the maximum number configurable. Isn't 1000 enough?


https://reviews.llvm.org/D26981





More information about the llvm-commits mailing list