Skip to content

Commit

Permalink
Enable localization-of-domains optimization by default (chapel-lang#2…
Browse files Browse the repository at this point in the history
…5517)

[reviewed by @jabraham17]

This enables the localization-of-domains optimization by default, which
was added in chapel-lang#24391. We've been running this in the multi-locale
performance playground for the past month and haven't seen any
degradations due to the change, and are aware of the benefits when it
fires, so are enabling it by default.

This PR simply flips the existing flag and removes the explicit setting
of it in tests that were designed to make sure it was working as
expected.
  • Loading branch information
bradcray authored Aug 23, 2024
2 parents aadcfc0 + a8b6a1a commit 799140a
Show file tree
Hide file tree
Showing 6 changed files with 8 additions and 10 deletions.
6 changes: 2 additions & 4 deletions modules/internal/ChapelArray.chpl
Original file line number Diff line number Diff line change
Expand Up @@ -3057,11 +3057,9 @@ module ChapelArray {
//
// These control an optimization in which copying an array from one
// locale to another will also copy the array's domain if we believe
// it's safe to do so. This optimization is currently off by
// default because it was completed close to Chapel 2.1 and we
// wanted more time to live with it before having it on by default.
// it's safe to do so. The optimization is currently on by default.
//
config param localizeConstDomains = false,
config param localizeConstDomains = true,
debugLocalizedConstDomains = false;

pragma "init copy fn"
Expand Down
2 changes: 1 addition & 1 deletion test/arrays/localizeConstDom.compopts
Original file line number Diff line number Diff line change
@@ -1 +1 @@
-slocalizeConstDomains=true -sdebugLocalizedConstDomains=true
-sdebugLocalizedConstDomains=true
4 changes: 2 additions & 2 deletions test/arrays/sparse/copyRemoteSparse-cs.compopts
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
-slocalizeConstDomains=true -sdebugLocalizedConstDomains=true --no-cache-remote -scompressRows=true
-slocalizeConstDomains=true -sdebugLocalizedConstDomains=true --no-cache-remote -scompressRows=false
-sdebugLocalizedConstDomains=true --no-cache-remote -scompressRows=true
-sdebugLocalizedConstDomains=true --no-cache-remote -scompressRows=false
2 changes: 1 addition & 1 deletion test/arrays/sparse/copyRemoteSparse.compopts
Original file line number Diff line number Diff line change
@@ -1 +1 @@
-slocalizeConstDomains=true -sdebugLocalizedConstDomains=true --no-cache-remote
-sdebugLocalizedConstDomains=true --no-cache-remote
Original file line number Diff line number Diff line change
@@ -1 +1 @@
-slocalizeConstDomains=true -sdebugLocalizedConstDomains=true
-sdebugLocalizedConstDomains=true
2 changes: 1 addition & 1 deletion test/studies/spsMatMatMult/driver.compopts
Original file line number Diff line number Diff line change
@@ -1 +1 @@
--no-cache-remote -slocalizeConstDomains=true
--no-cache-remote

0 comments on commit 799140a

Please sign in to comment.