[PATCH] D100937: [ARM][Driver][Windows] Allow command-line upgrade to Armv8.

Simon Tatham via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 21 03:10:02 PDT 2021


simon_tatham added a comment.

Yes, it looks easy enough to add something in `llvm/unittests/ADT/TripleTest.cpp` to directly test `getARMCPUForArch`.

I'd mildly prefer to do that //as well// as having the test here, because the call site in the clang driver is quite complicated. My real aim is that asking for `crypto` (or any other Armv8-specific extension) should actually get it; I think any accidental regression of that behavior in future is at least as likely to arise from changes in `clang/lib/Driver` as `llvm/lib/Support`, so I'd like there to be a test that will detect the regression if it occurs for //any// reason.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100937



More information about the llvm-commits mailing list