[PATCH] Add "native" to config.available_features, to make it easier to disable non-x-compile-safe tests
Artyom.Skrobov at arm.com
Fri Sep 13 05:29:48 PDT 2013
Thank you Renato.
Taking this over to cfe-commits, to get some more feedback on x_compile_tests_clang.patch, and more specifically, on the cross-compilation compatibility of test/Index/cindex-from-source.m and test/Index/complete-pch.m
Our experience shows that the special testing executables used by these tests (clang-test and c-index-test) always assume the host arch.
So these tests create a PCH file using the target arch, then test it with one of the above apps using the host arch.
From: Renato Golin [mailto:renato.golin at linaro.org]
Sent: 13 September 2013 10:39
To: Artyom Skrobov
Cc: llvm-commits at cs.uiuc.edu
Subject: Re: [PATCH] Add "native" to config.available_features, to make it easier to disable non-x-compile-safe tests
That's a good idea. Patch looks good, too.
On 13 September 2013 09:10, Artyom Skrobov <Artyom.Skrobov at arm.com> wrote:
We're proposing adding "native" to config.available_features when host arch == target arch; then, tests which require native compilation environment can be marked as "REQUIRES: native".
As the first use case, we're updating two clang tests to include "REQUIRES: native", and marking the whole test/ExecutionEngine and test/ExecutionEngine/MCJIT directories as unsupported for cross-compilation.
-- IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.
ARM Limited, Registered office 110 Fulbourn Road, Cambridge CB1 9NJ, Registered in England & Wales, Company No: 2557590
ARM Holdings plc, Registered office 110 Fulbourn Road, Cambridge CB1 9NJ, Registered in England & Wales, Company No: 2548782
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 2354 bytes
More information about the cfe-commits