This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [patch] Enable the use of a resolution file

From: Cary Coutant <ccoutant at google dot com>
To: Rafael Espindola <espindola at google dot com>
Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Diego Novillo <dnovillo at google dot com>
Date: Tue, 10 Nov 2009 14:27:59 -0800
Subject: Re: [patch] Enable the use of a resolution file
References: <38a0d8450911100939w5dc23fbam13a4ea2c16ef5a37@mail.gmail.com> <38a0d8450911101158m7fb893d7pfc12ba6d93d797e1@mail.gmail.com>

@@ -334,7 +331,7 @@ write_resolution (void)
+	  fprintf (f, "%d %s %s \n", slot, lto_resolution_str[resolution],
syms[j].name);

Is the space before \n intentional?


@@ -468,6 +465,9 @@ all_symbols_read_handler (void)
+  resolution_file = make_temp_file ("");
+  printf("resolution: %s\n", resolution_file);

Debugging printf?

Aside from that, the lto-plugin part of the patch looks good.

-cary

Follow-Ups:
- Re: [patch] Enable the use of a resolution file
  - From: Rafael Espindola

References:
- [patch] Enable the use of a resolution file
  - From: Rafael Espindola
- Re: [patch] Enable the use of a resolution file
  - From: Rafael Espindola

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]