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

[Matt Arsenault via Phabricator via llvm-commits]

arsenm 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;
----------------
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


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130517