[llvm] [GVN] Excluding dead blocks before full redundancy eliminations (PR #88556)

[Nikita Popov via llvm-commits]

nikic wrote:

Would it be possible / make sense to remove the assertion instead? As I understand it, doing a replacement with undef should be correct here, right?

Looking at the commit that introduced the assertion (https://github.com/llvm/llvm-project/commit/3168ab337635b6ed5a279cd432f95dc3da17fb2c) I don't think think that the fix being applied here is right. It sounds like maybe DeadBlocks is out of sync?

https://github.com/llvm/llvm-project/pull/88556