[PATCH] D16124: Add to the split module utility an SCC based method which allows not to globalize any local variables.

Sergei Larin via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 13 10:54:59 PST 2016


slarin added a comment.

Peter,

  Here is one issue with EquivalenceClasses. It uses 

std::set<ECValue> TheMapping;

for members. We purposefully used SetVector since we are dealing with pointers... As a result I have non deterministic mapping - which is not cool.

I will see if I can do some sorting prior to partition assignment, but if it fails I do not think this will work.

If you have better suggestions - please speak up.

Sergei


http://reviews.llvm.org/D16124





More information about the llvm-commits mailing list