[PATCH] D75395: [clang][Modules] Add -fsystem-module flag

Duncan P. N. Exon Smith via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 2 14:09:05 PST 2020


dexonsmith added inline comments.


================
Comment at: clang/include/clang/Driver/Options.td:1448
   Flags<[CC1Option]>, Alias<fmodule_name_EQ>;
+def fsystem_module : Flag<["-"], "fsystem-module">, Flags<[CC1Option]>,
+  HelpText<"Build this module as a system module. Only used with -emit-module">;
----------------
bruno wrote:
> I wonder if `-isystem-module` wouldn't be better since it's kinda similar with `-isystem` for headers, but for modules.
FWIW, I prefer `-fsystem-module`.  I would expect `-i*` to be modifying search paths, and this option does not do that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75395





More information about the cfe-commits mailing list