tejohnson marked 2 inline comments as done.
tejohnson added inline comments.
================
Comment at: include/clang/Driver/Options.td:2255
HelpText<"Save intermediate compilation results.">;
-def save_temps : Flag<["-", "--"], "save-temps">, Flags<[DriverOption]>,
+def save_temps : Flag<["-", "--"], "save-temps">, Flags<[CC1Option,
DriverOption]>,
Alias<save_temps_EQ>, AliasArgs<["cwd"]>,
----------------
pcc wrote:
> This is still just a DriverOption, right?
No, we need to pass to cc1 to get it into the CodeGenOptions. See the changes
to Clang.cpp (passes to cc1 invocation) and CompilerInvocation.cpp (consumes
during cc1 invocation).
================
Comment at: lib/Frontend/CompilerInvocation.cpp:747
+ .Case("obj", FrontendOpts.OutputFile)
+ .Default("./" +
+ llvm::sys::path::filename(FrontendOpts.OutputFile).str());
----------------
pcc wrote:
> Why do you need the `"./"` part?
Not needed, removed.
================
Comment at: test/CodeGen/thinlto_backend.ll:30
; Ensure f2 was imported
-; RUN: %clang -target x86_64-unknown-linux-gnu -O2 -o %t3.o -x ir %t1.o -c
-fthinlto-index=%t.thinlto.bc
+; RUN: %clang -target x86_64-unknown-linux-gnu -O2 -o %t3.o -x ir %t1.o -c
-fthinlto-index=%t.thinlto.bc -save-temps=obj
; RUN: llvm-nm %t3.o | FileCheck --check-prefix=CHECK-OBJ %s
----------------
pcc wrote:
> Should there be another test for `-save-temps=cwd`?
Added -save-temps=cwd and -save-temps
Repository:
rC Clang
https://reviews.llvm.org/D45217
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
