[PATCH] D152772: [LoopSink] Allow sinking to PHI-use

Wed Jun 21 08:38:22 PDT 2023

alexfh added a comment.

In D152772#4437840 <https://reviews.llvm.org/D152772#4437840>, @alexfh wrote:

> We're still trying to get a reproducer, but it's somewhat complicated given that the issue happens in ThinLTO builds. The problem manifests as:
>
>   Instruction does not dominate all uses!
>     %1480 = getelementptr inbounds %"class.absl::AlphaNum", ptr %24, i64 0, i32 1
>     %1478 = phi ptr [ %1476, %1474 ], [ %1480, %1472 ]
>   fatal error: error in backend: Broken module found, compilation aborted!
>
> Please revert while we're preparing a test case!

I reverted the commit in c96c85aba28089ec840a867e289782831e9637ee <https://reviews.llvm.org/rGc96c85aba28089ec840a867e289782831e9637ee>. We'll post a reduced test case as soon as we have one.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152772