Hi, Is this alright to commit?
Thanks Sri On Thu, Feb 7, 2013 at 4:10 PM, Sriraman Tallam <tmsri...@google.com> wrote: > On Thu, Feb 7, 2013 at 6:29 AM, Jason Merrill <ja...@redhat.com> wrote: >> On 02/06/2013 08:39 PM, Sriraman Tallam wrote: >>> >>> +// Test to check if an error is generated when virtual functions >>> +// are multiversioned. >> >> >> This seems like a TODO, rather than something to test for. I don't see any >> reason why we couldn't support multiversioned virtual functions. >> >>> error_at (DECL_SOURCE_LOCATION (decl), >>> - "Virtual function versioning not supported\n"); >>> + "Virtual function multiversioning not supported\n"); >> >> >> And so this should be a sorry rather than an error. > > Attached a new patch with these changes. Also made more minor changes > to config/i386/i386.c. > > Thanks > Sri > >> >> Jason >>