This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch] fix bootstrap on FreeBSD i386/arm
- From: Steve Ellcey <sellcey at imgtec dot com>
- To: Andreas Tobler <andreast-list at fgznet dot ch>
- Cc: Eric Botcazou <ebotcazou at adacore dot com>, <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 1 Jun 2015 08:20:31 -0700
- Subject: Re: [patch] fix bootstrap on FreeBSD i386/arm
- Authentication-results: sourceware.org; auth=none
- References: <6154248 dot kZpWmyJ86f at polaris> <1432919888 dot 20199 dot 42 dot camel at ubuntu-sellcey> <5569828A dot 9080304 at fgznet dot ch> <1433169813 dot 20199 dot 51 dot camel at ubuntu-sellcey> <556C753F dot 1050502 at fgznet dot ch>
- Reply-to: <sellcey at imgtec dot com>
On Mon, 2015-06-01 at 17:07 +0200, Andreas Tobler wrote:
> > This patch worked for me. I am curious about why you use
> > '!HAVE_DECL_ATOLL' in read-rtl.c instead of '!defined(HAVE_DECL_ATOLL)'.
> > The use of !defined seems more common and is used in the HAVE_ATOQ check
> > in the same #if statement.
>
> Because HAVE_DECL_ATOLL is either defined as 0 or 1. So it is always
> defined. HAVE_ATOQ on the other side is only defined if it is available.
>
> From config.in:
> ----
> /* Define to 1 if we found a declaration for 'atoll', otherwise define to 0.
> */
> #ifndef USED_FOR_TARGET
> #undef HAVE_DECL_ATOLL
> #endif
> ----
>
> Andreas
Ah, I looked at the code but didn't read the comment. When I look at
auto-host.h I see that the header does what the comment says it does.
Thanks for the explanation.
Steve Ellcey
sellcey@imgtec.com