[PATCH] Add flag -f(no-)modules-implicit-maps.
Richard Smith
richard at metafoo.co.uk
Mon Nov 24 10:38:04 PST 2014
One comment to address, then please go ahead and commit.
================
Comment at: lib/Frontend/CompilerInvocation.cpp:1467
@@ -1465,1 +1466,3 @@
+ Args.hasFlag(OPT_fmodules_implicit_maps, OPT_fno_modules_implicit_maps,
+ Opts.ModulesImplicitMaps);
Opts.CharIsSigned = Opts.OpenCL || !Args.hasArg(OPT_fno_signed_char);
----------------
You're initializing this from itself if neither flag is set. Did you mean `Opts.Modules` or `Opts.ModuleMaps`, or maybe `true`?
http://reviews.llvm.org/D6305
More information about the cfe-commits
mailing list