[PATCH] D88201: [DAGCombiner] Add decomposition patterns for Mul-by-Imm.

Mon Oct 5 06:40:16 PDT 2020

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

LGTM

================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:3602-3604
   if (isConstantOrConstantVector(N1, /*NoOpaques*/ true) &&
       DAG.isKnownToBeAPowerOfTwo(N1) &&
       (!VT.isVector() || Level <= AfterLegalizeVectorOps)) {
----------------
Independent of this patch, but if you are looking at making further changes in here...that looks strange/unnecessary. The other transforms around here are using `N1IsConst`; why is this one different?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88201