[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