[PATCH] D57123: [MergeSets] Add infrastructure to build merge sets based on Das and Ramakrishna's paper.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 6 12:19:36 PST 2019


fhahn added a comment.

In D57123#1387411 <https://reviews.llvm.org/D57123#1387411>, @hiraditya wrote:

> Interesting, I have a merge-sets implementation in my patch: https://reviews.llvm.org/D32140
>  It will be interesting to compare the two.


Interesting! It seems the biggest difference is that the implementation here does not explicitly construct the DJ graph and uses BitVectors instead of sets. It would probably make sense to split up the mergeset part and the PromoteMem2Reg part once they are ready for committing. Let me know if you are interesting in trying the implementation for your patch, I'd be happy to split it up sooner.


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

https://reviews.llvm.org/D57123





More information about the llvm-commits mailing list