[llvm] r200102 - Re-enabling MCJIT tests on ARM

[Alp Toker]

Glad I was able to fix it!

I think we should propose an update to the developer policy to indicate 
that an effort should to be made to support the current set of 
mainstream platforms when adding new features that aren't inherently 
platform-specific.

Would you like to have a shot at it or shall I draft something up?

Alp.



On 25/01/2014 23:38, Renato Golin wrote:
> Author: rengolin
> Date: Sat Jan 25 17:38:08 2014
> New Revision: 200102
>
> URL: http://llvm.org/viewvc/llvm-project?rev=200102&view=rev
> Log:
> Re-enabling MCJIT tests on ARM
>
> After several refactorings on the MCJIT remote communication, things are
> finally looking good on Clang-compiled LLVM regarding MCJIT remote tests,
> so I'm re-enabling them to see how the self-hosting buildbot behaves over
> a longer period.
>
> 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=200102&r1=200101&r2=200102&view=diff
> ==============================================================================
> --- llvm/trunk/test/ExecutionEngine/MCJIT/remote/lit.local.cfg (original)
> +++ llvm/trunk/test/ExecutionEngine/MCJIT/remote/lit.local.cfg Sat Jan 25 17:38:08 2014
> @@ -4,5 +4,5 @@ if 'armv4' in config.root.target_triple
>   
>   # This is temporary, until Remote MCJIT works on ARM
>   # See http://llvm.org/bugs/show_bug.cgi?id=18057
> -if 'armv7' in config.root.target_triple:
> -    config.unsupported = True
> +#if 'armv7' 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