Branch: refs/heads/master
Revision: 81371c9
Author: e-kayrakli
Log Message:
Merge pull request #17344 from e-kayrakli/sparse-subdomain-syntax-error
Limit sparse subdomain clause to a single argument
If you pass a list of actuals to sparse subdomain
clause, an assertion is
triggered, although that is illegal Chapel code and should be reported as such.
It is an easy to make mistake:
var s: sparse subdomain({1..10}, {1..10}); // actually meaning {1..10, 1..10}
This PR limits parser to read a single actual in sparse subdomain
. Arguably,
we can have a better error message there, but this is an easy and quick fix for
now.
[Reviewed by @bradcray]
Test
-
[x] standard
-
[x] gasnet
Modified Files:
A test/parsing/errors/sparseSubdomainActualList.chpl
A test/parsing/errors/sparseSubdomainActualList.good
M compiler/include/bison-chapel.h
M compiler/parser/bison-chapel.cpp
M compiler/parser/chapel.yppCompare: Comparing da7b6b2f7fa7...81371c98836f · chapel-lang/chapel · GitHub