<div>On Fri, Jan 11, 2013 at 7:05 PM, Lang Hames <span dir="ltr"><<a href="mailto:lhames@gmail.com" target="_blank">lhames@gmail.com</a>></span> wrote:</div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">At present, if a class contains any Non-POD members we perform element-wise copies for each field when generating the implicit copy-constructor and implicit copy-assignment operator (with an optimization for array members).<div>

<br></div><div>This patch changes this behavior - adjacent POD members will be memcpy'd, with Non-POD members output as usual.</div><div><br></div><div>This is an initial implementation that I'd like to get in tree. Obvious deficiencies are:</div>

<div><br></div><div>It punts entirely when LangOpts.getGC != None.</div><div>It doesn't handle inner classes/unions.</div><div>It doesn't attach any TBAA info, even when all members are the same type.</div>
<div>It isn't particularly smart about when it falls back on element-wise copies (at the moment it emits a memcpy any time it finds more than one POD field).</div><div><br></div><div>These should all be easy to test and remedy once the code's in tree though.</div>

<div><br></div><div>Does anybody see any problems with this?</div><div>Style comments?</div><div><br></div><div>Feedback much appreciated.</div></div></blockquote><div><br></div><div>This generally seems reasonable to me, but I've not looked at the patch in detail. Any reason not to apply this to move assignment operators too?</div>
<div><br></div><div>Do you handle bitfields? See r151963 for an example of that.</div></div>