[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#2860185 <https://reviews.llvm.org/D103799#2860185>, @RKSimon wrote:

> This is actually on my backlog - now that we've improved scalarization costs on x86 I'm hoping to add more realistic scalar div/rem costs instead of relying on this kludge. So I'm happy with (2) for now.

Thanks for confirming!

Would you or @dmgreen be happy to accept the patch in that case?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103799