Also the layout builder never gets the dense map.  So Microsoft Layout Builder doesn't know anything about the dense map.  It's ClangASTSource that iterates over the map, and all the layout builder sees is fields being added in the wrong order<br><div class="gmail_quote">On Tue, Mar 24, 2015 at 10:24 AM Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This bug isn't specific to Microsoft layout. Clang as a general rule<br>
requires fields to be added in offset order, and iterating over a dense map<br>
doesn't give you that.<br>
<br>
I'll let majnemer or rnk comment on the ExternalASTSource interface<br>
<br>
<br>
REPOSITORY<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D8512" target="_blank">http://reviews.llvm.org/D8512</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/<u></u>settings/panel/<u></u>emailpreferences/</a><br>
<br>
<br>
</blockquote></div>