On Thu, Nov 28, 2013 at 03:29:02PM +0400, Yury Gribov wrote:
> > I wonder if this can't be done through
> > just adding -fno-rtti to AM_CXXFLAGS and say
> > ubsan_handlers_cxx.% ubsan_type_hash.% : AM_CXXFLAGS += -frtti
>
> Yup, that did the trick. Attaching modified patch.
> @@ -13,11 +13,13 @@ toolexeclib_LTLIBRARIES = libubsan.la
> ubsan_files = \
> ubsan_diag.cc \
> ubsan_handlers.cc \
> - ubsan_handlers_cxx.cc \
> - ubsan_type_hash.cc \
> ubsan_value.cc
>
> -libubsan_la_SOURCES = $(ubsan_files)
> +ubsan_cxx_files = \
> + ubsan_handlers_cxx.cc \
> + ubsan_type_hash.cc
> +
> +libubsan_la_SOURCES = $(ubsan_files) $(ubsan_cxx_files)
> libubsan_la_LIBADD = $(top_builddir)/sanitizer_common/libsanitizer_common.la
> if !USING_MAC_INTERPOSE
> libubsan_la_LIBADD += $(top_builddir)/interception/libinterception.la
The above hunk is not needed anymore, is it?
Ok for trunk without that hunk (and with Makefile.in regenerated again).
> 2013-11-28 Jakub Jelinek <ja...@redhat.com>
> Yury Gribov <y.gri...@samsung.com>
>
> PR sanitizer/59106
> * ubsan/Makefile.am (AM_CXXFLAGS): Disable -frtti for files that
> don't need it.
> * ubsan/Makefile.in: Regenerated.
Jakub
