Convert VLOG to DVLOG (2016 edition) |
|
Issue descriptionIt was noted long ago at https://groups.google.com/a/chromium.org/d/topic/chromium-dev/nAiKeh-zLws/discussion and https://bugs.chromium.org/p/chromium/issues/detail?id=101424 that VLOGs were a sizable chunk of the released code with limited utility. At r148448ef, Windows, target_cpu=x86, is_debug=false, is_chrome_branded=true, is_official_build=true. chrome.dll: 40,403,456 chrome_child.dll: 49,829,376 chrome_elf.dll: 415,232 chrome.exe: 940,533 Then, with VLOG/VLOG_IF compiled out https://codereview.chromium.org/2386613003 : chrome.dll: 40,254,464 chrome_child.dll: 49,748,992 chrome_elf.dll: 414,208 chrome.exe: 938,496 For a difference of 232,255. We should probably declare VLOG bankruptcy again and convert VLOG()s to DVLOG()s, again.
,
Apr 30 2018
VLOG also consumes quite a lot CPU cycles. It would be nice to keep it out of release version. |
|
►
Sign in to add a comment |
|
Comment 1 by scottmg@chromium.org
, Sep 30 2016