[PATCH] D142998: [SVE][codegen] Add test case for a fused multiply-add (NFC)

David Sherwood via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 1 03:25:48 PST 2023


david-arm added a comment.

Thanks for putting these tests in a precommit patch @sushgokh!



================
Comment at: llvm/test/CodeGen/AArch64/sve-multiply-add-accumulate.ll:1
+; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py
+; RUN: llc -mtriple=aarch64-unknown-linux-gnu -mattr=+sve < %s | FileCheck %s
----------------
Perhaps it's better to just add these tests to the existing sve-int-mad-pred.ll file?


================
Comment at: llvm/test/CodeGen/AArch64/sve-multiply-add-accumulate.ll:165
+; negative integer splat as one of the addend
+define <vscale x 8 x i16> @muladd_generic_test4(<vscale x 8 x i16> %a, <vscale x 8 x i16> %b)
+; CHECK-LABEL: muladd_generic_test4:
----------------
I'm not sure what this is doing that is different to `@muladd_i8_negativeAddend`? Are you just trying to test a case where the immediate is too big to fit into the 'add'?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142998



More information about the llvm-commits mailing list