[PATCH] D99951: [NFC] Add scalable vectorisation tests for int/FP <> int/FP conversions

[David Sherwood via Phabricator via llvm-commits]

david-arm added a comment.

Sure, I can combine them into a single test if you think that's better? I wasn't sure whether in general we prefer to have more concise test files or to have fewer, larger files. I could create a file with a name like sve-type-conv.ll.



================
Comment at: llvm/test/Transforms/LoopVectorize/AArch64/sve-conv-int-to-int.ll:6
+; If this check fails please read test/CodeGen/AArch64/README for instructions on how to resolve it.
+; WARN-NOT: warning
+
----------------
fhahn wrote:
> Do we still need this? IIRC most those checks have been removed by now?
Yes you're right. When I created the patch originally I think we still needed them, but I'll rebase and remove them.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99951