http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
--- Comment #19 from ygribov at gcc dot gnu.org ---
Author: ygribov
Date: Thu Nov 28 12:20:23 2013
New Revision: 205482
URL: http://gcc.gnu.org/viewcvs?rev=205482root=gccview=rev
Log:
2013-11-28 Jakub Jelinek ja...@redhat.com
Yury
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
Yury Gribov y.gribov at samsung dot com changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
--- Comment #17 from ygribov at gcc dot gnu.org ---
Author: ygribov
Date: Mon Nov 18 08:03:16 2013
New Revision: 204934
URL: http://gcc.gnu.org/viewcvs?rev=204934root=gccview=rev
Log:
libsanitizer:
2013-11-18 Yury Gribov y.gri...@samsung.com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
--- Comment #18 from Yury Gribov y.gribov at samsung dot com ---
Created attachment 31233
-- http://gcc.gnu.org/bugzilla/attachment.cgi?id=31233action=edit
Libubsan -fno-rtti patch
(In reply to Jakub Jelinek from comment #16)
See my
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
--- Comment #14 from Yury Gribov y.gribov at samsung dot com ---
Sent to gcc-patches: http://gcc.gnu.org/ml/gcc-patches/2013-11/msg01990.html
As for libubsan, it does not seem to support static link anyway:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
--- Comment #15 from Kostya Serebryany kcc at gcc dot gnu.org ---
(In reply to Yury Gribov from comment #14)
Sent to gcc-patches: http://gcc.gnu.org/ml/gcc-patches/2013-11/msg01990.html
As for libubsan, it does not seem to support static link
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
--- Comment #16 from Jakub Jelinek jakub at gcc dot gnu.org ---
See my http://gcc.gnu.org/ml/gcc-patches/2013-11/msg01874.html, that should fix
that -ldl -lpthread already.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
--- Comment #11 from Kostya Serebryany kcc at gcc dot gnu.org ---
Trying to build chrome with gcc's asan...
The build barks as in this bug -- we do need to build libsanitizer (at least
asan) with -fno-rtti
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
--- Comment #12 from Yury Gribov y.gribov at samsung dot com ---
(In reply to Kostya Serebryany from comment #11)
The build barks as in this bug
Did it work with the patch?
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
--- Comment #13 from Kostya Serebryany kcc at gcc dot gnu.org ---
Did it work with the patch?
Yes!
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
--- Comment #1 from Yury Gribov y.gribov at samsung dot com ---
Created attachment 31206
-- http://gcc.gnu.org/bugzilla/attachment.cgi?id=31206action=edit
Proposed patch
Does this look sane?
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
--- Comment #2 from Kostya Serebryany kcc at gcc dot gnu.org ---
We should be building libsanitizer with -fno-rtti
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
--- Comment #3 from Yury Gribov y.gribov at samsung dot com ---
(In reply to Kostya Serebryany from comment #2)
We should be building libsanitizer with -fno-rtti
Ubsan seems to require it:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
Alexey Samsonov samsonov at google dot com changed:
What|Removed |Added
CC||samsonov at
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
--- Comment #5 from Kostya Serebryany kcc at gcc dot gnu.org ---
a small part of ubsan does indeed require rtti.
See ubsan/CMakeLists.txt (UBSAN_SOURCES vs UBSAN_CXX_SOURCES)
The rest of libsanitizer does not and should use -fno-rtti.
OMG, I
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
Yury Gribov y.gribov at samsung dot com changed:
What|Removed |Added
Attachment #31206|0 |1
is
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
--- Comment #7 from Kostya Serebryany kcc at gcc dot gnu.org ---
Patch looks ok.
Please send it in a usual way and someone else (Jakub?) will need to approve
it.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
--- Comment #8 from Jakub Jelinek jakub at gcc dot gnu.org ---
(In reply to Alexey Samsonov from comment #4)
Yeah, in LLVM/Clang we build C++-specific part of UBSan with -frtti (and
link it to C++ programs only), and the rest sanitizers with
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
Kostya Serebryany kcc at gcc dot gnu.org changed:
What|Removed |Added
CC||nlewycky at
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
Yury Gribov y.gribov at samsung dot com changed:
What|Removed |Added
Attachment #31207|0 |1
is
20 matches
Mail list logo