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:[email protected]]
Sent: 13 September 2013 10:39
To: Artyom Skrobov
Cc: [email protected]
Subject: Re: [PATCH] Add "native" to config.available_features, to make it 
easier to disable non-x-compile-safe tests

Hi Artyom,

That's a good idea. Patch looks good, too.

cheers,
--renato

On 13 September 2013 09:10, Artyom Skrobov <[email protected]> wrote:
Hello,

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

Attachment: x_compile_tests_clang.patch
Description: x_compile_tests_clang.patch

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to