This is an archive of the discontinued LLVM Phabricator instance.

[libc++][NFC] Remove mentions of warn_unused_result
ClosedPublic

Authored by philnik on Aug 21 2022, 6:05 PM.

Details

Summary

We don't use clang::warn_unused_result anymore, so let's remove the mentions of it from the tests

Diff Detail

Event Timeline

philnik created this revision.Aug 21 2022, 6:05 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 21 2022, 6:05 PM
philnik requested review of this revision.Aug 21 2022, 6:05 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 21 2022, 6:05 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript
Mordante accepted this revision.Aug 25 2022, 9:36 AM

Thanks for the cleanup! LGTM!

The AIX CI runner was cancelled, can you either rebase to restart the CI or keep an eye on the CI after landing this.

This revision is now accepted and ready to land.Aug 25 2022, 9:36 AM
This revision was landed with ongoing or failed builds.Aug 25 2022, 1:28 PM
This revision was automatically updated to reflect the committed changes.