[llvm-commits] [llvm] r165390 - in /llvm/trunk: test/ExecutionEngine/2002-12-16-ArgTest.ll test/ExecutionEngine/test-fp-no-external-funcs.ll test/ExecutionEngine/test-fp.ll unittests/ExecutionEngine/JIT/JITTest.cpp

Jim Grosbach grosbach at apple.com
Mon Oct 8 18:32:11 PDT 2012


Sweet. Makes me happy to see JIT tests getting whipped into shape!

-Jim

On Oct 8, 2012, at 6:06 AM, James Molloy <James.Molloy at arm.com> wrote:

> Author: jamesm
> Date: Mon Oct  8 08:06:30 2012
> New Revision: 165390
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=165390&view=rev
> Log:
> Some regression tests which are testing the old jit and are exercising functionality which is both known to be broken and not expected to be fixed in the old jit. To remove these from the regression test output, I've marked them XFAIL (for lit tests) and ifdef'd them out (unit tests). These modifications remove the last long-standing regression test failures from the buildbots (though updating the triple to reflect new ubuntu configuration has temporarily caused some new failures). Tested on x86-64 and ARM Linux.
> 
> Patch by David Tweed!
> 
> 
> Modified:
>    llvm/trunk/test/ExecutionEngine/2002-12-16-ArgTest.ll
>    llvm/trunk/test/ExecutionEngine/test-fp-no-external-funcs.ll
>    llvm/trunk/test/ExecutionEngine/test-fp.ll
>    llvm/trunk/unittests/ExecutionEngine/JIT/JITTest.cpp
> 
> Modified: llvm/trunk/test/ExecutionEngine/2002-12-16-ArgTest.ll
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/2002-12-16-ArgTest.ll?rev=165390&r1=165389&r2=165390&view=diff
> ==============================================================================
> --- llvm/trunk/test/ExecutionEngine/2002-12-16-ArgTest.ll (original)
> +++ llvm/trunk/test/ExecutionEngine/2002-12-16-ArgTest.ll Mon Oct  8 08:06:30 2012
> @@ -1,4 +1,5 @@
> ; RUN: %lli %s > /dev/null
> +; XFAIL: arm
> 
> @.LC0 = internal global [10 x i8] c"argc: %d\0A\00"		; <[10 x i8]*> [#uses=1]
> 
> 
> Modified: llvm/trunk/test/ExecutionEngine/test-fp-no-external-funcs.ll
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/test-fp-no-external-funcs.ll?rev=165390&r1=165389&r2=165390&view=diff
> ==============================================================================
> --- llvm/trunk/test/ExecutionEngine/test-fp-no-external-funcs.ll (original)
> +++ llvm/trunk/test/ExecutionEngine/test-fp-no-external-funcs.ll Mon Oct  8 08:06:30 2012
> @@ -1,4 +1,5 @@
> ; RUN: %lli  %s > /dev/null
> +; XFAIL: arm
> 
> define double @test(double* %DP, double %Arg) {
> 	%D = load double* %DP		; <double> [#uses=1]
> 
> Modified: llvm/trunk/test/ExecutionEngine/test-fp.ll
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/test-fp.ll?rev=165390&r1=165389&r2=165390&view=diff
> ==============================================================================
> --- llvm/trunk/test/ExecutionEngine/test-fp.ll (original)
> +++ llvm/trunk/test/ExecutionEngine/test-fp.ll Mon Oct  8 08:06:30 2012
> @@ -1,4 +1,5 @@
> ; RUN: %lli %s > /dev/null
> +; XFAIL: arm
> 
> define double @test(double* %DP, double %Arg) {
> 	%D = load double* %DP		; <double> [#uses=1]
> 
> Modified: llvm/trunk/unittests/ExecutionEngine/JIT/JITTest.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/unittests/ExecutionEngine/JIT/JITTest.cpp?rev=165390&r1=165389&r2=165390&view=diff
> ==============================================================================
> --- llvm/trunk/unittests/ExecutionEngine/JIT/JITTest.cpp (original)
> +++ llvm/trunk/unittests/ExecutionEngine/JIT/JITTest.cpp Mon Oct  8 08:06:30 2012
> @@ -224,6 +224,9 @@
>   OwningPtr<ExecutionEngine> TheJIT;
> };
> 
> +// Tests on ARM disabled as we're running the old jit
> +#if !defined(__arm__)
> +
> // Regression test for a bug.  The JIT used to allocate globals inside the same
> // memory block used for the function, and when the function code was freed,
> // the global was left in the same place.  This test allocates a function
> @@ -292,6 +295,8 @@
>   EXPECT_EQ(3, *GPtr);
> }
> 
> +#endif // !defined(__arm__)
> +
> int PlusOne(int arg) {
>   return arg + 1;
> }
> @@ -521,6 +526,9 @@
> }
> #endif  // !ARM && !PPC
> 
> +// Tests on ARM disabled as we're running the old jit
> +#if !defined(__arm__)
> +
> TEST_F(JITTest, FunctionPointersOutliveTheirCreator) {
>   TheJIT->DisableLazyCompilation(true);
>   LoadAssembly("define i8()* @get_foo_addr() { "
> @@ -555,6 +563,8 @@
> #endif
> }
> 
> +#endif //!defined(__arm__)
> +
> // ARM does not have an implementation
> // of replaceMachineCodeForFunction(), so recompileAndRelinkFunction
> // doesn't work.
> @@ -599,6 +609,9 @@
> int32_t JITTest_AvailableExternallyGlobal = 42;
> namespace {
> 
> +// Tests on ARM disabled as we're running the old jit
> +#if !defined(__arm__)
> +
> TEST_F(JITTest, AvailableExternallyGlobalIsntEmitted) {
>   TheJIT->DisableLazyCompilation(true);
>   LoadAssembly("@JITTest_AvailableExternallyGlobal = "
> @@ -615,7 +628,7 @@
>   EXPECT_EQ(42, loader()) << "func should return 42 from the external global,"
>                           << " not 7 from the IR version.";
> }
> -
> +#endif //!defined(__arm__)
> }  // anonymous namespace
> // This function is intentionally defined differently in the statically-compiled
> // program from the IR input to the JIT to assert that the JIT doesn't use its
> 
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits




More information about the llvm-commits mailing list