<div dir="ltr">I talked to some of the clang guys here and they said to go back to DenseMaps to conform to the ExternalASTSource interface, but make sure you iterate over fields and bases in offset order, which means sorting the list before doing the insertion.  So I will submit a patch to do that today.<br></div><br><div class="gmail_quote">On Tue, Mar 24, 2015 at 10:39 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">I see what you mean about the interface. I guess that settles that then,<br>
the interface needs to be consistent. I'll add override keywords so the<br>
compiler can catch that for me in the future.<br>
<br>
But our implementation still needs to add fields in the correct order, so<br>
iterating over the DenseMap is still not the right approach<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>