r193901 - Fixes lit test that doesn't fail on windows

Richard Smith richard at metafoo.co.uk
Fri Nov 1 17:18:58 PDT 2013


Please add a target triple to this test.


On Fri, Nov 1, 2013 at 5:12 PM, Warren Hunt <whunt at google.com> wrote:

> Author: whunt
> Date: Fri Nov  1 19:12:15 2013
> New Revision: 193901
>
> URL: http://llvm.org/viewvc/llvm-project?rev=193901&view=rev
> Log:
> Fixes lit test that doesn't fail on windows
>
> long in linux 64 is 64 bits but is always 32 bits on windows.  The lit
> test was modified
> to use long long instead of long and check for 64-bit mangling.
>
>
> Modified:
>     cfe/trunk/test/CodeGenCXX/builtins.cpp
>
> Modified: cfe/trunk/test/CodeGenCXX/builtins.cpp
> URL:
> http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGenCXX/builtins.cpp?rev=193901&r1=193900&r2=193901&view=diff
>
> ==============================================================================
> --- cfe/trunk/test/CodeGenCXX/builtins.cpp (original)
> +++ cfe/trunk/test/CodeGenCXX/builtins.cpp Fri Nov  1 19:12:15 2013
> @@ -17,13 +17,13 @@ S *addressof(bool b, S &s, S &t) {
>  }
>
>  extern "C" int __builtin_abs(int); // #1
> -long __builtin_abs(long);          // #2
> +long long __builtin_abs(long long);          // #2
>  extern "C" int __builtin_abs(int); // #3
>
>  int x = __builtin_abs(-2);
>  // CHECK: entry:
>  // CHECK-NEXT:  store i32 2, i32* @x, align 4
>
> -long y = __builtin_abs(-2l);
> +long y = __builtin_abs(-2ll);
>  // CHECK: entry:
> -// CHECK-NEXT:  %call = call i32 @_Z13__builtin_absl(i32 -2)
> +// CHECK-NEXT:  %call = call i64 @_Z13__builtin_absx(i64 -2)
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20131101/3316671f/attachment.html>


More information about the cfe-commits mailing list