<br><br><div class="gmail_quote">On Tue, Feb 12, 2013 at 9:06 AM, Krzysztof Parzyszek <span dir="ltr"><<a href="mailto:kparzysz@codeaurora.org" target="_blank">kparzysz@codeaurora.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On 2/11/2013 4:20 PM, Eric Christopher wrote:<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
[...] Also, I'd hope that whatever way you're iterating over the list is<div class="im"><br>
stable. I didn't think to check that, so if it isn't please make sure it<br>
is.<br>
</div></blockquote>
<br>
The StringMap iterator will give the same sequence for each map that has been filled with the same contents and the individual insertions were done in the same order.  Unless the code that inserts the names into the map changes, the results should be the same.<br>

<br>
On a somewhat different note---is there a way to test the change to llvm-dwarfdump?  I already committed it without a test, and testing it would require something like a pre-existing object file to parse.  I saw that there were other testcases that had object files associated with them---is this the usual strategy?<div class="HOEnZb">
<div class="h5"><br></div></div></blockquote><div><br></div><div>Yep.</div><div><br></div><div>-eric </div></div>