[Chapel Merge] Remove isAggregateTypeAndConstructor()

Branch: refs/heads/master
Revision: 9516253
Author: vasslitvinov
Log Message:

Merge pull request #16637 from vasslitvinov/isAggTypeAndConst

Remove isAggregateTypeAndConstructor()

Remove code that should no longer fire, since we do not have C+±constructor-style naming:

// the now-removed code allowed ‘R’ mean both a type and a function
record R {…}
proc R(…){…} // used to be a constructor function

An error in the above code saying “this is a constructor, use an initializer instead” is issued elsewhere. It is not affected by this change.

r: @mppf

Modified Files:
M compiler/include/ResolveScope.h
M compiler/passes/ResolveScope.cpp

Compare: https://github.com/chapel-lang/chapel/compare/9a1e0f75d0e7...951625329ba8