[PATCH] D93764: [LoopUnswitch] Implement first version of partial unswitching.

Fri Jan 22 03:08:47 PST 2021

fhahn added a comment.

In D93764#2514037 <https://reviews.llvm.org/D93764#2514037>, @dmajor wrote:

> To give you an early heads up, we're seeing crashes in Firefox after this commit. Unfortunately it's only happening on Mac, which I am not equipped to debug. I may need to try enlisting a colleague to investigate and/or throw more test suites at it and hope we can capture something on Linux or Windows.
>
> Since there's a release branch point coming soon, if we don't get anywhere by the end of the week, what do you think about reverting until after 12.0.0-rc1 is tagged?

In D93764#2514813 <https://reviews.llvm.org/D93764#2514813>, @mstorsjo wrote:

> I'm also seeing lots of hangs/infinite loops in code built after this commit. I don't have anything nicely reduced to dig in on (yet), but it broke a significant amount of my testing setup.

Thanks for the heads up! It would be great to get a reproducer, but I'll revert by the end of day if the issue is not resolved by then.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93764/new/

https://reviews.llvm.org/D93764