[PATCH] D71539: [SCEV] Look through trivial PHIs.

Wed Apr 20 09:20:06 PDT 2022

bmahjour added a comment.

In D71539#3461039 <https://reviews.llvm.org/D71539#3461039>, @fhahn wrote:

> The last known crash has been fixed by now and I plan to re-commit this soon.
>
> AFAIK the only remaining issue is with DependenceAnalysis. @bmahjour do you know if there has been any progress on the fixes you shared a while ago?

@fhahn I'm still awaiting feedback on D110972 <https://reviews.llvm.org/D110972> and D110973 <https://reviews.llvm.org/D110973>. Honestly neither of them are good solutions but they should be useful in understanding the problem and getting some conversation started on how to proceed. I do think that the damage to DA would easily outweigh the benefit of this change, so I'd strongly suggest we hold off on recommitting this until a solution for the DA problem is found. This would be a good topic to discuss further in future #loopoptwg <https://reviews.llvm.org/tag/loopoptwg/> calls if need be.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71539