rjmccall added inline comments.

================
Comment at: include/clang/Driver/CC1Options.td:707
+def fset_visibility_for_decls : Flag<["-"], "fset-visibility-for-decls">,
+  HelpText<"Apply global symbol visibility to declarations without an explicit 
visibility">;
 def ftemplate_depth : Separate<["-"], "ftemplate-depth">,
----------------
"Declaration" doesn't unambiguously mean "non-definition" in language parlance;
maybe something like "external declaration" in both help text and the option 
name?
So maybe `-fextern-visibility`?


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

https://reviews.llvm.org/D56868



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to