[PATCH] D34472: [DAG] Rewrite areNonVolatileConsecutiveLoads to use BaseIndexOffset
Vadzim Dambrouski via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 28 01:52:59 PDT 2017
pftbest added inline comments.
================
Comment at: test/CodeGen/MSP430/Inst16mm.ll:68
+; CHECK-DAG: mov.w 2(r1), 6(r1)
+; CHECK-DAG: mov.w 0(r1), 4(r1)
}
----------------
RKSimon wrote:
> If these are DAG checks would it better to replace the hard coded registers with regex?
I think this test checks that we do direct memory to memory transfer without loading into intermediate register. r1 is a stack pointer, so it's fixed and cannot change.
What can change is the order of this two movs. Is there some CHECK keyword that ignores the order of instructions?
https://reviews.llvm.org/D34472
More information about the llvm-commits
mailing list