[PATCH] Add similar function merging to MergeFunctions

Tobias von Koch tobias.von.koch at gmail.com
Sat Apr 5 17:27:02 PDT 2014


  Oh right - didn't realise commits wasn't on there! Also CCing Nick.

  There has been some work on MergeFunctions recently by Stepan so it probably needs to be rebased.

  Stepan - are you still working on this? If I recall correctly, you were considering implementing similar function merging on top of your ordering-based approach. I currently don't have the resources to completely rewrite the patch. If we want to support similar function merging right now, I see two options:

  a) try and add this as-is to MergeFunctions but this would likely mean we can't do your ordering-based merging,
  b) split MergeFunctions into two passes, one for merging identical functions (it could use your faster algorithm) and another just for similar function merging (based on parts of this patch).

  It would be nice to get this into trunk, because the results are good and it is in active use by customers. We also have an upcoming paper at LCTES with a more thorough evaluation.

http://reviews.llvm.org/D2591






More information about the llvm-commits mailing list