[PATCH] D20456: [CodeGen] Fix problem with X86 byte registers in CriticalAntiDepBreaker

Quentin Colombet via llvm-commits llvm-commits at lists.llvm.org
Wed May 25 10:52:27 PDT 2016

qcolombet added a comment.

Apparently I forgot to push submit!

Comment at: test/CodeGen/X86/pr27681.ll:5
@@ +4,3 @@
+; erroneously choose an unavailable byte register to break a dependence.
+; The test is of course highly dependent on register allocation.
+ at c = global i16 4, align 2
I would recommend to use a MIR test instead.
That way we do not need to be (un)lucky with the register allocation. Moreover the test should be much smaller and easier to understand that way.


More information about the llvm-commits mailing list