[PATCH] D66300: [TargetLowering] x s% C == 0 fold: vector divisor with INT_MIN handling

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 19 07:54:39 PDT 2019


RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.

LGTM with more minor I noticed



================
Comment at: llvm/lib/CodeGen/SelectionDAG/TargetLowering.cpp:5231
+
+  // If we didn't have channels with INT_MIN divisor, then we're done.
+  if (!HadIntMinDivisor)
----------------
(nit) channels -> lanes ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66300





More information about the llvm-commits mailing list