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

[Jay Foad via Phabricator via llvm-commits]

foad added inline comments.


================
Comment at: llvm/test/CodeGen/AMDGPU/vector_shuffle.packed.ll:1215
+; GFX9-NEXT:    v_lshl_or_b32 v1, v1, 16, v3
 ; GFX9-NEXT:    s_setpc_b64 s[30:31]
 ;
----------------
deadalnix wrote:
> Are you familiar with that backend? If so could you make a patch for this? The best way we get through this is by looking at all cases one by one and nailing them.
D127385, but I don't know how to add tests for it. It doesn't affect any current lit tests, and I don't have much time at the moment to try constructing new tests that it might affect.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127115