diff options
| author | Boris Kolpackov <boris@codesynthesis.com> | 2016-04-21 12:18:15 +0200 |
|---|---|---|
| committer | Boris Kolpackov <boris@codesynthesis.com> | 2016-04-21 12:18:15 +0200 |
| commit | 6082d76936b8a65380eb7af03b4167d8f0298158 (patch) | |
| tree | 4cfadd3f54c134a1a45086ad15015a88b902ffba /build2/cxx/compile.cxx | |
| parent | 0165fa7178319bb250be1882b3b457232236c820 (diff) | |
Implement short-circuiting to group state
This is necessary to get rid of bogus restarts in inject_prerequisites()
where it think a group member was updated since its state changed from
unknown to (group's) changed.
Diffstat (limited to 'build2/cxx/compile.cxx')
| -rw-r--r-- | build2/cxx/compile.cxx | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/build2/cxx/compile.cxx b/build2/cxx/compile.cxx index b7577d4..a7986c5 100644 --- a/build2/cxx/compile.cxx +++ b/build2/cxx/compile.cxx @@ -571,7 +571,9 @@ namespace build2 if (ns != os && ns != target_state::unchanged) { - l6 ([&]{trace << "updated " << pt;}); + l6 ([&]{trace << "updated " << pt + << "; old state " << os + << "; new state " << ns;}); return true; } } @@ -588,8 +590,8 @@ namespace build2 return false; }; - // Update and add header file to the list of prerequisite targets. - // Depending on the cache flag, the file is assumed to either have comes + // Update and add a header file to the list of prerequisite targets. + // Depending on the cache flag, the file is assumed to either have come // from the depdb cache or from the compiler run. Return whether the // extraction process should be restarted. // |