Let me ping Krzysztof and Sumanth for a review of the latest revision. This is ready to land as far as I am concerned.

Fixed run line in testcase

Formatted using clang-format.

Thanks @kparzysz - that fixed it in my local repo as well!

In rG0ee54cf88329c50f25872ac1c67d7ae60ee3154c#947183, @kparzysz wrote:

Should be fixed in https://reviews.llvm.org/rG5f4abb7fab1c.

Thanks @srj . I have been able to reproduce this internall on predicated_store_load.cpp. Ankit is looking into this.

@kparzysz has posted an alternative here https://reviews.llvm.org/D68666

While this change is valid in terms of functionality, I think Krzysztof has a point here (http://lists.llvm.org/pipermail/llvm-dev/2019-October/135709.html). Simply computing the live interval for a virtual register should not be changing code. That concern should be handled separately.

There is a bug in HexagonExpandCondsets being exposed by this (valid) change. We are working on a fix as we speak.

Update: This change is affecting the HexagonExpandCondsets pass. I hope to have more updates soon.