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
