[PATCH] D97837: [InstCombine] Remove gc.relocate duplicates

Serguei Katkov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 9 01:01:33 PST 2021


skatkov added a comment.

> I think the solution with EarlyCSE supporting elimination of dup phi nodes seems more interesting...

My prototype to remove dup ph nodes does not work as a single pass due to BasicBlock processing order - not all predecessors are handled before processing phi node.
So EarlyCSE is a dead way without changing basic block processing order.


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

https://reviews.llvm.org/D97837



More information about the llvm-commits mailing list