[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/v8i1-masks.ll:154
+; X32-AVX2-NEXT:    vpbroadcastd {{.*#+}} ymm1 = [1,1,1,1,1,1,1,1]
+; X32-AVX2-NEXT:    vpand %ymm1, %ymm0, %ymm0
 ; X32-AVX2-NEXT:    retl
----------------
RKSimon wrote:
> deadalnix wrote:
> > deadalnix wrote:
> > > This is new :(
> > @RKSimon I was able to reduce this to D129641 . This regression didn't exist before D129641 , but it does now.
> Sorry I should have said - yes this is caused by more yak shaving. I have a large number of other patches in flight that should help, but I'll add this one to the list.
Maybe I can help reviewing some?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127115