[PATCH] D13934: [ELF2] --strip-all/-s command line implemented

[George Rimar via llvm-commits]

grimar added inline comments.

================
Comment at: ELF/Config.h:59
@@ -58,2 +58,3 @@
+  bool StripAll;
   bool Static = false;
   bool Verbose;
----------------
davide wrote:
> This is not sorted.
Will fix.
I forgot that after first two characters there are still other ones, sorry :)

================
Comment at: ELF/Options.td:86
@@ +85,3 @@
+def strip_all : Flag<["--"], "strip-all">,
+  HelpText<"Omit all symbol information from the output file">;
+
----------------
davide wrote:
> Can't you just say "strip all symbols" as gold and ld do?
I took it from man ld, but will replace if you wish.

================
Comment at: test/elf2/strip-all.s:1
@@ +1,2 @@
+# Tests the --strip-all (-s) flag.
+# We expect that .symtab and .strtab are stripped from the final executable.
----------------
davide wrote:
> Please do not use obj2yaml, but use assembler for tests. This is the standard for the new ELF linker.
Ok. I expected that. Thats explains why I never saw its usings in new ELF branch.


http://reviews.llvm.org/D13934