Branch: refs/heads/main
Revision: c56bdec96cbe4bb9f4d855d05b32b9603d146439
Author: e-kayrakli
Link: Handle two issues that came up after some AVE improvements by e-kayrakli · Pull Request #26629 · chapel-lang/chapel · GitHub
Log Message:
Handle two issues that came up after some AVE improvements (#26629)
This PR handles two things:
- Trivially: adds/updates good files that were missed in my initial PR.
- More seriously: thwarts optimization when the rhs is a constant,
multidimensional array. This is done to work around
Non-inlined parallel iterators with `const ref` intent may lead to wrong results · Issue #26626 · chapel-lang/chapel · GitHub. When that issue is
resolved, the relevant commit can be reverted.
[Reviewed by @benharsh]
Test:
- local
- gasnet
Diff:
M compiler/optimizations/arrayViewElision.cpp
M modules/internal/ChapelArrayViewElision.chpl
A test/optimizations/arrayViewElision/localToDistributed.comm-none.good
M test/optimizations/arrayViewElision/localToDistributed.good
https://github.com/chapel-lang/chapel/pull/26629.diff