[Chapel Merge] Adjust grepstdchdrs and remove compiler/include/ch

Notifications Commits
1

Branch: refs/heads/master
Revision: d690e1b
Author: mppf
Log Message:

Merge pull request #17590 from mppf/adjust-grepstdchdrs

Adjust grepstdchdrs and remove compiler/include/chplmath.h

This PR makes several adjustments related to grepstdchdrs (which is used
to check for includes of the C name for a header with a different name in
C++ code in the compiler).

  • in grepstdchdrs, breaks the command in up among multiple lines to make
    diffs easier to understand
  • since the upcoming PR #17586 will use sys_basic.h in compiler code,
    adjust sys_basic.h to use the C++ names if compiling in C++ and then
    adjust grepstdchdrs to ignore the C names in this file
  • Adjusts grepstdchdrs to check for math.h and remove a workaround we had
    for math headers that is no longer necessary now that C++11 is required
    (see PR #12427). Removing the workaround consists of:
    • removing chplmath.h and replacing uses of it with include <cmath>
    • replacing calls like chpl_signbit with std::signbit
  • [x] full local testing

Reviewed by @lydia-duncan and @Maxrimus - thanks!

Modified Files:
R compiler/include/chplmath.h

M compiler/codegen/codegen.cpp
M compiler/ifa/num.cpp
M runtime/include/sys_basic.h
M util/devel/grepstdchdrs

Compare: https://github.com/chapel-lang/chapel/compare/1125df1d2b09...d690e1b53738