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

[Ed Maste via Phabricator via llvm-commits]

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.


https://reviews.llvm.org/D27716