tdlib / td

Cross-platform library for building Telegram clients
https://core.telegram.org/tdlib
Boost Software License 1.0
7.11k stars 1.44k forks source link

Remove -fno-rtti #2631

Closed ivanstepanovftw closed 1 year ago

ivanstepanovftw commented 1 year ago

-fno-rtti does not allow to build td with address sanitizers

levlam commented 1 year ago

fno-rtti reduces executable size and removes unneeded code generation. It is harmful to remove the compiler option. Moreover, the option doesn't affect address sanitizer in any way.

RTTI can be used by other sanitizers, for example, vptr sanitizer, which is part of undefined sanitizer, but you can disable it with -fno-sanitize=vptr when using -fsanitize=undefined.