public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Thomas Schwinge <thomas@codesourcery.com>
To: Sebastian Pop <sebpop@gmail.com>,
	David Edelsohn <dje.gcc@gmail.com>,
	Sebastian Pop <s.pop@samsung.com>,
	Alan Lawrence <alan.lawrence@arm.com>
Cc: GCC Patches <gcc-patches@gcc.gnu.org>,
	Richard Biener	<richard.guenther@gmail.com>
Subject: Re: [PATCH] GCC system.h and Graphite header order
Date: Fri, 27 Nov 2015 16:40:00 -0000	[thread overview]
Message-ID: <87bnafpg0m.fsf@kepler.schwinge.homeip.net> (raw)
In-Reply-To: <CAOckXuNOxnRswh06njeTzSDwZsacXv45vojN3kEB76fTNodfoQ@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 5136 bytes --]

Hi!

On Tue, 24 Nov 2015 10:32:12 +0000, Alan Lawrence <alan.lawrence@arm.com> wrote:
> I note doc/install.texi says that gcc uses "ISL Library version 0.15,
> 0.14, 0.13, or 0.12.2". This patch breaks the build with 0.12.2 (a
> subset of errors below)

<https://gcc.gnu.org/PR68540> has been filed.  I set you guys on CC.

> but seems fine with 0.14. I haven't tested
> 0.13. Do we want to update install.texi ?

I have a slight preference to keep ISL 0.12.2 supported, but can adapt to
a newer version, if necessary.

> In file included from
> /work/alalaw01/build-aarch64-none-elf/host-tools/include/isl/list.h:13:0,
>                  from
> /work/alalaw01/build-aarch64-none-elf/host-tools/include/isl/aff_type.h:4,
>                  from
> /work/alalaw01/build-aarch64-none-elf/host-tools/include/isl/local_space.h:4,
>                  from
> /work/alalaw01/build-aarch64-none-elf/host-tools/include/isl/constraint.h:13,
>                  from /work/alalaw01/src/gcc/gcc/graphite-optimize-isl.c:41:
> /work/alalaw01/build-aarch64-none-elf/host-tools/include/isl/ctx.h:108:8:
> error: attempt to use poisoned "malloc"
>         malloc(size)))
>         ^
> /work/alalaw01/build-aarch64-none-elf/host-tools/include/isl/ctx.h:112:8:
> error: attempt to use poisoned "realloc"
>         realloc(ptr,size)))
>         ^
> /usr/include/c++/4.8/bits/locale_facets.h:2566:44: error: macro
> "isdigit" passed 2 arguments, but takes just 1
>      isdigit(_CharT __c, const locale& __loc)
>                                             ^
> /usr/include/c++/4.8/bits/locale_facets.h:2572:44: error: macro
> "ispunct" passed 2 arguments, but takes just 1
>      ispunct(_CharT __c, const locale& __loc)
>                                             ^
> /usr/include/c++/4.8/bits/locale_facets.h:2578:45: error: macro
> "isxdigit" passed 2 arguments, but takes just 1
>      isxdigit(_CharT __c, const locale& __loc)
>                                              ^
> /usr/include/c++/4.8/bits/locale_facets.h:2584:44: error: macro
> "isalnum" passed 2 arguments, but takes just 1
>      isalnum(_CharT __c, const locale& __loc)
>                                             ^
> /usr/include/c++/4.8/bits/locale_facets.h:2590:44: error: macro
> "isgraph" passed 2 arguments, but takes just 1
>      isgraph(_CharT __c, const locale& __loc)
>                                             ^
> /usr/include/c++/4.8/bits/locale_facets.h:2596:44: error: macro
> "toupper" passed 2 arguments, but takes just 1
>      toupper(_CharT __c, const locale& __loc)
>                                             ^
> /usr/include/c++/4.8/bits/locale_facets.h:2602:44: error: macro
> "tolower" passed 2 arguments, but takes just 1
>      tolower(_CharT __c, const locale& __loc)
> 
> In file included from /usr/include/c++/4.8/bits/basic_ios.h:37:0,
>                  from /usr/include/c++/4.8/ios:44,
>                  from /usr/include/c++/4.8/ostream:38,
>                  from /usr/include/c++/4.8/iostream:39,
>                  from
> /work/alalaw01/build-aarch64-none-elf/host-tools/include/isl/int.h:17,
>                  from
> /work/alalaw01/build-aarch64-none-elf/host-tools/include/isl/ctx.h:16,
>                  from
> /work/alalaw01/build-aarch64-none-elf/host-tools/include/isl/list.h:13,
>                  from
> /work/alalaw01/build-aarch64-none-elf/host-tools/include/isl/aff_type.h:4,
>                  from
> /work/alalaw01/build-aarch64-none-elf/host-tools/include/isl/local_space.h:4,
>                  from
> /work/alalaw01/build-aarch64-none-elf/host-tools/include/isl/constraint.h:13,
>                  from /work/alalaw01/src/gcc/gcc/graphite-scop-detection.c:52:
> /usr/include/c++/4.8/bits/locale_facets.h:2530:5: error:
> ‘std::isspace’ declared as an ‘inline’ variable
>      isspace(_CharT __c, const locale& __loc)
>      ^
> /usr/include/c++/4.8/bits/locale_facets.h:2530:5: error: template
> declaration of ‘bool std::isspace’
> /usr/include/c++/4.8/bits/locale_facets.h:2531:7: error: expected
> primary-expression before ‘return’
>      { return use_facet<ctype<_CharT> >(__loc).is(ctype_base::space, __c); }
>        ^
> /usr/include/c++/4.8/bits/locale_facets.h:2531:7: error: expected ‘}’
> before ‘return’
> /usr/include/c++/4.8/bits/locale_facets.h:2536:5: error: ‘isprint’
> declared as an ‘inline’ variable
>      isprint(_CharT __c, const locale& __loc)
>      ^
> /usr/include/c++/4.8/bits/locale_facets.h:2536:5: error: template
> declaration of ‘bool isprint’
> /usr/include/c++/4.8/bits/locale_facets.h:2537:7: error: expected
> primary-expression before ‘return’
>      { return use_facet<ctype<_CharT> >(__loc).is(ctype_base::print, __c); }
>        ^
> /usr/include/c++/4.8/bits/locale_facets.h:2537:7: error: expected ‘}’
> before ‘return’
> /usr/include/c++/4.8/bits/locale_facets.h:2537:75: error: expected
> declaration before ‘}’ token
>      { return use_facet<ctype<_CharT> >(__loc).is(ctype_base::print, __c); }


Grüße
 Thomas

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 472 bytes --]

  reply	other threads:[~2015-11-27 16:24 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-21 22:17 David Edelsohn
2015-11-22  7:34 ` Sebastian Pop
2015-11-23 10:28   ` Richard Biener
2015-11-24 10:39     ` Alan Lawrence
2015-11-27 16:40       ` Thomas Schwinge [this message]
2015-11-27 17:05         ` David Edelsohn
2015-11-30  8:30           ` Thomas Schwinge

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87bnafpg0m.fsf@kepler.schwinge.homeip.net \
    --to=thomas@codesourcery.com \
    --cc=alan.lawrence@arm.com \
    --cc=dje.gcc@gmail.com \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=richard.guenther@gmail.com \
    --cc=s.pop@samsung.com \
    --cc=sebpop@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).