It should work :-)
I'd check chrome/app/chrome_dll.vcproj for changes.

The basic idea is:
   * these functions are now provided by an alternate libcmt which is in the
tcmalloc directory.
   * there is a build step as part of tcmalloc which creates this library
(prep_libc.sh)
   * chrome.dll now ignores libcmt.lib and manually links its own libcmt.

Mike



On Wed, Apr 22, 2009 at 7:25 AM, Sverrir Á. Berg <[email protected]>wrote:

> Even after doing the clobber & runhooks raindance I'm getting linker
> errors:
> 143>tcmalloc.lib(override_functions.obj) : error LNK2005: _malloc already
> defined in libcmt.lib(malloc.obj)
> 143>tcmalloc.lib(override_functions.obj) : error LNK2005: _free already
> defined in libcmt.lib(free.obj)
> 143>tcmalloc.lib(override_functions.obj) : error LNK2005: _calloc already
> defined in libcmt.lib(calloc.obj)
> ... (16 functions total)
>
> (VS2008/external depot)
>
> Any suggestions on how to solve this?
> Sverrir
>
>
> On Wed, Apr 22, 2009 at 10:05 AM, Mike Belshe <[email protected]> wrote:
>
>> I turned on tcmalloc last night as an experiment.  I left it on so far.
>> The good news is the perf:
>> http://build.chromium.org/buildbot/perf/dashboard/overview.html
>>
>> The bad news is the reliability bot shows some crashes.  I believe these
>> are crashes not caused by tcmalloc but exposed by it.  We'll figure out a
>> plan to fix them today, or to revert tcmalloc again.
>>
>> Thanks for your patience while we sort this out!
>>
>> Mike
>>
>>
>> >>
>>
>

--~--~---------~--~----~------------~-------~--~----~
Chromium Developers mailing list: [email protected] 
View archives, change email options, or unsubscribe: 
    http://groups.google.com/group/chromium-dev
-~----------~----~----~----~------~----~------~--~---

Reply via email to