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

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.
Herald added a subscriber: mingmingl.


================
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:
> > 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


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130517