arnamoy10 added a comment. OK, I will merge it on Monday unless @tskeith has any other comments.
================ Comment at: clang/include/clang/Driver/Options.td:4232-4233 HelpText<"Use INTEGER(KIND=8) for the result type in size-related intrinsics.">; +def fdebug_module_writer : Flag<["-"],"fdebug-module-writer">, Group<f_Group>, + HelpText<"Enables showing debug messages while writing module files.">; } ---------------- arnamoy10 wrote: > awarzynski wrote: > > IMO this should be a frontend-only option. Compiler end-users are unlikely > > to need this in their regular workflows, right? I recommend moving it [[ > > https://github.com/llvm/llvm-project/blob/adfd3c7083f9808d145239153c10f72eece485d8/clang/include/clang/Driver/Options.td#L4262-L4272 > > | here ]]. > Yes, it makes sense to keep it as frontend-only option, let me submit an > updated patch. > IMO this should be a frontend-only option. Compiler end-users are unlikely to > need this in their regular workflows, right? I recommend moving it [[ > https://github.com/llvm/llvm-project/blob/adfd3c7083f9808d145239153c10f72eece485d8/clang/include/clang/Driver/Options.td#L4262-L4272 > | here ]]. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96875/new/ https://reviews.llvm.org/D96875 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
