This is an archive of the discontinued LLVM Phabricator instance.

Fix iOSDarwin()'s doc comment.
ClosedPublic

Authored by varungandhi-apple on Nov 17 2019, 1:09 PM.

Details

Summary

I saw the doc comment using the Option + Click in Xcode and I was
confused by the fact that tvOS wasn't listed. The method definition
also doesn't make it clear that the check for tvOS is taking place as
the tvOS check is hidden in the isiOS() check.

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptNov 17 2019, 1:09 PM

Might as well change "OS X" to "macOS" while you're at it.

varungandhi-apple retitled this revision from Fix iOSDarwin()'s doc comment to also include tvOS. to Fix iOSDarwin()'s doc comment..

Addressed John's suggestion to replace OS X -> macOS.

This revision is now accepted and ready to land.Nov 18 2019, 12:19 PM
This revision was automatically updated to reflect the committed changes.