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

Nirav Dave via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 11 12:54:25 PDT 2017


niravd added a comment.

D'oh. It possible though hard. I'll add a cheap check and swap.

In https://reviews.llvm.org/D36581#839344, @efriedma wrote:

> Okay, that makes sense.
>
> Is it possible for N0 to be an ancestor of N1?  If it is, do we need to replace N1 before N0?





https://reviews.llvm.org/D36581





More information about the llvm-commits mailing list