[PATCH] Add option to use temporary file for assembling with clang

David Peixotto dpeixott at codeaurora.org
Thu Dec 5 10:06:30 PST 2013


> It seems a reasonable dev option to have.
> * Can you test that the file is deleted?

Is there an easy way to test for this? I looked for another example and could not find one. I can think of writing a test that executes a compilation with that flag and -v, then looks to see if the output file exists. I see a couple of issues with that test, however

1. It will only work for a specific target because it requires compilation to complete (e.g. REQUIRES: arm-registered-target)

2. How do I make a cross-platform check to see if a file exists? I could use ls or or something on linux/darwin, but what about windows?

Is it ok for the test to work for a specific target and exclude windows (is there a way to do that?)? Or is there a better way to write the test?

> * Do we have a more regular naming patter for developer options?
> Should this be a -ccc-* option? Should be be a InternalDebugOpt?

I welcome suggestions on naming the flag and putting it in the right group.






More information about the cfe-commits mailing list