diff options
| author | Paul Robinson <paul.robinson@sony.com> | 2023-01-25 13:58:34 -0800 |
|---|---|---|
| committer | Paul Robinson <paul.robinson@sony.com> | 2023-01-25 13:59:01 -0800 |
| commit | 7fc871591f1399cd88ff301ed84fa67dc3bf7a6b (patch) | |
| tree | 14aca37544130116199ba97d385ccd48ad726497 /llvm/lib/ProfileData/Coverage/CoverageMappingReader.cpp | |
| parent | 656b6d0b2371f4e67c075f43ed1f23bb25b5771f (diff) | |
| download | llvm-7fc871591f1399cd88ff301ed84fa67dc3bf7a6b.zip llvm-7fc871591f1399cd88ff301ed84fa67dc3bf7a6b.tar.gz llvm-7fc871591f1399cd88ff301ed84fa67dc3bf7a6b.tar.bz2 | |
[unittests] Use GTEST_SKIP() instead of return when appropriate
Basically NFC: A TEST/TEST_F/etc that bails out early (usually because
setup failed or some other runtime condition wasn't met) generally
should use GTEST_SKIP() to report its status correctly, unless it
takes steps to report another status (e.g., FAIL()).
I did see a handful of tests show up as SKIPPED after this change,
which is not unexpected. The status seemed appropriate in all the new
cases.
Diffstat (limited to 'llvm/lib/ProfileData/Coverage/CoverageMappingReader.cpp')
0 files changed, 0 insertions, 0 deletions