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

Richard Sandiford rsandifo at linux.vnet.ibm.com
Fri Dec 6 02:21:55 PST 2013


Rafael EspĂ­ndola <rafael.espindola at gmail.com> writes:
> 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.

Not helpful feedback, sorry, but just a +1 for the patch.  I'd resorted
to using a hacky wrapper script to test this locally but having a proper
option for it would be much more convenient.  Thanks David!

Richard





More information about the cfe-commits mailing list