[PATCH] D30760: Record command lines in objects built by clang, Clang part

George Burgess IV via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 24 15:14:26 PDT 2017


george.burgess.iv added inline comments.


================
Comment at: test/Driver/debug-options.c:201
 //
+// GRECORD: "-dwarf-debug-flags"
+// GRECORD: -### -c -grecord-gcc-switches
----------------
echristo wrote:
> This seems a little light on the testing, would you mind adding some more interesting lines here? (Also, -grecord-gcc-switches seems like an option we might want to ignore for this?)
re-pasting my comment, since it looks like it got dropped when we updated diffs:

nit: since `RenderAsInput` args (`-Xlinker`, `-o`, ...) have cases where they're printed without their arg, can we check that at least `-o -` is printed sanely, as well?


https://reviews.llvm.org/D30760





More information about the cfe-commits mailing list