aboutsummaryrefslogtreecommitdiffstats
path: root/cc/compiler.go
diff options
context:
space:
mode:
authorColin Cross <ccross@android.com>2016-10-26 10:03:47 -0700
committerColin Cross <ccross@android.com>2016-10-27 15:18:13 -0700
commit2f33635542b5fe97b6779cb2c6b3f5d2277c51b5 (patch)
tree1eebb57b097c0ab7b7ab44b727cb48fed3c463ab /cc/compiler.go
parent46974e2457c2d62366e463f84212caaac42ee8f2 (diff)
downloadbuild_soong-2f33635542b5fe97b6779cb2c6b3f5d2277c51b5.tar.gz
build_soong-2f33635542b5fe97b6779cb2c6b3f5d2277c51b5.tar.bz2
build_soong-2f33635542b5fe97b6779cb2c6b3f5d2277c51b5.zip
Simplify compileObjs
None of the direct users of compileObjs were using any of its complexity (generated sources, excluded sources, extra sources). Move the complexity back in to baseCompiler.compile. Test: m -j Change-Id: I2e59d216682c00dd12a1395cf2448827d1c48023
Diffstat (limited to 'cc/compiler.go')
-rw-r--r--cc/compiler.go33
1 files changed, 18 insertions, 15 deletions
diff --git a/cc/compiler.go b/cc/compiler.go
index a0068ada..32d98e4a 100644
--- a/cc/compiler.go
+++ b/cc/compiler.go
@@ -100,6 +100,7 @@ func NewBaseCompiler() *baseCompiler {
type baseCompiler struct {
Properties BaseCompilerProperties
+ deps android.Paths
}
var _ compiler = (*baseCompiler)(nil)
@@ -337,10 +338,21 @@ func ndkPathDeps(ctx ModuleContext) android.Paths {
func (compiler *baseCompiler) compile(ctx ModuleContext, flags Flags, deps PathDeps) android.Paths {
pathDeps := deps.GeneratedHeaders
pathDeps = append(pathDeps, ndkPathDeps(ctx)...)
+
+ srcs := ctx.ExpandSources(compiler.Properties.Srcs, compiler.Properties.Exclude_srcs)
+ srcs = append(srcs, deps.GeneratedSources...)
+
+ buildFlags := flagsToBuilderFlags(flags)
+
+ srcs, genDeps := genSources(ctx, srcs, buildFlags)
+
+ pathDeps = append(pathDeps, genDeps...)
+ pathDeps = append(pathDeps, flags.CFlagsDeps...)
+
+ compiler.deps = pathDeps
+
// Compile files listed in c.Properties.Srcs into objects
- objFiles := compileObjs(ctx, flags, "",
- compiler.Properties.Srcs, compiler.Properties.Exclude_srcs,
- deps.GeneratedSources, pathDeps)
+ objFiles := compileObjs(ctx, buildFlags, "", srcs, compiler.deps)
if ctx.Failed() {
return nil
@@ -350,17 +362,8 @@ func (compiler *baseCompiler) compile(ctx ModuleContext, flags Flags, deps PathD
}
// Compile a list of source files into objects a specified subdirectory
-func compileObjs(ctx android.ModuleContext, flags Flags,
- subdir string, srcFiles, excludes []string, extraSrcs, deps android.Paths) android.Paths {
-
- buildFlags := flagsToBuilderFlags(flags)
-
- inputFiles := ctx.ExpandSources(srcFiles, excludes)
- inputFiles = append(inputFiles, extraSrcs...)
- srcPaths, gendeps := genSources(ctx, inputFiles, buildFlags)
-
- deps = append(deps, gendeps...)
- deps = append(deps, flags.CFlagsDeps...)
+func compileObjs(ctx android.ModuleContext, flags builderFlags,
+ subdir string, srcFiles, deps android.Paths) android.Paths {
- return TransformSourceToObj(ctx, subdir, srcPaths, buildFlags, deps)
+ return TransformSourceToObj(ctx, subdir, srcFiles, flags, deps)
}