[PATCH] D56675: [lld] add options for context-sensitive PGO.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 14 10:45:16 PST 2019


ruiu added inline comments.


================
Comment at: ELF/Options.td:472-475
+def: F<"plugin-opt=cs-profile-generate">,
+  Alias<lto_cs_pgo_gen>, HelpText<"Alias for -lto-cs-pgo-gen">;
+def: J<"plugin-opt=cs-profile-path=">,
+  Alias<lto_cs_profile_file>, HelpText<"Alias for -lto-cs-profile-file">;
----------------
Actually, do you have to add these options? `-plugin-opt=foo` is an option for backward compatibility with gold. Since lld has a native LTO support, "plugin opt" doesn't make sense anymore. If they are new options, you shouldn't add `-plugin-opt` aliases.


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

https://reviews.llvm.org/D56675





More information about the llvm-commits mailing list