[llvm] r196257 - Disable Remote MCJIT tests on ARM

[Alp Toker]

On 03/12/2013 08:39, Renato Golin wrote:
> Author: rengolin
> Date: Tue Dec  3 02:39:15 2013
> New Revision: 196257
>
> URL: http://llvm.org/viewvc/llvm-project?rev=196257&view=rev
> Log:
> Disable Remote MCJIT tests on ARM
>
> The communication protocol is unstable on ARM when compiled
> with Clang, which is disrupting the self-hosting buildbots that
> are going to be added this week. I'm working on a solution, but
> remote MCJIT is not high-priority for ARM at the moment, so it
> might take a while.

Hi Renato,

I had a few questions in the review thread for this patch, did you see?

It's OK to skip over it on ARM, but I'd like to see a PR reference to 
track the problem given that socket programming errors could affect 
other platforms.

Alp.


>
> Modified:
>      llvm/trunk/test/ExecutionEngine/MCJIT/remote/lit.local.cfg
>
> Modified: llvm/trunk/test/ExecutionEngine/MCJIT/remote/lit.local.cfg
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/MCJIT/remote/lit.local.cfg?rev=196257&r1=196256&r2=196257&view=diff
> ==============================================================================
> --- llvm/trunk/test/ExecutionEngine/MCJIT/remote/lit.local.cfg (original)
> +++ llvm/trunk/test/ExecutionEngine/MCJIT/remote/lit.local.cfg Tue Dec  3 02:39:15 2013
> @@ -1,3 +1,2 @@
> -if 'armv4' in config.root.target_triple or \
> -        'armv5' in config.root.target_triple:
> +if 'arm' in config.root.target_triple:
>       config.unsupported = True
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

-- 
http://www.nuanti.com
the browser experts