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

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 1 08:03:52 PDT 2023


foad added a comment.

> Does anyone else have any thoughts on this? @foad @nikic @craig.topper ?

Not really :/ For my use cases (graphics shaders on AMDGPU) I am not too concerned, but to be safe we could run some fairly extensive performance tests. It would probably take about a week to get results from that.


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