[PATCH] D44958: [PostRASink]Add register dependency check for implicit operands

Jun Bum Lim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 9 10:13:31 PDT 2018


junbuml added inline comments.


================
Comment at: lib/CodeGen/MachineSink.cpp:1091
+      // other targets.
+    } else if (MO.isUse()) {
+      if (ModifiedRegs[Reg]) {
----------------
I keep isUse() here with FIXME since I'm not perfectly sure if skipping the internal read is safe in all other targets. 


https://reviews.llvm.org/D44958





More information about the llvm-commits mailing list