[PATCH] D82669: [ARM] Generate URHADD from (b - (~a)) >> 1

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 3 03:45:01 PDT 2020


dmgreen accepted this revision.
dmgreen added a comment.
This revision is now accepted and ready to land.

Thanks. There are always other ways to do this - in tablegen but the pattern would be giant, earlier as a dagcombine but it needn't be done like that in this case.  (We probably would have to do that for MVE, as the trunc would not remain legal). In the long run if something like this is done for MVE too, we may be able to share the code between the two places. But in the meantime this looks like a nice optimization.

LGTM.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82669





More information about the llvm-commits mailing list