[llvm-commits] [llvm] r157546 - /llvm/trunk/include/llvm/ADT/IntrusiveRefCntPtr.h

[Sebastian Redl]

On 29.05.2012 01:28, Richard Smith wrote:
>
> How about replacing all five operator= overloads (duplicating the work 
> of the constructors) and the 'replace' member function with just:
>
> IntrusiveRefCntPtr& operator=(IntrusiveRefCntPtr O) {
>   swap(O);
>   return *this;
> }
>
That would incur an additional AddRef/Release for the move case. Not 
sure if that is relevant.

Sebastian