<br><br><div class="gmail_quote">On Tue, Feb 12, 2013 at 9:13 AM, Eric Christopher <span dir="ltr"><<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br><br><div class="gmail_quote"><div class="im">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>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><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>

<div><br></div></div></blockquote><div><br></div></div><div>Yep.</div><span class="HOEnZb"><font color="#888888"><div><br></div></font></span></div></blockquote><div><br></div><div>To elaborate this object file should match the end-to-end testcase that you add for the emission that you also test.</div>
<div><br></div><div>-eric </div></div>