[PATCH] D126644: [llvm/CodeGen] Add ExpandLargeDivRem pass

Matthias Gehre via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 14 08:39:43 PDT 2022


mgehre-amd added a comment.

In D126644#3581484 <https://reviews.llvm.org/D126644#3581484>, @arsenm wrote:

> Actually, how is this really different from lib/Transforms/Utils/IntegerDivision.cpp? This just goes to 128 but the existing code goes up to 64. Can you merge them, or at least move the implementation bodies to the same place?

Thanks, I really didn't know about IntegerDivision.cpp! I'll try to merge both implementations. If that works, I can use it here and avoid emitting a new function.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126644



More information about the llvm-commits mailing list