[PATCH] D83152: llvm-nm: add flag to suppress no symbols warning

Fri Feb 12 01:10:42 PST 2021

jhenderson added a comment.

In D83152#2557844 <https://reviews.llvm.org/D83152#2557844>, @MaskRay wrote:

> I don't agree. If the coordination is done first, we have a potential to use `--quiet` to right away. Then we don't need to have a temporary `--no-warning-for-no-symbols` which will eventually be removed (and potential to break users).

Whilst what you are saying is entirely true, the issue is that it unnecessarily (in my opinion) raises the bar on a contribution that is already small and quite simple. Not all developers know or care how to discuss things with the GNU community. Saying that they do makes it more likely that they will simply walk away from contributing what might otherwise be a useful patch. In this case, it's clearly not a code quality issue, nor is it a tricky thing for us to change in the future should some future developer decide they want to discuss the option with GNU/GNU independently go their own way.

> The `no symbols` has caused us lots of trouble (see the history). I imagined that binutils folks would have some opinions on it. Given the contentious previous discussions, I think it is entirely justified to ask them for an opinion.

Could you point out where these contentious previous discussions were? I looked back through the history, and there didn't seem to be any real issues - it was just about getting the llvm-nm implementation right so that it matches the GNU behaviour, but it's quite possible I missed something, or I might just be misunderstanding you. I don't think difficulties trying to get our implementation to match GNU's are really relevant to this patch, since this is about a new option, so won't impact the default behaviour.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83152