diff options
| author | Cole Faust <colefaust@google.com> | 2024-01-08 17:51:55 -0800 |
|---|---|---|
| committer | Cole Faust <colefaust@google.com> | 2024-01-08 17:51:55 -0800 |
| commit | 12aef5a2be49b32f379ea04b9634a9957f473899 (patch) | |
| tree | 00abcc2d67bb114835cd5ed1887e88388a8a2134 | |
| parent | f09fd44326ca857723df01bd9df5f4c986775178 (diff) | |
| download | clang-12aef5a2be49b32f379ea04b9634a9957f473899.tar.gz | |
Remove recursive rule variable reference
The command variable was referring to the depfile variable in the same
rule scope. These inter-scope references are only possible in rules,
not builds or the global scope. (though the global scope can reference
earlier variables)
n2 doesn't want to support this because it would slow down evaluating
the build file, so remove the usage.
Bug: 318434287
Test: Presubmits
Change-Id: Iffbfe74dc7a400ff2fa827ab201995eeeba29280
| -rw-r--r-- | soong/tblgen.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/soong/tblgen.go b/soong/tblgen.go index 67b9bb8e5a..aa14b42168 100644 --- a/soong/tblgen.go +++ b/soong/tblgen.go @@ -37,10 +37,10 @@ var ( tblgenRule = pctx.StaticRule("tblgenRule", blueprint.RuleParams{ Depfile: "${out}.d", Deps: blueprint.DepsGCC, - Command: "${clangTblgen} ${includes} ${genopt} -d ${depfile} -o ${out} ${in}", + Command: "${clangTblgen} ${includes} ${genopt} -d ${out}.d -o ${out} ${in}", CommandDeps: []string{"${clangTblgen}"}, Description: "Clang TableGen $in => $out", - }, "includes", "depfile", "genopt") + }, "includes", "genopt") ) type tblgenProperties struct { |