[PATCH] D35379: Add documentation for @available

Nico Weber via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 14 09:27:07 PDT 2017


thakis added a comment.

Mostly done, thanks!



================
Comment at: docs/LanguageExtensions.rst:1274
 
+Objective-C @available
+----------------------
----------------
sdy wrote:
> sdy wrote:
> > I think "Objective-C" is redundant, this is already in the ObjC section and most of the other headers don't start with "Objective-C".
> `@available` should probably be wrapped in backticks.
The one below and the two above start with "Objective-C".


================
Comment at: docs/LanguageExtensions.rst:1278
+It is possible use the newest SDK but still build a program that can run on
+older macOS and iOS versions, by passing ``-mmacosx-version-info=`` /
+``--miphoneos-version-min=``.
----------------
sdy wrote:
> arphaman wrote:
> > Nit: older versions of macOS or iOS
> I thought this flag was `--mmacosx-version-min`?
> Nit: I would remove the comma after "versions".
Nicos-MacBook-Pro:llvm-build thakis$ bin/clang -mmacosx-version-min=10.11 -c test.mm
Nicos-MacBook-Pro:llvm-build thakis$ bin/clang --mmacosx-version-min=10.11 -c test.mm
clang-3.5: error: unsupported option '--mmacosx-version-min=10.11'

Removed comma.


https://reviews.llvm.org/D35379





More information about the cfe-commits mailing list