On Dec 17, 2012, at 11:14 AM, David Chisnall <[email protected]> wrote:
> On 17 Dec 2012, at 19:11, Rafael EspĂ­ndola wrote:
>>> static bool hasAtomicCopyHelperAPI(const ObjCRuntime &runtime) {
>>> -  // For now, only NeXT has these APIs.
>>> -  return runtime.isNeXTFamily();
>>> +  return runtime.hasAtomicCopyHelper();
>>> }
>> 
>> This function should probably just be inlined to all users, no?
> 
> Probably, but I'm not really sure why it existed in the first place.  

Most likely, it was abstracted out locally without considering that it should
just have been added to ObjCRuntime.

John.
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to