[PATCH] D22466: Avoid false dependencies of undef machine operands

Michael Kuperstein via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 1 11:00:11 PDT 2016


mkuper accepted this revision.
mkuper added a comment.
This revision is now accepted and ready to land.

LGTM, after you add (or document) the possibly missing test.


================
Comment at: lib/CodeGen/ExecutionDepsFix.cpp:500
@@ +499,3 @@
+    // dependency.
+    MO.setReg(CurrMO.getReg());
+    return;
----------------
Do we have a test-case where we expect this to happen?
If we don't, can you add one? If we do, can you clearly document it as such?


Repository:
  rL LLVM

https://reviews.llvm.org/D22466





More information about the llvm-commits mailing list