[PATCH] D122983: [C11/C2x] Change the behavior of the implicit function declaration warning

[Paul Walker via Phabricator via cfe-commits]

paulwalker-arm added a comment.

Thanks for this.  I can see about cleaning up the tests but I'm still not sure what the advantage is.  The affected RUN lines are already `-fsyntax-only` tests.  Is it about where the test files live or is there something else I should be considering?  The benefit of the current tests is that it's easy to spot holes as a single function tests both requirements.  My fear is that separating the tests based on Sema/CodeGen could mean we'll miss something and never know.


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

https://reviews.llvm.org/D122983