<div dir="ltr">Your load instructions have no alignment attribute on them,  this isn't the same as saying that they are unaligned.  Instead, it means that LLVM should use the preferred ABI alignment for that type.  InstCombine is following the rules by sticking an explicit alignment attribute on your instructions.</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jun 18, 2015 at 7:21 AM, Frank Winter <span dir="ltr"><<a href="mailto:fwinter@jlab.org" target="_blank">fwinter@jlab.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Is it correct behavior that 'instcombine' not only combines instructions but also adds alignment information, and why. (In my case I wished it wouldn't do that). Attached the input and output module.<br>
<br>
<br>
Thanks,<br>
Frank<br>
<br>
<br>
; ModuleID = '<stdin>'<br>
<br>
target datalayout = "e-m:e-i64:64-f80:128-n8:16:32:64-S128"<br>
<br>
target triple = "x86_64-unknown-linux-gnu"<br>
<br>
define void @main(float* noalias %arg0, float* noalias %arg1, float* noalias %arg2) {<br>
<br>
entrypoint:<br>
<br>
  %0 = bitcast float* %arg1 to <2 x float>*<br>
<br>
  %1 = bitcast float* %arg2 to <2 x float>*<br>
<br>
  %2 = bitcast <2 x float>* %0 to <4 x float>*<br>
<br>
  %3 = bitcast <2 x float>* %1 to <4 x float>*<br>
<br>
  %4 = bitcast float* %arg0 to <2 x float>*<br>
<br>
  %5 = load <4 x float>* %2<br>
<br>
  %6 = load <4 x float>* %3<br>
<br>
  %7 = fadd <4 x float> %6, %5<br>
<br>
  %8 = bitcast <2 x float>* %4 to <4 x float>*<br>
<br>
  store <4 x float> %7, <4 x float>* %8<br>
<br>
  %9 = getelementptr float* %arg1, i32 4<br>
<br>
  %10 = bitcast float* %9 to <2 x float>*<br>
<br>
  %11 = getelementptr float* %arg2, i32 4<br>
<br>
  %12 = bitcast float* %11 to <2 x float>*<br>
<br>
  %13 = getelementptr float* %arg0, i32 4<br>
<br>
  %14 = bitcast <2 x float>* %10 to <4 x float>*<br>
<br>
  %15 = bitcast <2 x float>* %12 to <4 x float>*<br>
<br>
  %16 = bitcast float* %13 to <2 x float>*<br>
<br>
  %17 = load <4 x float>* %14<br>
<br>
  %18 = load <4 x float>* %15<br>
<br>
  %19 = fadd <4 x float> %18, %17<br>
<br>
  %20 = bitcast <2 x float>* %16 to <4 x float>*<br>
<br>
  store <4 x float> %19, <4 x float>* %20<br>
<br>
  %21 = getelementptr float* %arg1, i32 8<br>
<br>
  %22 = bitcast float* %21 to <2 x float>*<br>
<br>
  %23 = getelementptr float* %arg2, i32 8<br>
<br>
  %24 = bitcast float* %23 to <2 x float>*<br>
<br>
  %25 = getelementptr float* %arg0, i32 8<br>
<br>
  %26 = bitcast <2 x float>* %22 to <4 x float>*<br>
<br>
  %27 = bitcast <2 x float>* %24 to <4 x float>*<br>
<br>
  %28 = bitcast float* %25 to <2 x float>*<br>
<br>
  %29 = load <4 x float>* %26<br>
<br>
  %30 = load <4 x float>* %27<br>
<br>
  %31 = fadd <4 x float> %30, %29<br>
<br>
  %32 = bitcast <2 x float>* %28 to <4 x float>*<br>
<br>
  store <4 x float> %31, <4 x float>* %32<br>
<br>
  %33 = getelementptr float* %arg1, i32 12<br>
<br>
  %34 = bitcast float* %33 to <2 x float>*<br>
<br>
  %35 = getelementptr float* %arg2, i32 12<br>
<br>
  %36 = bitcast float* %35 to <2 x float>*<br>
<br>
  %37 = getelementptr float* %arg0, i32 12<br>
<br>
  %38 = bitcast <2 x float>* %34 to <4 x float>*<br>
<br>
  %39 = bitcast <2 x float>* %36 to <4 x float>*<br>
<br>
  %40 = bitcast float* %37 to <2 x float>*<br>
<br>
  %41 = load <4 x float>* %38<br>
<br>
  %42 = load <4 x float>* %39<br>
<br>
  %43 = fadd <4 x float> %42, %41<br>
<br>
  %44 = bitcast <2 x float>* %40 to <4 x float>*<br>
<br>
  store <4 x float> %43, <4 x float>* %44<br>
<br>
  ret void<br>
<br>
}<br>
<br>
<br>
Output after llvm-3.6/bin/opt -instcombine -S < vec_add.ll<br>
<br>
<br>
; ModuleID = '<stdin>'<br>
<br>
target datalayout = "e-m:e-i64:64-f80:128-n8:16:32:64-S128"<br>
<br>
target triple = "x86_64-unknown-linux-gnu"<br>
<br>
define void @main(float* noalias %arg0, float* noalias %arg1, float* noalias %arg2) {<br>
<br>
entrypoint:<br>
<br>
  %0 = bitcast float* %arg1 to <4 x float>*<br>
<br>
  %1 = bitcast float* %arg2 to <4 x float>*<br>
<br>
  %2 = load <4 x float>* %0, align 16<br>
<br>
  %3 = load <4 x float>* %1, align 16<br>
<br>
  %4 = fadd <4 x float> %3, %2<br>
<br>
  %5 = bitcast float* %arg0 to <4 x float>*<br>
<br>
  store <4 x float> %4, <4 x float>* %5, align 16<br>
<br>
  %6 = getelementptr float* %arg1, i64 4<br>
<br>
  %7 = getelementptr float* %arg2, i64 4<br>
<br>
  %8 = getelementptr float* %arg0, i64 4<br>
<br>
  %9 = bitcast float* %6 to <4 x float>*<br>
<br>
  %10 = bitcast float* %7 to <4 x float>*<br>
<br>
  %11 = load <4 x float>* %9, align 16<br>
<br>
  %12 = load <4 x float>* %10, align 16<br>
<br>
  %13 = fadd <4 x float> %12, %11<br>
<br>
  %14 = bitcast float* %8 to <4 x float>*<br>
<br>
  store <4 x float> %13, <4 x float>* %14, align 16<br>
<br>
  %15 = getelementptr float* %arg1, i64 8<br>
<br>
  %16 = getelementptr float* %arg2, i64 8<br>
<br>
  %17 = getelementptr float* %arg0, i64 8<br>
<br>
  %18 = bitcast float* %15 to <4 x float>*<br>
<br>
  %19 = bitcast float* %16 to <4 x float>*<br>
<br>
  %20 = load <4 x float>* %18, align 16<br>
<br>
  %21 = load <4 x float>* %19, align 16<br>
<br>
  %22 = fadd <4 x float> %21, %20<br>
<br>
  %23 = bitcast float* %17 to <4 x float>*<br>
<br>
  store <4 x float> %22, <4 x float>* %23, align 16<br>
<br>
  %24 = getelementptr float* %arg1, i64 12<br>
<br>
  %25 = getelementptr float* %arg2, i64 12<br>
<br>
  %26 = getelementptr float* %arg0, i64 12<br>
<br>
  %27 = bitcast float* %24 to <4 x float>*<br>
<br>
  %28 = bitcast float* %25 to <4 x float>*<br>
<br>
  %29 = load <4 x float>* %27, align 16<br>
<br>
  %30 = load <4 x float>* %28, align 16<br>
<br>
  %31 = fadd <4 x float> %30, %29<br>
<br>
  %32 = bitcast float* %26 to <4 x float>*<br>
<br>
  store <4 x float> %31, <4 x float>* %32, align 16<br>
<br>
  ret void<br>
<br>
}<br>
<br>
<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:LLVMdev@cs.uiuc.edu" target="_blank">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" rel="noreferrer" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" rel="noreferrer" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
</blockquote></div><br></div>