https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111017

--- Comment #6 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-13 branch has been updated by Tobias Burnus
<bur...@gcc.gnu.org>:

https://gcc.gnu.org/g:d4648a00df7a6950f7e1d12743e17a8583af0e5f

commit r13-7756-gd4648a00df7a6950f7e1d12743e17a8583af0e5f
Author: Tobias Burnus <tob...@codesourcery.com>
Date:   Sat Aug 19 07:49:06 2023 +0200

    omp-expand.cc: Fix wrong code with non-rectangular loop nest [PR111017]

    Before commit r12-5295-g47de0b56ee455e, all gimple_build_cond in
    expand_omp_for_* were inserted with
      gsi_insert_before (gsi_p, cond_stmt, GSI_SAME_STMT);
    except the one dealing with the multiplicative factor that was
      gsi_insert_after (gsi, cond_stmt, GSI_CONTINUE_LINKING);

    That commit for PR103208 fixed the issue of some missing regimplify of
    operands of GIMPLE_CONDs by moving the condition handling to the new
function
    expand_omp_build_cond. While that function has an 'bool after = false'
    argument to switch between the two variants.

    However, all callers ommited this argument. This commit reinstates the
    prior behavior by passing 'true' for the factor != 0 condition, fixing
    the included testcase.

            PR middle-end/111017
    gcc/
            * omp-expand.cc (expand_omp_for_init_vars): Pass after=true
            to expand_omp_build_cond for 'factor != 0' condition, resulting
            in pre-r12-5295-g47de0b56ee455e code for the gimple insert.

    libgomp/
            * testsuite/libgomp.c-c++-common/non-rect-loop-1.c: New test.

    (cherry picked from commit 1dc65003b66e5a97200f454eeddcccfce34416b3)

Reply via email to