[PATCH] D91120: [DAGCombine][PowerPC] Convert negated abs to trivial arithmetic ops

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 17 06:28:52 PST 2020


spatel added reviewers: jonpa, uweigand.
spatel added a comment.

It seems strange that we would need to add a TLI hook to avoid regressing a target that has abs/nabs support in the ISA.
If I'm seeing it correctly, SystemZ relies on ISD::ABS being expanded and then pattern-matching the expansion back to `lpgr` for example. Could we change that target to declare ISD::ABS as legal and adjust the tablegen matching for it?


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