[PATCH] D80712: [SVE] Add checks for no warnings in SVE tests

[David Sherwood via Phabricator via cfe-commits]

david-arm added a comment.

Hi @efriedma, at least amongst all the tests in llvm/test/CodeGen/AArch64/sve-* there are still 66 with warnings. @sdesmalen and I discussed this and our reason for adding checks for warnings is mainly to do with the fact we are still fixing up cases and implementing SVE codgen support, so there is a chance that somewhere we'll introduce a regression without realising it. That's the main rationale for this, but I do realise that ultimately getVectorNumElements() will be deprecated. I also realised that the warnings I've added to the clang tests aren't that useful unless we're compiling them to real assembly so I may add some new RUN lines for those.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80712