[PATCH] D21681: [ELF] - Implemented support of default/non-default symbols versions

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 24 04:37:47 PDT 2016


grimar added a comment.

In http://reviews.llvm.org/D21681#466350, @ruiu wrote:

> This may be an expensive feature because it works for each symbol. Can you run benchmarks with this change?


Sure. But I think it should not be expencive. For symbols that does not have non-default versions nothing changed.
I think there are no that many symbols that are affected by this change. Do you have suggestions for what to use for benchmark ?


http://reviews.llvm.org/D21681





More information about the llvm-commits mailing list