rephrase diagnostic

[hkareemllc]

Removed Objective-C from diagnostic message to so that it is generic.
Resolves SR-2409.

[jrose-apple]

Hm, that's a little more than I meant. It's still useful to know that it's an imported C or Objective-C module instead of a Swift module. How about using theASTContext.LangOpts.EnableObjCInterop to choose between "C" and "Objective-C"?

[jrose-apple]

Also, can you make the commit message a little more descriptive, and reference the SR there? That way people don't have to come find the pull request or look at the diff to see what's going on.

[jrose-apple]

Thanks for taking this!

[hkareemllc]

@jrose-apple so would i change it to could not build theASTContext.LangOpts.EnableObjCInterop module 'Foo'?

[jrose-apple]

Not quite. If you look at the other diagnostics, you can see that some of them take arguments and use a %select construct to choose which text to show. So we'd pass the value of EnableObjCInterop along when we emit the diagnostic, and get the correct text out.

[CodaFi]

@hkareem2222 Ping.

[CodaFi]

This pull request has been superseded by #8147. Thank you for your patch @hkareem2222