[PATCH] D80485: [DAGCombiner][PowerPC][AArch64] Remove isMulhCheaperThanMulShift TLI hook. Use isOperationLegal directly instead.

Amy Kwan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 26 11:26:08 PDT 2020


amyk added a comment.

In D80485#2054050 <https://reviews.llvm.org/D80485#2054050>, @craig.topper wrote:

> Thanks for the clarification. I'll go ahead and extract the AArch64 change here since it should be correct regardless. I'll hold on to this until after D78272 <https://reviews.llvm.org/D78272> is moved to DAG combine and we can revisit.


I've updated D78272 <https://reviews.llvm.org/D78272> (moving the transformation into `DAGCombiner.cpp`). Thanks for getting back to me regarding this.


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

https://reviews.llvm.org/D80485





More information about the llvm-commits mailing list