[PATCH] D26921: [AArch64LoadStoreOptimizer] Don't treat write to XZR/WZR as a clobber.

[Geoff Berry via llvm-commits]

gberry marked an inline comment as done.
gberry added inline comments.


================
Comment at: test/CodeGen/AArch64/ldst-opt.ll:1561
+; it isn't actually clobbered.
+define i1 @no-clobber-zr(i64* %p, i64 %x) nounwind {
+; CHECK-LABEL: no-clobber-zr
----------------
junbuml wrote:
> I cannot see difference with/without this patch in this test. It appears that there is no instruction  between the two stores.  
This test is sensitive to changes in instruction scheduling.  I can re-write this as a MIR test that just runs the aarch64-ldst optimization to make it more robust.


https://reviews.llvm.org/D26921