[PATCH] D80485: [DAGCombiner][PowerPC] Remove isMulhCheaperThanMulShift TLI hook. Use isOperationLegalOrCustom directly instead.
Amy Kwan via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 15 08:56:21 PDT 2020
amyk added a comment.
Thanks @dmgreen and @craig.topper for reviewing. :-)
I will fix the indentation in DAGCombiner on the commit, if that's okay.
================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:8212
+ if (!TLI.isOperationLegalOrCustom(MulhOpcode, NarrowVT))
+ return SDValue();
+
----------------
dmgreen wrote:
> This formatting is a touch off.
Oops, you're right. I can fix that on the commit if it's okay.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D80485/new/
https://reviews.llvm.org/D80485
More information about the llvm-commits
mailing list