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

Matthias Gehre via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 15 00:55:44 PDT 2022


mgehre-amd added a comment.

In D126644#3645177 <https://reviews.llvm.org/D126644#3645177>, @nikic wrote:

> I'm a bit confused, where in the code does this actually create functions? It doesn't look like expandDivision() does that, and I can't spot anything else creating `__llvm_udivXXX` either.

Yes, I changed that approach to directly generate a loop in place of the original instruction as it was requested by one of the initial review comments here. I now also updated the PR description to reflect that.


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