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

[Amaury SECHET via Phabricator via llvm-commits]

deadalnix added inline comments.


================
Comment at: llvm/test/CodeGen/X86/avx512bwvl-intrinsics-upgrade.ll:4850
+; X64-NEXT:    kunpckdq %k0, %k0, %k0 # encoding: [0xc4,0xe1,0xfc,0x4b,0xc0]
+; X64-NEXT:    kmovq %k0, %r8 # encoding: [0xc4,0x61,0xfb,0x93,0xc0]
 ; X64-NEXT:    vpcmpgtb %ymm0, %ymm1, %k0 # encoding: [0x62,0xf1,0x75,0x28,0x64,0xc0]
----------------
RKSimon wrote:
> Some of these are fixed in rG91adbc3208d0ce42550e42237b8d97fb866165a1
There are not just fixed now, they are improved.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127115