[PATCH] D142313: [PowerPC] Replace PPCISD::VABSD(X,Y,0) cases with generic ISD::ABDU(X,Y) node

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 25 00:43:34 PST 2023


RKSimon added a subscriber: dmgreen.
RKSimon added a comment.

In D142313#4075615 <https://reviews.llvm.org/D142313#4075615>, @nemanjai wrote:

> Thanks for updating this. I am just curious, do the combines you removed just end up getting combined to a `VSELECT`?

@dmgreen added them to DAGCombiner as part of adding the generic abds/abdu nodes

What should be done with the PPCISD::VABSD nodes? They can only be used for the PPCISD::VABSD(X, 1) case now - should they be simplified/renamed?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142313



More information about the llvm-commits mailing list