[PATCH] D18183: [ELF] - -pie/--pic-executable option implemented

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 16 14:01:21 PDT 2016


grimar added inline comments.

================
Comment at: ELF/Options.td:102
@@ -100,1 +101,3 @@
+  HelpText<"Create a position independent executable">;
+
 def print_gc_sections: Flag<["--"], "print-gc-sections">,
----------------
ruiu wrote:
> grimar wrote:
> > btw, not about this patch, but in general.
> > If we have errors/warnings in lowercase. Do we want HelpText here to be lowercase as well ? I guess yes.
> I don't know which is better, but this is where we just want to follow the convention. What do other command do for --help?
Hmm. clang, ld and gold has uppercase for help text. So seems no need to change anything here.


http://reviews.llvm.org/D18183





More information about the llvm-commits mailing list