r294855 - docs: update docs for objc_storeStrong behaviour

Hans Wennborg via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 14 09:47:36 PST 2017


Sure, r295076.

Thanks,
Hans

On Sat, Feb 11, 2017 at 12:29 PM, Saleem Abdulrasool
<compnerd at compnerd.org> wrote:
> 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


More information about the cfe-commits mailing list