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: [RFC] Fix for PR58201


Hi Honza,

On 09/06/2013 01:05 AM, Jan Hubicka wrote:
Hi,
this is the patch I commited after testing on x86_64-linux.

Honza

Index: ChangeLog
===================================================================
*** ChangeLog	(revision 202271)
--- ChangeLog	(working copy)
***************
*** 1,3 ****
--- 1,9 ----
+ 2013-09-04  Jan Hubicka  <jh@suse.cz>
+
+ 	PR middle-end/58201
+ 	* cgraphunit.c (analyze_functions): Clear AUX fields
+ 	after processing; initialize assembler name has.
+
I checked and double checked and with this commit a C++ test regressed:

FAIL: g++.dg/template/cond2.C -std=gnu++98 (test for warnings, line 9)
FAIL: g++.dg/template/cond2.C -std=gnu++11 (test for warnings, line 9)

In practice we emit a message off by one line, line 10 instead of the correct line 9. Is it possible that you are clearing too much, so to speak?

After the recent breakages, up to date testresults are arriving only now on gcc-testresults confirming my finding, for example Andreas Schwab already posted a couple.

Thanks!
Paolo.


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