On Thu, 27 Mar 2025 09:33:40 +0000
Iain Sandoe <iains....@gmail.com> wrote:

> As noted in the commit log, the macOS version of cmath (at least) has
> conflicts with parse.y.  Tested on x86_64,aarch64 linux,
> x86_64-darwin. OK for trunk?
> thanks,
> Iain

LGTM, Iain.  Any header file that isn't needed, isn't needed.  

If in the future we decide to use <cmath>, we'll rename any conflicting
names in parse.y.  

--jkl


> --- 8< ---
> 
> Several of enumerators in parse.y conflict with ones declared in at
> least some versions of <cmath> .. e.g. "OVERFLOW".  The header is no
> longer needed since the FE is not trying to do host arithmetic.
> 
> gcc/cobol/ChangeLog:
> 
>       * cobol-system.h: Remove <cmath>.
> 
> Signed-off-by: Iain Sandoe <i...@sandoe.co.uk>
> ---
>  gcc/cobol/cobol-system.h | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/gcc/cobol/cobol-system.h b/gcc/cobol/cobol-system.h
> index 81529bd3a67..ff9583530e2 100644
> --- a/gcc/cobol/cobol-system.h
> +++ b/gcc/cobol/cobol-system.h
> @@ -53,7 +53,6 @@
>  #include <deque>
>  #include <numeric>
>  #include <limits>
> -#include <cmath>
>  
>  #include <unordered_map>
>  #include <unordered_set>
> -- 
> 2.39.2 (Apple Git-143)
> 

Reply via email to