This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [build] Move ENABLE_EXECUTE_STACK to toplevel libgcc
- From: "Joseph S. Myers" <joseph at codesourcery dot com>
- To: Paolo Bonzini <bonzini at gnu dot org>
- Cc: Rainer Orth <ro at CeBiTec dot Uni-Bielefeld dot DE>, gcc-patches at gcc dot gnu dot org, Ralf Wildenhues <Ralf dot Wildenhues at gmx dot de>, Mike Stump <mikestump at comcast dot net>, "Loren J. Rittle" <ljrittle at acm dot org>, Kai Tietz <ktietz at redhat dot com>, Dave Korn <dave dot korn dot cygwin at gmail dot com>, Jason Thorpe <thorpej at netbsd dot org>, Krister Walfridsson <krister dot walfridsson at gmail dot com>, Uros Bizjak <ubizjak at gmail dot com>, Richard Henderson <rth at redhat dot com>, Eric Botcazou <ebotcazou at adacore dot com>
- Date: Tue, 7 Jun 2011 13:59:40 +0000 (UTC)
- Subject: Re: [build] Move ENABLE_EXECUTE_STACK to toplevel libgcc
- References: <yddipssrxgx.fsf@manam.CeBiTec.Uni-Bielefeld.DE> <4DE4AEC2.3030502@gnu.org> <yddwrh6rifi.fsf@manam.CeBiTec.Uni-Bielefeld.DE> <4DE50FFB.90501@gnu.org>
On Tue, 31 May 2011, Paolo Bonzini wrote:
> > I'd thought about it, but refrained since HAVE_ENABLE_EXECUTE_STACK
> > affects only three cpus. Currently, our documentation of libgcc
> > configury and macros used is close to non-existant. That's probably for
> > someone who has implemented this stuff.
>
> True, OTOH HAVE_ENABLE_EXECUTE_STACK is a target macro, and those are well
> documented. Just say that it has to be defined if libgcc provides a
> non-trivial implementation of __enable_execute_stack; it doesn't need to delve
> into how to hack libgcc.
As I understand it, HAVE_ENABLE_EXECUTE_STACK is only used in code under
gcc/config/. That is, it is not a target macro as usually understood but
is logically private to a few back ends (and it would be a bug to
introduce uses of it elsewhere, just like it was a bug to introduce uses
of TARGET_64BIT outside gcc/config/ when that macro also is logically
private).
--
Joseph S. Myers
joseph@codesourcery.com