[PATCH] D12775: Use new TokenFactor chain when merging stores

[hfinkel@anl.gov via llvm-commits]

hfinkel added a comment.

> I haven't managed to find a case where the nodes do end up reordered with an in tree target.


Obviously less than ideal, but you can write a 'REQUIRES: asserts' test using -debug that directly checks the SDAG nodes.


================
Comment at: test/CodeGen/X86/merge-store-partially-alias-loads.ll:10
@@ +9,3 @@
+; CHECK-NEXT: movw %ax, 1(%rdi)
+; CHECK-NEXT: movb	%cl, 3(%rdi)
+; CHECK-NEXT: retq
----------------
Seems like these should be CHECK-DAG (with some regexes for the registers).


http://reviews.llvm.org/D12775