[PATCH] D130138: [modules] Replace `-Wauto-import` with `-Rmodule-include-translation`.

Iain Sandoe via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 20 01:01:27 PDT 2022


iains added a subscriber: MaskRay.
iains added a comment.

makes sense to me...

I guess the name looks long at first, but it's specific (I find that easy-to-remember flag names are more important than short-to-type ones, but maybe that's just me) - I wonder if @MaskRay has any comments on the flag name,

For the record:
GCC uses : -flang-info-include-translate (-flang for flag-language, rather than 'flang', of course).
(which is a note, i.e. similar idea to the remark)

BTW.. (not for this patch, of course)
GCC also has 
info-include-translate-not
Note #include directives not translated to import declarations, and not known to be textual.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130138



More information about the cfe-commits mailing list