[PATCH] D91120: [DAGCombine][PowerPC] Fold negated abs to avoid generating `select_cc`

Kai Luo via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 12 23:32:07 PST 2020


lkail added a comment.

In D91120#2391105 <https://reviews.llvm.org/D91120#2391105>, @spatel wrote:

> Note that we are planning to canonicalize IR to the abs() intrinsic - D87188 <https://reviews.llvm.org/D87188>.
> Assuming that comes through relatively soon (cc @nikic), should we hold off adding what would probably become dead code in the backend?

Thanks for the info. Let me apply D87188 <https://reviews.llvm.org/D87188> and see what will be going on. After D87188 <https://reviews.llvm.org/D87188> is landed, I'll update the patch accordingly if necessary.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91120



More information about the llvm-commits mailing list