[PATCH] D100937: [ARM][Driver][Windows] Allow command-line upgrade to Armv8.
Simon Tatham via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Apr 21 03:10:01 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 cfe-commits
mailing list