[PATCH] D54489: Implement -frecord-command-line (-frecord-gcc-switches)

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 16 13:02:35 PST 2018


rjmccall added inline comments.


================
Comment at: docs/ClangCommandLineReference.rst:799
+command-line. Command-lines in the section are surrounded and separated by null
+bytes. Spaces and backslashes in the command-line are escaped with backslashes.
+(ELF Only)
----------------
How about something like "After linking, the section may contain multiple command lines, which will be individually terminated by null bytes.  Separate arguments within a command line are combined with spaces; spaces and backslashes within an argument are escaped with backslashes."

And please write out "(ELF Only)" as something more like "This option is only currently supported on ELF targets."


https://reviews.llvm.org/D54489





More information about the cfe-commits mailing list