[PATCH] D27716: [ELF] - Implemented --retain-symbols-file option

Ed Maste via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 13 09:30:17 PST 2016

emaste added a comment.

How do we handle combinations of related options?

Comment at: ELF/Config.h:39
+// For --discard-{all,locals,none} and --retain-symbols-file.
+enum class DiscardPolicy { Default, All, Locals, File, None };
DiscardPolicy File seems confusing to me - in that it seems File would list the symbols to discard. Not sure of a good word to encompass Discard/Retain though.


More information about the llvm-commits mailing list