[PATCH] D14489: [AArch64] Applying load pair optimization for volatile load/store

[Jun Bum Lim via llvm-commits]

junbuml added a comment.

Thanks Tim for the clarification.  As long as we are not accessing volatile first in ldp, reordering volatile with non-volatile is fine. right ? So, I may want to confirm what I understand using below two more cases.

5.This must be fine ?

  ldr x9, [x0] 
  ldr x10, [x1] (volatile) 
  ldr x8, [x0, #8] 
    ; into
  ldr x10, [x1] (volatile) 
  ldp x9, x8, [x0]

6.This must be not safe?

  ldr x9, [x0]   (volatile) 
  ldr x8, [x0, #8] 
    ; into
  ldp x9, x8, [x0]


http://reviews.llvm.org/D14489