> > > I've been investigating a testsuite failure which shows up with the new > TER implementation, and it appears to be a bug in the code for > expand_builtin_memcpy. > > Does that seem reasonable? or would everyone prefer I get it fixed > before checking in the TER code?
I would prefer you don't expose latent bugs and fix them before applying the patch. If we go by this, I would have applied my fold-const.c patch to convert *(othertype*)&type to VIEW_CONVERT_EXPR a long time ago but it exposes problems with stuff like ((structtype*)&b)->c = e; if b is a scalar. Thanks, Andrew Pinski