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

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 9 06:16:00 PDT 2021


fhahn added a comment.

In D99951#2679444 <https://reviews.llvm.org/D99951#2679444>, @david-arm wrote:

> 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.

Sounds good to me. I don't think there's a general answer really. In this case it seems like they all test the same/similar code path and there might be less processing overhead with fewer, larger files than with many small files.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99951



More information about the llvm-commits mailing list