[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