[PATCH] D137925: [AArch64] FNMUL

Madhur Amilkanthwar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 14 01:22:27 PST 2022


madhur13490 added a comment.

Can we please more verbose title for the patch?



================
Comment at: llvm/test/CodeGen/AArch64/fnmul.ll:53
+entry:
+  %fneg = fneg fast double %x
+  %mul = fmul fast double %fneg, %y
----------------
The test cases cover -x * y test cases. Can we have x * -y too? I believe this patch will handle those cases too. However, it is good to have a test case.


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

https://reviews.llvm.org/D137925



More information about the llvm-commits mailing list