[PATCH] D150855: MachineCombiner: consider all uses in getLatency()

[Sergei Barannikov via Phabricator via llvm-commits]

barannikov88 added a comment.

This will still work because virtual registers cannot be implicit operands, and there is a check for virtualness above.
Still, using `defs` and `uses` should be avoided because they have unexpected behavior.
I hope they can be removed someday and then reimplemented correctly w.r.t. implicit operands.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150855