Andy Porter

Results 719 comments of Andy Porter

Fix is relatively straightforward - just use `SIZE` if we don't have an ArrayMixin. However, I've also found a bug in the validation of ArrayAssignment2LoopsTrans that I need to address.

* Error in part 1. "sched" should be "psyir": ![image](https://github.com/stfc/PSyclone/assets/17409212/3a1e50a4-41f3-4bd7-a331-a53278f8ea67)

Can't you change PROFILE_LINK? If I build dl_timer in MPI mode and then set: ``` PROFILE_DIR ?= ../../../../lib/profiling/dl_timer PROFILE_LIB = ${PROFILE_DIR}/libdl_timer_psy.a PROFILE_LINK = ${PROFILE_LIB} ~/Projects/dl_timer/libdl_timer_mpi.a ``` and `export F90=mpif90` then...

Error in tutorial 3: ![image](https://github.com/stfc/PSyclone/assets/17409212/b2d005ff-a730-4565-8b06-03787213647a) (note use of `child` in loop body)

3.1: OMPParallelLoopTrans (probably ParallelLoopTrans) `validate` just says there's a CodeBlock. It would be better if it printed the contents of the CodeBlock.

3_nemo_omp/omp_trans.py already has a `walk` in it but the tutorial instructions say to add one: ![image](https://github.com/stfc/PSyclone/assets/17409212/f3f21af9-dca5-4402-a65d-6210683cc56b) I guess originally we just had a loop over `children`? And the text above...

> btw, now that the psyir starts at root and is easier to handle multiple routines easily we could split the tra_adv into multiple routines. This would allow the default...

Difficulties validating the OpenACC Kernels version of the tra_adv miniapp on NVIDIA GPU - does this work on Glados?

There is an assignment to a scalar (zdt) which rings a bell for me.

Expose ability to run backend checks to user/script?