
<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class="">In r248292, the added unit test fails without the changes.</div><br class=""><div><blockquote type="cite" class=""><div class="">On Sep 18, 2015, at 5:49 PM, Manuel Klimek via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org" class="">cfe-commits@lists.llvm.org</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class="">LG in general; I think if we like the order to be deterministic we should try to come up with a unit test so nobody regresses this in the future.</div><br class=""><div class="gmail_quote"><div dir="ltr" class="">On Fri, Sep 18, 2015 at 4:44 PM Argyrios Kyrtzidis <<a href="mailto:kyrtzidis@apple.com" class="">kyrtzidis@apple.com</a>> wrote:<br class=""></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br class="">

<br class="">

I have found it useful for the getAllCompileCommands() to return the commands in the order that they were provided in the JSON file.<br class="">

This is useful for debugging of issues and reduction of test cases.<br class="">

For example, an issue may show up due to the order that some commands were processed. It is convenient to be able to remove commands from the file and still preserve the order that they are returned, instead of getting a completely different order when removing a few commands.<br class="">

<br class="">

Let me know what you think!<br class="">

<br class="">

</blockquote></div>

_______________________________________________<br class="">cfe-commits mailing list<br class=""><a href="mailto:cfe-commits@lists.llvm.org" class="">cfe-commits@lists.llvm.org</a><br class="">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits<br class=""></div></blockquote></div><br class=""></body></html>