[PATCH] D142878: Add testing for Fuchsia multilib
Michael Platings via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Feb 6 03:30:13 PST 2023
michaelplatings abandoned this revision.
michaelplatings added a comment.
> A more scalable approach would be to check only the minimal set of combination necessary to achieve full coverage.
In that case the testing you've already got in place in `clang/test/Driver/fuchsia.cpp` is adequate. There was a potential bug here so I was being extra cautious, but in practice the test added in this change didn't pick up anything that wasn't also caught by the existing tests. I'll abandon this change and remove it from the stack. The change stack now begins at D142893 <https://reviews.llvm.org/D142893>.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D142878/new/
https://reviews.llvm.org/D142878
More information about the cfe-commits
mailing list