[PATCH] D103799: [CostModel] Express cost(urem) as cost(div+mul+sub) when set to Expand.

[Sander de Smalen via Phabricator via llvm-commits]

sdesmalen added a comment.

In D103799#2818051 <https://reviews.llvm.org/D103799#2818051>, @dmgreen wrote:

> This sounds OK to me, so long as the X86 numbers are not bonkers.

Thanks!

@RKSimon do the X86 cost numbers look sensible to you?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103799