Skip to content

Commit

Permalink
Update comment around optimization and update tests that were enablin…
Browse files Browse the repository at this point in the history
…g manually

---
Signed-off-by: Brad Chamberlain <bradcray@users.noreply.github.com>
  • Loading branch information
bradcray committed Aug 21, 2024
1 parent 42b7f96 commit a8b6a1a
Show file tree
Hide file tree
Showing 6 changed files with 7 additions and 9 deletions.
4 changes: 1 addition & 3 deletions modules/internal/ChapelArray.chpl
Original file line number Diff line number Diff line change
Expand Up @@ -3057,9 +3057,7 @@ 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 = true,
debugLocalizedConstDomains = false;
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 a8b6a1a

Please sign in to comment.