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, 
   Alias<save_temps_EQ>, AliasArgs<["cwd"]>,
pcc wrote:
> tejohnson wrote:
> > 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).
> You are producing a `-save-temps=` flag, not `-save-temps`. 
> i.e. I think you should be able to remove the CC1Option from this line and 
> keep it on line 2253.
Oh, I see what you are saying. Removed CC1Option from the plain -save-temps.

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 
+; 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:
> tejohnson wrote:
> > pcc wrote:
> > > Should there be another test for `-save-temps=cwd`?
> > Added -save-temps=cwd and -save-temps
> This isn't actually testing that the behaviour is different between 
> `-save-temps=obj` and `-save-temps=cwd`, right?
> Maybe create a new directory, cd there and verify that the files are created 
> there?
Using different directories for all 3 cases now.

  rC Clang


cfe-commits mailing list

Reply via email to