[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