From 23b9efa8ff460825e7a2abf6a6a22229152eb711 Mon Sep 17 00:00:00 2001 From: Dan Willemsen Date: Wed, 21 Feb 2018 18:47:18 -0800 Subject: Add SourceFileGenerator.GeneratedDeps() To clean up unnecessary dependencies when using generated headers, GeneratedDeps() should return a single file from every build entry. GeneratedSourceFiles() used to be used from this, but now if you create one build rule with multiple outputs, it's valid to only return a single output file. In this case, ModuleBuild is called once per source file, each outputing a single output file, so all of them still need to be listed. Bug: 73745773 Test: diff out/soong/build.ninja Test: diff out/soong/Android-aosp_arm.mk Test: rm -rf out; m Change-Id: I073e3be7ecd9545f4002ad94d3da44f1a6652baa --- soong/tblgen.go | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/soong/tblgen.go b/soong/tblgen.go index b82c81efb0a1..b0067be36d0a 100644 --- a/soong/tblgen.go +++ b/soong/tblgen.go @@ -145,6 +145,10 @@ func (t *tblgen) GeneratedHeaderDirs() android.Paths { } func (t *tblgen) GeneratedSourceFiles() android.Paths { + return nil +} + +func (t *tblgen) GeneratedDeps() android.Paths { return t.generatedHeaders } -- cgit v1.2.3