[PATCH] D86925: [MachineSink] add one more profitable pattern for sinking

[Quentin Colombet via Phabricator via llvm-commits]

qcolombet added a comment.

Hi @shchenz ,

> 1: add one more mir test case

Did you forget to do `git add`, I don't see it.

Cheers,
-Quentin



================
Comment at: llvm/lib/CodeGen/MachineSink.cpp:622
+    const TargetRegisterClass *RC = MRI->getRegClass(Reg);
+    RegClassWeight W = TRI->getRegClassWeight(RC);
+    // Users for the defs are all dominated by SuccToSinkTo.
----------------
This variable is a left over of the previous version of the patch.
Please delete it.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D86925/new/

https://reviews.llvm.org/D86925