[PATCH] D141017: [MachineCombiner] Use default latency model when no detailed model available

[Anton Sidorenko via Phabricator via llvm-commits]

asi-sc added a comment.

> Previously, we unconditionally applied the first legal pattern - without any consideration of profitability

Thanks for addressing this issue for default scheduling models! I also faced it some time ago, but I didn't have time to take a closer look.

LGTM (I don't check all x86 tests)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141017