PATCH to c-pretty-print handling of VECTOR_TYPE

Gabriel Dos Reis gdr@cs.tamu.edu
Tue Mar 2 20:07:00 GMT 2010


Jason Merrill <jason@redhat.com> writes:

| A patch I'm working on for c++/41959 will give an error in the
| compiler rather than the assembler when we see a mangled name
| collision, but the error was a bit confusing:
| 
| error: declaration of 'void f(float __vector__)'
| error: conflicts with previous declaration 'void f(float __vector__)'
| 
| when in fact the two argument types have different vector sizes.  So
| this patch changes them to "float __vector[4]" and "float
| __vector[8]".
| 
| OK for 4.5 or 4.6?

This yields a far better diagnostic than what we currently have; so,
yes OK everywhere.  Thanks!

-- Gaby



More information about the Gcc-patches mailing list