[PATCH] D142933: Add -print-multi-selection-flags-experimental option

Petr Hosek via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 6 22:40:45 PDT 2023


phosek added inline comments.


================
Comment at: clang/include/clang/Driver/Options.td:4261
 def print_multi_lib : Flag<["-", "--"], "print-multi-lib">;
+def print_multi_selection_flags : Flag<["-", "--"], "print-multi-selection-flags-experimental">,
+  HelpText<"Print the flags used for selecting multilibs (experimental)">;
----------------
This is just a suggestion, but I'd consider dropping `selection`, that is the name of the flag would be just `-print-multi-flags`. It's not clear to me what the "selection" means in this context and I'm not sure if it provides any additional value.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142933



More information about the cfe-commits mailing list