build: Updated precompiled header support (enabled by default)
Change-Id: Iaab62c740ac87b84256a3f71d934760865a54be5
diff --git a/.waf-tools/pch.py b/.waf-tools/pch.py
index c0e3039..1b963ae 100644
--- a/.waf-tools/pch.py
+++ b/.waf-tools/pch.py
@@ -1,75 +1,148 @@
#! /usr/bin/env python
# encoding: utf-8
+# Alexander Afanasyev (UCLA), 2014
-from waflib import Logs, Utils, Task, TaskGen, Build
+"""
+Enable precompiled C++ header support (currently only clang++ and g++ are supported)
+
+To use this tool, wscript should look like:
+
+ def options(opt):
+ opt.load('pch')
+ # This will add `--with-pch` configure option.
+ # Unless --with-pch during configure stage specified, the precompiled header support is disabled
+
+ def configure(conf):
+ conf.load('pch')
+ # this will set conf.env.WITH_PCH if --with-pch is specified and the supported compiler is used
+ # Unless conf.env.WITH_PCH is set, the precompiled header support is disabled
+
+ def build(bld):
+ bld(features='cxx pch',
+ target='precompiled-headers',
+ name='precompiled-headers',
+ headers='a.h b.h c.h', # headers to pre-compile into `precompiled-headers`
+
+ # Other parameters to compile precompiled headers
+ # includes=...,
+ # export_includes=...,
+ # use=...,
+ # ...
+
+ # Exported parameters will be propagated even if precompiled headers are disabled
+ )
+
+ bld(
+ target='test',
+ features='cxx cxxprogram',
+ source='a.cpp b.cpp d.cpp main.cpp',
+ use='precompiled-headers',
+ )
+
+ # or
+
+ bld(
+ target='test',
+ features='pch cxx cxxprogram',
+ source='a.cpp b.cpp d.cpp main.cpp',
+ headers='a.h b.h c.h',
+ )
+
+Note that precompiled header must have multiple inclusion guards. If the guards are missing, any benefit of precompiled header will be voided and compilation may fail in some cases.
+"""
+
+import os
+from waflib import Task, TaskGen, Logs, Utils
from waflib.Tools import c_preproc, cxx
-def options(opt):
- opt.add_option('--with-pch', action='store_true', default=False, dest='with_pch',
- help='''Try to use precompiled header to speed up compilation '''
- '''(only gcc and clang)''')
-
-def configure(conf):
- conf.env['WITH_PCH'] = conf.options.with_pch
-
-class gchx(Task.Task):
- run_str = '${CXX} -x c++-header ${CXXFLAGS} ${FRAMEWORKPATH_ST:FRAMEWORKPATH} ' + \
- '${CPPPATH_ST:INCPATHS} ${DEFINES_ST:DEFINES} ' + \
- '${CXX_SRC_F}${SRC} ${CXX_TGT_F}${TGT}'
- scan = c_preproc.scan
- color = 'BLUE'
-
PCH_COMPILER_OPTIONS = {
- 'clang++': ['-include', '.pch', ''],
- 'g++': ['-include', '.gch', ''],
+ 'clang++': [['-include'], '.pch', ['-x', 'c++-header']],
+ 'g++': [['-include'], '.gch', ['-x', 'c++-header']],
}
-@TaskGen.extension('.cpp')
-def cxx_hook(self, node):
- if (self.env.WITH_PCH and
- self.env['COMPILER_CXX'] in PCH_COMPILER_OPTIONS.keys() and
- getattr(self, 'pch', None)):
- (flag, pch_ext, include_ext) = PCH_COMPILER_OPTIONS[self.env['COMPILER_CXX']]
+def options(opt):
+ opt.add_option('--without-pch', action='store_false', default=True, dest='with_pch', help='''Try to use precompiled header to speed up compilation (only g++ and clang++)''')
- if not getattr(self, 'pch_task', None):
- if isinstance(self.pch, str):
- self.pch = self.path.find_node(self.pch)
+def configure(conf):
+ if (conf.options.with_pch and conf.env['COMPILER_CXX'] in PCH_COMPILER_OPTIONS.keys()):
+ conf.env.WITH_PCH = True
+ flags = PCH_COMPILER_OPTIONS[conf.env['COMPILER_CXX']]
+ conf.env.CXXPCH_F = flags[0]
+ conf.env.CXXPCH_EXT = flags[1]
+ conf.env.CXXPCH_FLAGS = flags[2]
- output = self.pch.change_ext('.hpp%s' % pch_ext)
- try:
- # == Will reuse the existing task ==
- # This could cause problem if different compiler flags were used for
- # previous compilation of the precompiled header
- existingTask = self.bld.get_tgen_by_name(output.abspath())
- self.pch_task = existingTask
- except:
- self.pch_task = self.create_task('gchx', self.pch, output)
- self.pch_task.name = output.abspath()
+@TaskGen.feature('pch')
+@TaskGen.before('process_source')
+def apply_pch(self):
+ if not self.env.WITH_PCH:
+ return
- self.bld.task_gen_cache_names = {}
- self.bld.add_to_group(self.pch_task, group=getattr(self, 'group', None))
+ if getattr(self.bld, 'pch_tasks', None) is None:
+ self.bld.pch_tasks = {}
- task = self.create_compiled_task('cxx_pch', node)
- # task.run_after.update(set([self.pch_task]))
+ if getattr(self, 'headers', None) is None:
+ return
- task.pch_flag = flag
- task.pch_include_file = self.pch.change_ext('.hpp%s' % include_ext).get_bld()
+ self.headers = self.to_nodes(self.headers)
- out = '%s%s' % (self.pch.name, pch_ext)
- task.pch_file = self.pch.parent.get_bld().find_or_declare(out)
- else:
- self.create_compiled_task('cxx', node)
+ if getattr(self, 'name', None):
+ try:
+ task = self.bld.pch_tasks[self.name]
+ self.bld.fatal("Duplicated 'pch' task with name %r" % self.name)
+ except KeyError:
+ pass
-class cxx_pch(Task.classes['cxx']):
- run_str = '${CXX} ${tsk.pch_flag} ${tsk.pch_include_file.abspath()} '\
- '${ARCH_ST:ARCH} ${CXXFLAGS} ${CPPFLAGS} ' \
- '${FRAMEWORKPATH_ST:FRAMEWORKPATH} ${CPPPATH_ST:INCPATHS} ${DEFINES_ST:DEFINES} ' \
- '${CXX_SRC_F}${SRC} ${CXX_TGT_F}${TGT[0].abspath()}'
+ out = '%s.%d%s' % (self.target, self.idx, self.env['CXXPCH_EXT'])
+ out = self.path.find_or_declare(out)
+ task = self.create_task('gchx', self.headers, out)
- def scan(self):
- (nodes, names) = c_preproc.scan(self)
- nodes.append(self.pch_file)
- return (nodes, names)
+ # target should be an absolute path of `out`, but without precompiled header extension
+ task.target = out.abspath()[:-len(out.suffix())]
+
+ self.pch_task = task
+ if getattr(self, 'name', None):
+ self.bld.pch_tasks[self.name] = task
+
+@TaskGen.feature('cxx')
+@TaskGen.after_method('process_source', 'propagate_uselib_vars')
+def add_pch(self):
+ if not (self.env['WITH_PCH'] and getattr(self, 'use', None) and getattr(self, 'compiled_tasks', None) and getattr(self.bld, 'pch_tasks', None)):
+ return
+
+ pch = None
+ # find pch task, if any
+
+ if getattr(self, 'pch_task', None):
+ pch = self.pch_task
+ else:
+ for use in Utils.to_list(self.use):
+ try:
+ pch = self.bld.pch_tasks[use]
+ except KeyError:
+ pass
+
+ if pch:
+ for x in self.compiled_tasks:
+ x.env.append_value('CXXFLAGS', self.env['CXXPCH_F'] + [pch.target])
+
+class gchx(Task.Task):
+ run_str = '${CXX} ${ARCH_ST:ARCH} ${CXXFLAGS} ${CPPFLAGS} ${CXXPCH_FLAGS} ${FRAMEWORKPATH_ST:FRAMEWORKPATH} ${CPPPATH_ST:INCPATHS} ${DEFINES_ST:DEFINES} ${CXXPCH_F:SRC} ${CXX_SRC_F}${SRC[0].abspath()} ${CXX_TGT_F}${TGT[0].abspath()}'
+ scan = c_preproc.scan
+ color = 'BLUE'
+ ext_out=['.h']
+
+ def runnable_status(self):
+ try:
+ node_deps = self.generator.bld.node_deps[self.uid()]
+ except KeyError:
+ node_deps = []
+ ret = Task.Task.runnable_status(self)
+ if ret == Task.SKIP_ME and self.env.CXX_NAME == 'clang':
+ t = os.stat(self.outputs[0].abspath()).st_mtime
+ for n in self.inputs + node_deps:
+ if os.stat(n.abspath()).st_mtime > t:
+ return Task.RUN_ME
+ return ret
diff --git a/.waf-tools/sphinx_build.py b/.waf-tools/sphinx_build.py
index bd3905b..e61da6e 100644
--- a/.waf-tools/sphinx_build.py
+++ b/.waf-tools/sphinx_build.py
@@ -12,8 +12,8 @@
def __str__(self):
env = self.env
- src_str = ' '.join([a.nice_path()for a in self.inputs])
- tgt_str = ' '.join([a.nice_path()for a in self.outputs])
+ src_str = ' '.join([a.path_from(a.ctx.launch_node()) for a in self.inputs])
+ tgt_str = ' '.join([a.path_from(a.ctx.launch_node()) for a in self.outputs])
if self.outputs: sep = ' -> '
else: sep = ''
return'%s [%s]: %s%s%s\n'%(self.__class__.__name__.replace('_task',''),