[PATCH] D65222: [IPSCCP] Add assertion to surface cases where we zap returns with overdefined users.

[Peter Smith via Phabricator via llvm-commits]

peter.smith added a comment.

In D65222#1602697 <https://reviews.llvm.org/D65222#1602697>, @fhahn wrote:

> In D65222#1602666 <https://reviews.llvm.org/D65222#1602666>, @peter.smith wrote:
>
> > A couple of our CI jobs have found this assertion triggering. The first is when compiling the linux kernel using allmodconfig. The second is on this reduced example using LTO:
>
>
> Thanks, that's interesting, I'll take a look. This is also triggering a different assertion in this job: https://ci.linaro.org/job/tcwg_kernel-build-llvm-master-aarch64-mainline-allmodconfig/401/ (@nickdesaulniers made me aware) and I think I should have a reproducer for that issue as well shortly. I should know what's going on on Monday the latest. If it is blocking your jobs, should we revert it in the meantime?


Thanks for taking a look. From a Linaro perspective there isn't anything that depends on this that needs to run over the weekend so there is no immediate need to revert if you are going to look at it next week. Nick might have more urgent jobs that are affected though so I'm happy to let his opinion take precedence.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D65222