[cmake-developers] [PATCH for bug 15253] Ninja: Pass only the flags relevant to the language.

Brad King brad.king at kitware.com
Wed Jan 28 09:19:11 EST 2015


On 01/28/2015 09:06 AM, Brian Smith wrote:
> it seemed wrong to me to inherit the C/C++ include directories for
> assembly language compilations, so I removed it.

I don't think it is getting the C/C++ flags.  It's getting the
preprocessor flags.  Anyway, we agree to skip this hunk.

> Good point. Again, BoringSSL's assembly language code doesn't use
> ".include" so I'm not sure what is correct. However, both the nasm
> documentation and the yasm documentation say that "-M" should be used
> to generate makefile dependencies.
> 
> IMO, it is a good idea to fix this bug by committing a variant of the
> second hunk, and then file a new bug about correctly getting
> dependencies from nasm/yasm. If you agree, I'm happy to send an
> updated patch that removes the first hunk and that changes the deptype
> line in the second hunk to something else. In that case, what should I
> change the deptype line to?

The original issue could have been worded to explain that nasm/yasm
dependencies are not handled correctly, so I'd like to treat it that
way.  If these tools have the -M flag then a "deptype = gcc" code path
should work.  The usingMSVC boolean should be updated to be false
for lang != C|CXX|RC so that the "else" code path is taken.  Then

 CMAKE_DEPFILE_FLAGS_<LANG>

should be added to the platform modules for these toolchains.

Thanks,
-Brad



More information about the cmake-developers mailing list