[PATCH] D127115: [RFC][DAGCombine] Make sure combined nodes are added back to the worklist in topological order.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jul 23 13:49:53 PDT 2022


RKSimon added a comment.

In D127115#3674112 <https://reviews.llvm.org/D127115#3674112>, @deadalnix wrote:

> Looking at rG2421a5af72e7da7c29fd153dcee0981deaaeba64 <https://reviews.llvm.org/rG2421a5af72e7da7c29fd153dcee0981deaaeba64> , isn't it something we'd want the Combiner to do rather than the legalizer?

Please have a go if you're interested - I've wasted ages trying to get it to work in a better way.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127115



More information about the llvm-commits mailing list