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

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 13 17:03:52 PDT 2022


efriedma added a comment.

My primary concern here is that we're unconditionally iterating over every instruction in the module, even if the pass has nothing to do.  In the grand scheme of things, it isn't *that* expensive, but it's still not cheap, and we seem to have grown a number of passes doing similar walks.  Can we merge this with some other walk over the module?


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