[PATCH] D124951: [SelectionDAG] Teach getNode to fold (max/min X, X) -> X.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 4 12:37:43 PDT 2022


craig.topper added a comment.

In D124951#3491982 <https://reviews.llvm.org/D124951#3491982>, @RKSimon wrote:

> Don't we need this in DAGCombine as well to handle cases where the same arg appears after another fold?

Yes. I could also just move this to DAGCombine only and not affect pr55271.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124951



More information about the llvm-commits mailing list