[PATCH] D107582: [CodeGen] Fix dependence breaking for tied operands

[Danila Malyutin via Phabricator via llvm-commits]

danilaml marked an inline comment as done.
danilaml added inline comments.


================
Comment at: llvm/lib/CodeGen/CriticalAntiDepBreaker.cpp:224
+
+  for (unsigned i = 0, e = MI.getNumOperands(); i != e; ++i) {
+    MachineOperand &MO = MI.getOperand(i);
----------------
MatzeB wrote:
> MatzeB wrote:
> > 
> Would it make sense to compute a `bool HasTiedOperand` variable in the first loop and skip the 2nd one if we didn't see any tied operands?
Maybe. Not sure if we want to attempt to optimize it if it's supposed to be deprecated though.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107582