r294855 - docs: update docs for objc_storeStrong behaviour

[Saleem Abdulrasool via cfe-commits]

Hi Hans,

Would you mind grabbing this for the 4.0 release as well?  It's merely
correcting the documentation, so should have no impact on the toolchain
itself.

On Sat, Feb 11, 2017 at 9:24 AM, Saleem Abdulrasool via cfe-commits <
cfe-commits at lists.llvm.org> wrote:

> Author: compnerd
> Date: Sat Feb 11 11:24:09 2017
> New Revision: 294855
>
> URL: http://llvm.org/viewvc/llvm-project?rev=294855&view=rev
> Log:
> docs: update docs for objc_storeStrong behaviour
>
> objc_storeStrong does not return a value.
>
> Modified:
>     cfe/trunk/docs/AutomaticReferenceCounting.rst
>     cfe/trunk/lib/CodeGen/CodeGenModule.h
>
> Modified: cfe/trunk/docs/AutomaticReferenceCounting.rst
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/docs/
> AutomaticReferenceCounting.rst?rev=294855&r1=294854&r2=294855&view=diff
> ============================================================
> ==================
> --- cfe/trunk/docs/AutomaticReferenceCounting.rst (original)
> +++ cfe/trunk/docs/AutomaticReferenceCounting.rst Sat Feb 11 11:24:09 2017
> @@ -2258,16 +2258,13 @@ non-block type [*]_.  Equivalent to the
>
>  .. code-block:: objc
>
> -  id objc_storeStrong(id *object, id value) {
> -    value = [value retain];
> +  void objc_storeStrong(id *object, id value) {
>      id oldValue = *object;
> +    value = [value retain];
>      *object = value;
>      [oldValue release];
> -    return value;
>    }
>
> -Always returns ``value``.
> -
>  .. [*] This does not imply that a ``__strong`` object of block type is an
>     invalid argument to this function. Rather it implies that an
> ``objc_retain``
>     and not an ``objc_retainBlock`` operation will be emitted if the
> argument is
>
> Modified: cfe/trunk/lib/CodeGen/CodeGenModule.h
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/
> CodeGenModule.h?rev=294855&r1=294854&r2=294855&view=diff
> ============================================================
> ==================
> --- cfe/trunk/lib/CodeGen/CodeGenModule.h (original)
> +++ cfe/trunk/lib/CodeGen/CodeGenModule.h Sat Feb 11 11:24:09 2017
> @@ -166,7 +166,7 @@ struct ObjCEntrypoints {
>    /// void objc_release(id);
>    llvm::Constant *objc_release;
>
> -  /// id objc_storeStrong(id*, id);
> +  /// void objc_storeStrong(id*, id);
>    llvm::Constant *objc_storeStrong;
>
>    /// id objc_storeWeak(id*, id);
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>



-- 
Saleem Abdulrasool
compnerd (at) compnerd (dot) org
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170211/379e838b/attachment.html>