Dear Sandra,

Sandra Loosemore wrote:
This patch consolidates some repetitive code from gfc_trans_omp_clauses
into new functions, and adds some comments.  There are no functional
changes.
(This is for both the data-mapping clauses and for the depend/affinity clausse, which all support an 'iterator' modifier.)

gcc/fortran/ChangeLog
        * trans-openmp.cc (handle_iterator):  Add comments.  Rename
        iter_block parameter to outer_block, since that's what's actually
        being passed; the start/end/step expressions are outside the scope
        of the iterator variables.
        (start_iterator_group, finish_iterator_group): New functions,
        broken out from...
        (gfc_trans_omp_clauses): ...here.

LGTM - and thanks for the cleanup!

Tobias

Reply via email to