This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH GCC]Fix broken Canadian when checking isl library support
- From: Richard Biener <richard dot guenther at gmail dot com>
- To: Bin Cheng <bin dot cheng at arm dot com>, Tobias Grosser <tobias at grosser dot es>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 21 Aug 2014 11:08:31 +0200
- Subject: Re: [PATCH GCC]Fix broken Canadian when checking isl library support
- Authentication-results: sourceware.org; auth=none
- References: <000801cfbd19$ec953ff0$c5bfbfd0$ at arm dot com>
On Thu, Aug 21, 2014 at 10:28 AM, Bin Cheng <bin.cheng@arm.com> wrote:
> Hi,
> Canadian build of arm/aarch64 (and other targets) toolchains are broken
> because of isl library check. There is below code in top level gcc
> configuration.
>
> { $as_echo "$as_me:${as_lineno-$LINENO}: checking for version 0.12 of
> ISL" >&5
> $as_echo_n "checking for version 0.12 of ISL... " >&6; }
> if test "$cross_compiling" = yes; then :
> gcc_cv_isl=yes
> else
> cat confdefs.h - <<_ACEOF >conftest.$ac_ext
>
> For Canadian build, corss_compiling is set to `yes', then gcc_cv_isl is set
> to `yes' accordingly, no matter whether isl library is available or not.
> We also can't set it to `no' by default, because "--with-isl=xxxx" option
> would be nullified in this way. I think the best we can do here is add
> AC_LINK_IFELSE when checking.
>
> This patch fixes the issue.
>
> Is it OK?
I think it would be better to identify a set of features we rely on that
are not present in earlier versions and make the test a link
test unconditionally.
Tobias, are there include files / types / functions we require
that are not available in earlier versions?
Unfortunately ISL upstream doesn't have ISL version defines in
version.h (and no, don't go the cloog route of auto-generating
the version.h file!). That way we still can't do any sanity check
of an in-tree version (which we have to check from the source,
before compiling it).
If you don't want to work on this patch further the present patch is ok.
Thanks,
Richard.
> 2014-08-21 Bin Cheng <bin.cheng@arm.com>
>
> * config/isl.m4 (ISL_CHECK_VERSION): Check link of isl library
> for cross_compiling.
> * configure: Regenerated.