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

Rafael EspĂ­ndola rafael.espindola at gmail.com
Thu Dec 5 10:36:35 PST 2013


On 5 December 2013 13:06, David Peixotto <dpeixott at codeaurora.org> wrote:
>> 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)

Yes, that is what I had in mind, but you are right, it doesn't seem worth it.

The option is fine by be, just wait a bit longer to see if there is
more feedback.

Cheers,
Rafael




More information about the cfe-commits mailing list