@aciidb0mb3r do you know what is left here? I know we still are missing support for Obj-C/C++ source suffixes, I'm not exactly sure what else there is. We also should ensure we have test coverage for everything we support.

except obj-c/c++ everything in proposal is implemented and I think we have pretty good test coverage for C modules features we support (eg iquote, modulemap generation etc)
can you elaborate a bit on how objc/c++ is supposed to work? is it allowed to have mixture of c/objc/c++ code? do we need to use clang++ instead if language is cpp?

No, no mixed targets, we just need to recognize the full set of source suffixes. To start with, I am fine with using Clang's default arguments for each language, and we don't need to use `clang++` anywhere.

We should recognize these suffixes (case folded):

Objective-C: ["m"]
Objective-C++: ["mm"]
C++: ["cc", "cpp", "cxx"]

and for headers we should also recognize:

C++: ["hpp", "inc"]

Historically some system would treat uppercase suffixes as C++, but I don't think that is a behavior we should replicate since it is hard to ensure reliable cross platform behavior. Any users of such conventions will have to migrate.

@ddunbar Do you think anything else is pending or can this be marked resolved now?

This is done.

Comment by Myke Olson (JIRA)

@swift-ci create

Comment by Myke Olson (JIRA)

Need to sync to radar and I can't modify the radar URL field when it's closed. Will close again in a minute.