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: PR c++/35441 [4.2/4.3/4.4 regression] Completely broken diagnostics


On Fri, 7 Nov 2008, Manuel López-Ibáñez wrote:

> Bootstrapped and regression tested on x86_64-unknown-linux-gnu with
> --enable-languages=all,obj-c++,ada
> 
> OK for trunk?
> 
> 2008-11-07  Manuel Lopez-Ibanez  <manu@gcc.gnu.org>
> 
> 	PR c++/35441
> 	* c-typeck.c (build_function_call): Do not print arbitrary
> 	expressions.
> testsuite/
> 	* gcc.dg/pr35441.C: New.

Is this supposed to be C or C++?  You're adding a .C testcase (C++?) but 
adding it to a C tests directory and changing a C front end file.

And since this is reducing diagnostic quality for cases where this did 
previously work, you need another (lower-priority) regression PR for the 
loss of diagnostic information.  (The long-term plan for fixing that being 
accurate start and end locations so the expression in question can be 
found again in the source text.)

-- 
Joseph S. Myers
joseph@codesourcery.com

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