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

Fri Jan 22 07:21:37 PST 2021

dmajor added a comment.

In D93764#2515449 <https://reviews.llvm.org/D93764#2515449>, @fhahn wrote:

> One issue with the file was that the patch did not account for atomic loads and would add additional atomic loads. I fixed that in 86991d323133 <https://reviews.llvm.org/rG86991d3231334538cccf6732056cbb641046bd49>. Please let me know if that fixes your issue. Otherwise I'll revert the patch to investigate further.

Ah, that sounds promising! On our side we haven't got down to specific details yet but it _may_ be related to atomics. I've kicked off new tests on 86991d323133 <https://reviews.llvm.org/rG86991d3231334538cccf6732056cbb641046bd49>, should have results in 2-3 hours.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93764