[PATCH] D67039: [ELF] Add a spell corrector for "undefined symbol" diagnostics

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 3 04:32:06 PDT 2019


thakis added a comment.

>> For this to fire and be a true positive, you'd have to have an incorrect declaration but a correct definition of a function. I can't remember ever running into this – and the compiler seems to be in a better place to print a diagnostic for this case in almost all cases (except for if the TU with the correct definition doesn't happen to see the incorrect declaration).
> 
> The file that provides the definition may have a mismatch of the declaration and the definition, e.g.

Well, I guess we'll see how useful it turns out to be in practice :)


Repository:
  rLLD LLVM Linker

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67039/new/

https://reviews.llvm.org/D67039





More information about the llvm-commits mailing list