Bug 131662 – WTF should have an optional mbmalloc target so we can benchmark FastMalloc

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Geoffrey Garen]

WTF should have an optional mbmalloc target so we can benchmark FastMalloc

[Geoffrey Garen]

Created attachment 229342 [details]
Patch

[Mark Rowe (bdash)]

Comment on attachment 229342 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=229342&action=review

> Source/WTF/WTF.xcodeproj/project.pbxproj:328
> +		1447AEBE18FCE56900B3D7FF /* libmbmalloc.dylib */ = {isa = PBXFileReference; explicitFileType = "compiled.mach-o.dylib"; includeInIndex = 0; path = libmbmalloc.dylib; sourceTree = BUILT_PRODUCTS_DIR; };
> +		1447AEC518FCE57700B3D7FF /* Foundation.framework */ = {isa = PBXFileReference; lastKnownFileType = wrapper.framework; name = Foundation.framework; path = Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.9.sdk/System/Library/Frameworks/Foundation.framework; sourceTree = DEVELOPER_DIR; };

These paths need to be SDK-relative, not relative to the developer directory.

> Source/WTF/WTF.xcodeproj/project.pbxproj:1191
> +		1447AEBD18FCE56900B3D7FF /* mbmalloc */ = {

Can you give the target a more descriptive name?

> Source/WTF/WTF.xcodeproj/project.pbxproj:1383
> +			buildSettings = {

These settings all need to live in a .xcconfig file, and should be consistent with our other targets.

[Geoffrey Garen]

Created attachment 229401 [details]
Patch

[Geoffrey Garen]

> > Source/WTF/WTF.xcodeproj/project.pbxproj:328
> > +		1447AEBE18FCE56900B3D7FF /* libmbmalloc.dylib */ = {isa = PBXFileReference; explicitFileType = "compiled.mach-o.dylib"; includeInIndex = 0; path = libmbmalloc.dylib; sourceTree = BUILT_PRODUCTS_DIR; };
> > +		1447AEC518FCE57700B3D7FF /* Foundation.framework */ = {isa = PBXFileReference; lastKnownFileType = wrapper.framework; name = Foundation.framework; path = Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.9.sdk/System/Library/Frameworks/Foundation.framework; sourceTree = DEVELOPER_DIR; };
> 
> These paths need to be SDK-relative, not relative to the developer directory.

Fixed to use the "<absolute>" style, like WTF and JavaScriptCore.

> > Source/WTF/WTF.xcodeproj/project.pbxproj:1191
> > +		1447AEBD18FCE56900B3D7FF /* mbmalloc */ = {
> 
> Can you give the target a more descriptive name?

This name is required by MallocBench. (That's what the "mb" is for.)

> > Source/WTF/WTF.xcodeproj/project.pbxproj:1383
> > +			buildSettings = {
> 
> These settings all need to live in a .xcconfig file, and should be consistent with our other targets.

Fixed.

[Darin Adler]

Comment on attachment 229401 [details]
Patch

Looks like a Mac-specific or Cocoa-specific patch, but bug title didn’t make that clear.

[Geoffrey Garen]

Committed r167355: <http://trac.webkit.org/changeset/167355>