[PATCH] D130517: [GlobalISel] Add sdiv exact (X, constant) -> mul combine.

[Amara Emerson via Phabricator via llvm-commits]

aemerson added inline comments.


================
Comment at: llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp:4956-4959
+  // Don't do this for minsize because the instruction sequence is usually
+  // larger.
+  if (MF.getFunction().hasMinSize())
+    return false;
----------------
arsenm wrote:
> aemerson wrote:
> > arsenm wrote:
> > > aemerson wrote:
> > > > arsenm wrote:
> > > > > This greatly depends on the target handling of division and probably should be attached to the tablegen addition
> > > > Not sure what you mean by "tablegen addition"?
> > > I mean GICombineRules should have predicate fields, analogous to PatFags
> > Is there a patch to do that already?
> No, not that I'm aware of
Implementing the infrastructure for that seems a separate issue though? This kind of minsize check is mirroring the same check done for SelectionDAG.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130517