[PATCH] D38097: [IVUsers] Changes to make IVUsers's results robust to instruction and uselist ordering

[Hal Finkel via Phabricator via llvm-commits]

hfinkel added a comment.

In https://reviews.llvm.org/D38097#876817, @dneilson wrote:

> In https://reviews.llvm.org/D38097#876806, @hfinkel wrote:
>
> > Any performance changes in the test suite?
>
>
> I don't have access to SPEC source, so I cannot test those. I'm unfamiliar with the performance test suite, so I don't know what all else is there. I'm going to set up a run against our internal (Java-based) performance suite to see where that stands.


Great.

You can run LLVM's test suite using LNT by following the directions here: http://llvm.org/docs/lnt/quickstart.html

> Functionally, this patch doesn't regress anything in 'ninja check-all'

Sounds good (although I have no idea how good our coverage is in this regard).


https://reviews.llvm.org/D38097