Hi Yuao,

Yuao Ma wrote:
Following up on your review comments, I have updated the patch.

Thanks - LGTM.

Two minor comments, but I have already pushed the commit
as r16-602-gb239e9cf98ca92

First:

        * gfortran.dg/dec_math.f90: Add atand(y, x) testcase.

Also for the documentation, the (...) syntax is used for
the sections like "(atand):". Admittedly, the full section
title or @node does not always work.
Also it is more than adding a testcase, you also did a
cleanup.

Secondly, I think it made sense to update the wording at
the top of the testcase. In principle having a testcase
with -std=f2023 makes sense as that checks that the -std=
handling works correctly.

I have now pushed a wording commit as follow up to make
it clear to a testcase reader that everything but
cotan/cotand is now included in F2023:

r16-603-gabe8cd9ba6bf85 gfortran.dg/dec_math.f90: Add comment regarding F2023 [PR113413] And I have now closed the PR. Thanks again for the patch! Tobias

Reply via email to