PSyclone icon indicating copy to clipboard operation
PSyclone copied to clipboard

(towards #2577) Add support for pointer assignments

Open sergisiso opened this issue 1 year ago • 1 comments

sergisiso avatar May 09 '24 22:05 sergisiso

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.85%. Comparing base (4d10caa) to head (09d4cec).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2583      +/-   ##
==========================================
- Coverage   99.86%   99.85%   -0.01%     
==========================================
  Files         351      352       +1     
  Lines       48396    48455      +59     
==========================================
+ Hits        48331    48386      +55     
- Misses         65       69       +4     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar May 09 '24 22:05 codecov[bot]

This is ready for a first look from @arporter or @hiker . It adds support for pointer assignments (removes the CodeBlocks), but not symbol attributes (does not remove UnknownFortranType for its symbols). It also adds support in the frontend for Fortran2003.Data_Pointer_Object fparser object. For now I added some broad protections in the base loop_trans and region_trans to prevent transformation to apply to code that have pointer assignments (which can be disabled with a transformation option).

sergisiso avatar Jun 25 '24 09:06 sergisiso

@arporter This is ready for next review. In comparison to the first review I found out that pointer expressions that have array accessor ranges in the inner element are not described like regular arrays in the standard/fparser. So I decided to not support these cases yet - they are CodeBlocks as before.

sergisiso avatar Jul 01 '24 11:07 sergisiso

@arporter This is ready for another review

sergisiso avatar Jul 02 '24 08:07 sergisiso