public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH] Fix PR55961, do not include gmp.h for building host tools
@ 2013-01-15 13:16 Richard Biener
  2013-01-15 15:00 ` Ian Lance Taylor
  0 siblings, 1 reply; 2+ messages in thread
From: Richard Biener @ 2013-01-15 13:16 UTC (permalink / raw)
  To: gcc-patches


This makes sure to not include gmp.h for building generator programs
on the host (and thus require gmp.h on the host).

Configury might need updating for the use of GENERATOR_FILE
from system.h as well, but the patch below is a strict improvement
over the current situation, restoring previous behavior.

Bootstrapped on x86_64-unknown-linux-gnu.

Ok?

Thanks,
Richard.

2013-01-15  Richard Biener  <rguenther@suse.de>

	PR bootstrap/55961
	* system.h: Do not include gmp.h for building host tools.

Index: gcc/system.h
===================================================================
*** gcc/system.h	(revision 195194)
--- gcc/system.h	(working copy)
*************** extern int vsnprintf(char *, size_t, con
*** 636,642 ****
--- 636,645 ----
  #include <dlfcn.h>
  #endif
  
+ /* Do not introduce a gmp.h dependency on the build system.  */
+ #ifndef GENERATOR_FILE
  #include <gmp.h>
+ #endif
  
  /* Get libiberty declarations.  */
  #include "libiberty.h"

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] Fix PR55961, do not include gmp.h for building host tools
  2013-01-15 13:16 [PATCH] Fix PR55961, do not include gmp.h for building host tools Richard Biener
@ 2013-01-15 15:00 ` Ian Lance Taylor
  0 siblings, 0 replies; 2+ messages in thread
From: Ian Lance Taylor @ 2013-01-15 15:00 UTC (permalink / raw)
  To: Richard Biener; +Cc: gcc-patches

On Tue, Jan 15, 2013 at 5:16 AM, Richard Biener <rguenther@suse.de> wrote:

> 2013-01-15  Richard Biener  <rguenther@suse.de>
>
>         PR bootstrap/55961
>         * system.h: Do not include gmp.h for building host tools.

Looks OK to me.

Ian

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2013-01-15 15:00 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-01-15 13:16 [PATCH] Fix PR55961, do not include gmp.h for building host tools Richard Biener
2013-01-15 15:00 ` Ian Lance Taylor

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).