(Closes #2561) Add support for allocatables in the HoistLocalArraysTrans

[sergisiso]

No description provided.

[codecov]

Codecov Report

Attention: Patch coverage is 96.82540% with 2 lines in your changes missing coverage. Please review.

Project coverage is 99.89%. Comparing base (7ef4ec2) to head (d95917d).

Files with missing lines	Patch %	Lines
.../psyir/transformations/hoist_local_arrays_trans.py	96.82%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2893      +/-   ##
==========================================
- Coverage   99.89%   99.89%   -0.01%     
==========================================
  Files         362      362              
  Lines       51571    51616      +45     
==========================================
+ Hits        51519    51562      +43     
- Misses         52       54       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[sergisiso]

Integration test passed but the change was not applied almost anywhere because the arrays of interest have:
real(kind=wp), ...
but the transformation validation does:

if isinstance(sym.datatype.intrinsic, DataTypeSymbol):
    continue
if isinstance(sym.datatype.precision, DataSymbol):
    continue

I think we could refine this to check that this symbol is not local to the subroutine and the subroutine doesn't have wilcard imports.