[PATCH] D95218: [AArch64] Merge [US]MULL with half adds and subs into [US]ML[AS]L

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 22 08:38:56 PST 2021


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

LGTM



================
Comment at: llvm/test/CodeGen/AArch64/mla_mls_merge.ll:273
+
+; Function Attrs: nofree nosync nounwind readnone willreturn
+declare <8 x i16> @llvm.aarch64.neon.umull.v8i16(<8 x i8>, <8 x i8>)
----------------
avieira wrote:
> dmgreen wrote:
> > You can remove these too.
> I assume you meant the function attrs and not the intrinsic declarations.
Yep. That's great. Thanks!


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

https://reviews.llvm.org/D95218



More information about the llvm-commits mailing list