<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Apr 30, 2013 at 5:44 PM, Howard Hinnant <span dir="ltr"><<a href="mailto:hhinnant@apple.com" target="_blank">hhinnant@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Author: hhinnant<br>
Date: Tue Apr 30 16:44:48 2013<br>
New Revision: 180811<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=180811&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=180811&view=rev</a><br>
Log:<br>
This patch introduces an alternative layout for basic_string which when the string is short, the data pointer will be word-aligned.  It can be activated with -D_LIBCPP_ALTERNATE_STRING_LAYOUT.  These two different layouts (the default and _LIBCPP_ALTERNATE_STRING_LAYOUT) are not ABI compatible with each other.  Once one is chosen for a given platform, it is disruptive to change it.<br>

<br></blockquote><div><br></div><div><div><br class="">Maybe the name of the #define name should contain "ABI" in it somewhere just to make it extra clear that it tampers with ABI? It seems a lot less likely for someone to slip up a flag called _LIBCPP_ALTERNATE_STRING_LAYOUT_ABI. There aren't many ways to interpret "layout" other than being ABI breaking, but still, having an easy mental pattern-match on "ABI" seems a lot less error-prone (the fact that you explicitly had to mention that this was an ABI choice in the commit message suggests that it's not entirely obvious).</div>
<div><br></div><div>Also, "alternate" is really vague (and what will you call the next layout change, "extra alternate"?). Why not _LIBCPP_WORD_ALIGNED_STRING_LAYOUT_ABI, which is self-documenting?</div>
<div><br></div><div>-- Sean Silva</div></div><div> </div></div></div></div>