Hi Thomas, This looks very good. OK for trunk.
Thanks Paul On 2 January 2018 at 15:53, Thomas Koenig <tkoe...@netcologne.de> wrote: > Hello world, > > the attached patch implements simplification for cshift completely. > It also fixes a bug where compile-time simplification was handled > incorrectly for a negative value. For PR 83650 (the wrong > simplification) for the other branches I suspect a quite simple fix will > be possible, which I will address separately. > > Regression-tested. OK for trunk? > > Regards > > Thomas > > 2018-01-02 Thomas Koenig <tkoe...@gcc.gnu.org> > > PR fortran/45689 > PR fortran/83650 > * simplify.c (gfc_simplify_cshift): Re-implement to allow full > range of arguments. > > 2018-01-02 Thomas Koenig <tkoe...@gcc.gnu.org> > > PR fortran/45689 > PR fortran/83650 > * gfortran.dg/simplify_cshift_1.f90: Correct erroneous case. > * gfortran.dg/simplify_cshift_4.f90: New test. > -- "If you can't explain it simply, you don't understand it well enough" - Albert Einstein