diff options
author | Colin Cross <ccross@android.com> | 2016-10-26 10:03:47 -0700 |
---|---|---|
committer | Colin Cross <ccross@android.com> | 2016-10-27 15:18:13 -0700 |
commit | 2f33635542b5fe97b6779cb2c6b3f5d2277c51b5 (patch) | |
tree | 1eebb57b097c0ab7b7ab44b727cb48fed3c463ab /cc/compiler.go | |
parent | 46974e2457c2d62366e463f84212caaac42ee8f2 (diff) | |
download | build_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.go | 33 |
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) } |