[PATCH] D84572: Allow .dSYM's to be directly placed in an alternate directory

Jonas Devlieghere via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 27 09:39:24 PDT 2020


JDevlieghere added inline comments.


================
Comment at: clang/include/clang/Driver/Options.td:693
 def e : JoinedOrSeparate<["-"], "e">, Group<Link_Group>;
+def external_dsym_dir : JoinedOrSeparate<["-"], "external-dsym-dir">,
+  Flags<[DriverOption, RenderAsInput]>,
----------------
Could we drop the `external` prefix or did you pick this for consistency with other flags? It seems rather specific for the externalize debug info scenario while the functionality seems generic enough by itself. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84572





More information about the cfe-commits mailing list