currently MMGC_MEMORY_PROFILER is hardcoded ON for some platforms, OFF for the rest in MMgc.h. We should move it into the existing config system so that embedders can enable it (or not) in line with everything else.
Assignee: nobody → treilly
Status: NEW → ASSIGNED
Priority: -- → P2
Target Milestone: --- → flash10.1
Comment on attachment 396454 [details] [diff] [review] Patch VMCFG_MEMORY_PROFILER since MMGC_ names are slated to disappear sooner or later, but I'm not going to insist - we can fix that later.
pushed as changeset: 2400:158ddad2728e
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
For some reason it appears that the memory profiler is not being enabled by default on windows. The code change in shell/avmshell-features.h should turn on the profiler on windows but not ARM, this is the exact same check as before in MMgc/MMgc.h This is currently causing the "smokes" to fail as there is a memstat test that is not working.
change: AVMPLUS_(MAC|ARM|WIN32) to AVMFEATURE_(MAC|ARM|WIN32) MMGC_PPC to AVMSYSTEM_PPC MMGC_64BIT to SIXTYFOURBIT
Use AVMSYSTEM_64BIT instead of SIXTYFOURBIT which is undef'd
Apologies -- bad fix on my part :-/
Comment on attachment 396738 [details] [diff] [review] patch avmshell-features.h approved, but I think we prefer #if FOO (not FOO==1) and #if !FOO (not FOO==0) as general form...
Attachment #396738 - Flags: review?(stejohns) → review+
Updated patch and pushed as 2405:aefaa59b3743
Resolved fixed engineering / work item that has been pushed. Setting status to verified.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.