[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