[PATCH] D127118: [AArch64] Autogenerate sve-fixed-length tests. NFC

Paul Walker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 9 04:40:08 PDT 2022


paulwalker-arm added a comment.

To be honest these tests are just not a good fit for auto-generation, which is why we didn't use `update_llc_test_checks.py` originally.  The best I could come up with is to disable the `NO_SVE` run line when running the script and then enable it afterwards.  I do have the workings of a patch to add a `--ignored-check-prefixes` option to `update_llc_test_checks.py` but even then the decision as to which CHECK lines are emitted doesn't really match what we wanted to achieve.  If autogeneration is preferable and you're happy to wait a week or two I think it'll be better if I just restructure all the SVE fixed length tests (essentially split them based on register size) and enable auto-generation at the same time.  I think this is worth doing anyway to increase our test coverage and test parallelism as the existing layout means these tests are the most time consuming ones with the AArch64 directory.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127118



More information about the llvm-commits mailing list