[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?


More information about the llvm-commits mailing list