MaskRay added a comment. In D130138#3664913 <https://reviews.llvm.org/D130138#3664913>, @iains wrote:
> 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. Using remarks is suitable and the new name -Rmodule-include-translation looks good to me. Thanks for mentioning similar options from GCC. The names feel a bit odd to me as I am thinking what "lang-info" means. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130138/new/ https://reviews.llvm.org/D130138 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
