[llvm-commits] [llvm] r93189 - /llvm/trunk/test/CodeGen/X86/2010-01-07-UAMemFeature.ll

[Jeffrey Yasskin]

I just committed r93279 to fix this on ARM and PPC, which it crashed:
http://google1.osuosl.org:8011/builders/llvm-arm-linux/builds/1427/steps/test-llvm/logs/2010-01-07-uamemfeature.ll
and http://google1.osuosl.org:8011/builders/llvm-ppc-linux/builds/3820/steps/test-llvm/logs/2010-01-07-uamemfeature.ll

Should an unsupported -mattr flag really cause llc to crash though?

On Mon, Jan 11, 2010 at 1:58 PM, Chris Lattner <sabre at nondot.org> wrote:
> Author: lattner
> Date: Mon Jan 11 15:58:19 2010
> New Revision: 93189
>
> URL: http://llvm.org/viewvc/llvm-project?rev=93189&view=rev
> Log:
> reduce this to a sensible testcase.
>
> Modified:
>    llvm/trunk/test/CodeGen/X86/2010-01-07-UAMemFeature.ll
>
> Modified: llvm/trunk/test/CodeGen/X86/2010-01-07-UAMemFeature.ll
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/X86/2010-01-07-UAMemFeature.ll?rev=93189&r1=93188&r2=93189&view=diff
>
> ==============================================================================
> --- llvm/trunk/test/CodeGen/X86/2010-01-07-UAMemFeature.ll (original)
> +++ llvm/trunk/test/CodeGen/X86/2010-01-07-UAMemFeature.ll Mon Jan 11 15:58:19 2010
> @@ -1,24 +1,11 @@
>  ; RUN: llc -mattr=vector-unaligned-mem < %s | FileCheck %s
> -; CHECK: addps{{[ \t]+}}(
> +; CHECK: addps (
>
>  target datalayout = "e-p:64:64:64-i1:8:8-i8:8:8-i16:16:16-i32:32:32-i64:64:64-f32:32:32-f64:64:64-v64:64:64-v128:128:128-a0:0:64-s0:64:64-f80:128:128"
>  target triple = "x86_64-unknown-linux-gnu"
>
> -define i32 @foo(i32 %n1, float* %A2, float* %B3, float* %C4) {
> -"file loop.c, line 1, bb1":    ; srcLine 1
> -       %"$LCS_4" = alloca i64, align 8         ; <i64*> [#uses=5]      ; [oox.86 : sln.1]
> -       %"$LCS_5" = alloca i64, align 8         ; <i64*> [#uses=5]      ; [oox.87 : sln.1]
> -       %"$LCS_6" = alloca i64, align 8         ; <i64*> [#uses=5]      ; [oox.88 : sln.1]
> -
> -       %r128 = load i64* %"$LCS_4", align 8            ; <i64> [#uses=1]       ; [oox.192 : sln.6]
> -       %r129 = inttoptr i64 %r128 to <4 x float>*              ; <<4 x float>*> [#uses=1]      ; [oox.192 : sln.6]
> -       %r130 = load <4 x float>* %r129, align 4                ; <<4 x float>> [#uses=1]       ; [oox.192 : sln.6]
> -       %r131 = load i64* %"$LCS_5", align 8            ; <i64> [#uses=1]       ; [oox.192 : sln.6]
> -       %r132 = inttoptr i64 %r131 to <4 x float>*              ; <<4 x float>*> [#uses=1]      ; [oox.192 : sln.6]
> -       %r133 = load <4 x float>* %r132, align 4                ; <<4 x float>> [#uses=1]       ; [oox.192 : sln.6]
> -       %r134 = add <4 x float> %r130, %r133            ; <<4 x float>> [#uses=1]       ; [oox.192 : sln.6]
> -       %r135 = load i64* %"$LCS_6", align 8            ; <i64> [#uses=1]       ; [oox.192 : sln.6]
> -       %r136 = inttoptr i64 %r135 to <4 x float>*              ; <<4 x float>*> [#uses=1]      ; [oox.192 : sln.6]
> -       store <4 x float> %r134, <4 x float>* %r136, align 4    ; [oox.192 : sln.6]
> -       ret i32 0       ; [oox.189 : sln.10]
> +define <4 x float> @foo(<4 x float>* %P, <4 x float> %In) nounwind {
> +       %A = load <4 x float>* %P, align 4
> +       %B = add <4 x float> %A, %In
> +       ret <4 x float> %B
>  }
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>