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

Amaury SECHET via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 7 15:50:00 PDT 2023


deadalnix added a comment.

@RKSimon You did well to take this over.



================
Comment at: llvm/test/CodeGen/X86/add-and-not.ll:305
+; X64-NEXT:    addq %rdi, %rax
 ; X64-NEXT:    retq
   %old = ptrtoint ptr %a to i64
----------------
I was able to repro this one standalone in D147821 .

For some reason, this fix work on 32 bits, but there is still a missing piece for 64bits.


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