Skip to content

Commit

Permalink
Reorder fields in query map result to help reduce memory footprint
Browse files Browse the repository at this point in the history
Signed-off-by: Danila Fedorin <[email protected]>
  • Loading branch information
DanilaFe committed Oct 11, 2024
1 parent 611510a commit 9bc8e83
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions frontend/include/chpl/framework/Context-detail.h
Original file line number Diff line number Diff line change
Expand Up @@ -275,8 +275,6 @@ class QueryMapResultBase {
// extra boolean is fine.
mutable bool beingTestedForReuse = false;

mutable QueryDependencyVec dependencies;

// Whether or not errors from this query result have been shown to the
// user (they may not have been if some query checked for errors).
mutable bool emittedErrors = false;
Expand All @@ -287,6 +285,8 @@ class QueryMapResultBase {
// This is not too strongly connected to emittedErrors (which tracks whether
// errors --- if any --- were shown to the user for this query result only)
mutable bool errorsPresentInSelfOrDependencies = false;

mutable QueryDependencyVec dependencies;
mutable std::set<const QueryMapResultBase*> recursionErrors;
mutable QueryErrorVec errors;

Expand Down
2 changes: 1 addition & 1 deletion frontend/lib/framework/Context.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1351,9 +1351,9 @@ QueryMapResultBase::QueryMapResultBase(RevisionNumber lastChecked,
: lastChecked(lastChecked),
lastChanged(lastChanged),
beingTestedForReuse(beingTestedForReuse),
dependencies(),
emittedErrors(emittedErrors),
errorsPresentInSelfOrDependencies(errorsPresentInSelfOrDependencies),
dependencies(),
recursionErrors(std::move(recursionErrors)),
errors(),
parentQueryMap(parentQueryMap) {
Expand Down

0 comments on commit 9bc8e83

Please sign in to comment.