Project

General

Profile

Actions

Bug #5212

closed

PCH is not very effective

Added by Davide Pesavento almost 2 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Normal
Category:
Build
Target version:
Start date:
06/04/2022
Due date:
% Done:

100%

Estimated time:

Description

The precompiled header core/common.hpp is used only for translation units in core/ (basically only network.cpp since version.cpp doesn't need any include). This makes the PCH feature almost useless. The bulk of NFD's code is in daemon/ and that's where the PCH should be applied.

Moreover, it would be beneficial to precompile a few additional headers on top of core/common.hpp, such as ndn-cxx/util/logger.hpp which turned out to be one of the slowest to compile due to lots of transitive Boost includes.

Actions #1

Updated by Davide Pesavento almost 2 years ago

  • Description updated (diff)
Actions #2

Updated by Davide Pesavento almost 2 years ago

  • Status changed from New to Code review
  • % Done changed from 0 to 100
Actions #3

Updated by Davide Pesavento almost 2 years ago

  • Status changed from Code review to Closed
Actions

Also available in: Atom PDF