[PATCH] D57925: [TTI] Add generic cost model for fixed point smul/umul

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 12 13:20:09 PST 2019


nikic added a comment.

> An alternative would be to assume the mull/mulh cost is the same as 2 * mul - what do people think?

Would it be possible to add a separate method for getting the widening multiplication cost, defaulting to your current implementation? It would come in handy for MULO cost modeling as well. And probably more likely to receive a good target-specific implementation than the full smul.fix intrinsics.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D57925





More information about the llvm-commits mailing list