[PATCH] D52896: MergeSimilarFunctions: a code size pass to merge functions with small differences

[JF Bastien via Phabricator via llvm-commits]

jfb added a comment.

I don't understand: why is it desirable to have this as a separate thing compared to merge funcs? It seems better to use the same code for both, and when calling mergefuncs pass in a mode which chooses to merge exact matches, or similar matches.

Do you have up-to-date compile-time numbers (at O0 and https://reviews.llvm.org/owners/package/2/), as well as size numbers?


https://reviews.llvm.org/D52896