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