[PATCH] D106494: WIP: [AArch64][GlobalISel] Legalize ctpop s128

Jon Roelofs via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 22 18:33:58 PDT 2021


jroelofs added a comment.

In D106494#2898756 <https://reviews.llvm.org/D106494#2898756>, @paquette wrote:

> Do you have any idea of how to improve the zext behaviour?

I was thinking about making `narrowScalar` do: `zext(add(trunc(ctpop(hi)), trunc(ctpop(lo)))` with the hope that some combine folds the inner `trunc(zext(ctpop(x))) => ctpop(x)`, but I haven't tried this yet.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106494



More information about the llvm-commits mailing list