[Chapel Merge] Fix auto aggregation tests

Branch: refs/heads/main
Revision: 35877b9
Author: ronawho
Link: Fix auto aggregation tests by ronawho · Pull Request #19423 · chapel-lang/chapel · GitHub
Log Message:

Merge pull request #19423 from ronawho/fix-auto-agg-tests

Fix auto aggregation tests

[reviewed by @bradcray]

Fix auto aggregation tests that print verbose output and have
auto-aggregation call the aggregator implementation directly

The auto aggregation tests compile with verboseAggregation, but that
param was made private in #19386, so we need to give the fully qualified
path now.

In #19386 I added a wrapper to avoid aggregating for comm=none and to
make documenting easier. This was causing some compiler failures for
cleanupRemainingAggCondStmts() because the AST now had an additional
wrapper. I didn't feel comfortable updating this code, so for now I just
updated the code to call the non-wrapped aggregator directly.

Modified Files:
M modules/internal/ChapelAutoAggregation.chpl

M test/optimizations/autoAggregation/COMPOPTS
M test/optimizations/autoAggregation/perf/autoAggNegativeImpact.compopts

Compare: https://github.com/chapel-lang/chapel/compare/a43fc8872f8e...35877b9abab8