[PATCH] D36581: [DAG] Fix Node Replacement in PromoteIntBinOp

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 15 08:16:13 PDT 2017


RKSimon added a comment.

In https://reviews.llvm.org/D36581#839584, @dbabokin wrote:

> In https://reviews.llvm.org/D36581#838767, @niravd wrote:
>
> >   I also have a patch that more aggressively causes invalidation-based assertions where latent bugs may lie; I think it'll catch the lion share of the latent bugs.
>
>
> Let me know if you need to run assertion patch through random testing (which finds all this cases I'm filing as bugs).


Similarly, is there anything that we could add to EXPENSIVE_CHECKS builds to help identify these?


https://reviews.llvm.org/D36581





More information about the llvm-commits mailing list