<html>
    <head>
      <base href="https://llvm.org/bugs/" />
    </head>
    <body><table border="1" cellspacing="0" cellpadding="8">
        <tr>
          <th>Bug ID</th>
          <td><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW --- - Invalid optimization on ARM + Neon"
   href="https://llvm.org/bugs/show_bug.cgi?id=24669">24669</a>
          </td>
        </tr>

        <tr>
          <th>Summary</th>
          <td>Invalid optimization on ARM + Neon
          </td>
        </tr>

        <tr>
          <th>Product</th>
          <td>libraries
          </td>
        </tr>

        <tr>
          <th>Version</th>
          <td>3.6
          </td>
        </tr>

        <tr>
          <th>Hardware</th>
          <td>PC
          </td>
        </tr>

        <tr>
          <th>OS</th>
          <td>Linux
          </td>
        </tr>

        <tr>
          <th>Status</th>
          <td>NEW
          </td>
        </tr>

        <tr>
          <th>Severity</th>
          <td>normal
          </td>
        </tr>

        <tr>
          <th>Priority</th>
          <td>P
          </td>
        </tr>

        <tr>
          <th>Component</th>
          <td>Backend: ARM
          </td>
        </tr>

        <tr>
          <th>Assignee</th>
          <td>unassignedbugs@nondot.org
          </td>
        </tr>

        <tr>
          <th>Reporter</th>
          <td>pitrou@free.fr
          </td>
        </tr>

        <tr>
          <th>CC</th>
          <td>llvm-bugs@lists.llvm.org
          </td>
        </tr>

        <tr>
          <th>Classification</th>
          <td>Unclassified
          </td>
        </tr></table>
      <p>
        <div>
        <pre>This is with 3.6.2.  If you take the following IR:

define i32 @get_set_complex({ float, float }* noalias nocapture %retptr,
                            { i8*, i32 }** noalias nocapture readnone %excinfo,
                            i8* noalias nocapture readnone %env,
                            [38 x i8]* nocapture %arg.rec,
                            float %arg.val.0, float %arg.val.1)
{
entry:
  %inserted.real = insertvalue { float, float } undef, float %arg.val.0, 0
  %inserted.imag = insertvalue { float, float } %inserted.real, float
%arg.val.1, 1
  %.15 = getelementptr inbounds [38 x i8]* %arg.rec, i32 0, i32 10
  %.16 = bitcast i8* %.15 to { float, float }*
  %.17 = bitcast i8* %.15 to float*
  %.18 = load float* %.17, align 1
  %.19 = getelementptr inbounds [38 x i8]* %arg.rec, i32 0, i32 14
  %0 = bitcast i8* %.19 to float*
  %.20 = load float* %0, align 1
  %inserted.real.1 = insertvalue { float, float } undef, float %.18, 0
  %inserted.imag.1 = insertvalue { float, float } %inserted.real.1, float %.20,
1
  store { float, float } %inserted.imag, { float, float }* %.16, align 1
  store { float, float } %inserted.imag.1, { float, float }* %retptr, align 4
  ret i32 0
}


and feed it into llc as follows:

$ llc -O3 -code-model=default -relocation-model=default
-mtriple=armv7l-unknown-linux-gnueabihf -mcpu=generic


the assembler output is buggy, as it does the loads after the stores:

@ BB#0:                                 @ %entry
    vmov    r1, s1
    str    r1, [r3, #14]
    vmov    r1, s0
    str    r1, [r3, #10]
    ldr    r1, [r3, #10]
    ldr    r2, [r3, #14]
    stm    r0, {r1, r2}
    mov    r0, #0
    bx    lr

If you replace "-O3" with "-O0", you get correct assembler output (loads are
issued before stores):

@ BB#0:                                 @ %entry
    .save    {r11, lr}
    push    {r11, lr}
    movw    r1, #0
    ldr    r2, [r3, #10]
    ldr    r12, [r3, #14]
    vmov    lr, s1
    str    lr, [r3, #14]
    vmov    lr, s0
    str    lr, [r3, #10]
    str    r12, [r0, #4]
    str    r2, [r0]
    mov    r0, r1
    pop    {r11, pc}</pre>
        </div>
      </p>
      <hr>
      <span>You are receiving this mail because:</span>
      
      <ul>
          <li>You are on the CC list for the bug.</li>
      </ul>
    </body>
</html>