[PATCH] D20627: Do not rename registers that do not start an independent live range

Krzysztof Parzyszek via llvm-commits llvm-commits at lists.llvm.org
Wed May 25 10:05:08 PDT 2016


kparzysz added inline comments.

================
Comment at: lib/CodeGen/AggressiveAntiDepBreaker.cpp:918
@@ +917,3 @@
+            if (R == AntiDepReg || TRI->isSubRegister(AntiDepReg, R))
+              continue;
+            AntiDepReg = 0;
----------------
An output dependency could still defeat this test (i.e. we'd need to make sure that the later def in the output dependency starts a new live range), but I'll wait for more comments before making this change.


Repository:
  rL LLVM

http://reviews.llvm.org/D20627





More information about the llvm-commits mailing list