Include dependencies for clang-tidy and header-abi-dumper (6b4419cc) · Commits · e / os / android_build_soong

We've been getting these dependencies transitively through the dependency on the object itself (which is a workaround for the lack of dep file support in these tools). But for remote builds to work, we need to know about these dependencies like any other object compilation. For regular builds, this increases the size of the ninja file by a few tens of megabytes (~1-2%). WITH_TIDY builds were already larger (~40-50%), but are now about 90% larger than a normal build. Test: treehugger Change-Id: Icdb4ca3d4d08d5706593d96d5c627149fa14fed8

cc/builder.go

+5 −1

Original line number	Diff line number	Diff line
		@@ -499,6 +499,8 @@ func TransformSourceToObj(ctx android.ModuleContext, subdir string, srcFiles and
		// We must depend on objFile, since clang-tidy doesn't
		// support exporting dependencies.
		Implicit: objFile,
		Implicits: cFlagsDeps,
		OrderOnly: pathDeps,
		Args: map[string]string{
		"cFlags": moduleToolingCflags,
		"tidyFlags": flags.tidyFlags,
		@@ -516,6 +518,8 @@ func TransformSourceToObj(ctx android.ModuleContext, subdir string, srcFiles and
		Output: sAbiDumpFile,
		Input: srcFile,
		Implicit: objFile,
		Implicits: cFlagsDeps,
		OrderOnly: pathDeps,
		Args: map[string]string{
		"cFlags": moduleToolingCflags,
		"exportDirs": flags.sAbiFlags,