add support for building --enable-jemalloc with VC2010

RESOLVED WONTFIX

Status

()

Core
Build Config
RESOLVED WONTFIX
7 years ago
7 years ago

People

(Reporter: ted, Unassigned)

Tracking

Trunk
x86
Windows 7
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

7 years ago
We should add a CRT patch to support building jemalloc with VC2010.
(Reporter)

Updated

7 years ago
Blocks: 563318
(Reporter)

Updated

7 years ago
Blocks: 563319
Blocks: 563293

Comment 1

7 years ago
I've seen some performance improvements when compiling with VC10 instead VC9 or VC8 using the same code compiling a cpu intensive application. are about 10-15%.

http://blogs.msdn.com/b/vcblog/archive/2009/12/01/gl-and-pgo.aspx

^^ hope you can make this CRT patch soon

cheers.

Comment 2

7 years ago
I'm currently trying to adopt the patches from VC9 to VC10, but I'm stuck at the *.rc and *.def files because there are no such files in Visual Studio 2010. Probably because Microsoft dropped support for building CRT libraries (see below).


http://msdn.microsoft.com/en-us/library/bb531344.aspx
---
Support has been removed for users to build the CRT, MFC, and ATL libraries. For example, an appropriate nmake file is not provided. However, users still have access to the source code for these libraries. And a document that describes the MSBuild options that Microsoft uses to build these libraries will probably be posted in a Visual C++ Team Blog.
------

Comment 3

7 years ago
Created attachment 448100 [details] [diff] [review]
partial patch for c/cpp/h files

adopt the patches from VC9 to VC10
(Reporter)

Comment 4

7 years ago
In light of that, I don't know that it makes sense to try to do this. We should probably WONTFIX this in favor of bug 515492.
(Reporter)

Updated

7 years ago
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → WONTFIX
(Reporter)

Updated

7 years ago
No longer blocks: 563318, 563319
No longer blocks: 563293
You need to log in before you can comment on or make changes to this bug.