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@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@lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>



-- 
Saleem Abdulrasool
compnerd (at) compnerd (dot) org
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to