http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
Yury Gribov changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
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=205482&root=gcc&view=rev
Log:
2013-11-28 Jakub Jelinek
Yury Gribov
PR san
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
--- Comment #18 from Yury Gribov ---
Created attachment 31233
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=31233&action=edit
Libubsan -fno-rtti patch
(In reply to Jakub Jelinek from comment #16)
> See my http://gcc.gnu.org/ml/gcc-patches/2
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=204934&root=gcc&view=rev
Log:
libsanitizer:
2013-11-18 Yury Gribov
PR sanitizer/591
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
--- Comment #16 from Jakub Jelinek ---
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 #15 from Kostya Serebryany ---
(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 anyway:
you need -l
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
--- Comment #14 from Yury Gribov ---
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:
/home/ygribov/install/gcc-master/bin/gcc -g repro.i -fsanitize=un
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
--- Comment #13 from Kostya Serebryany ---
> Did it work with the patch?
Yes!
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
--- Comment #12 from Yury Gribov ---
(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 #11 from Kostya Serebryany ---
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
Yury Gribov changed:
What|Removed |Added
Attachment #31207|0 |1
is obsolete|
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
Kostya Serebryany changed:
What|Removed |Added
CC||nlewycky at google dot com
--- Commen
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
--- Comment #8 from Jakub Jelinek ---
(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 -fno-rtti.
Can't we just
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
--- Comment #7 from Kostya Serebryany ---
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
Yury Gribov changed:
What|Removed |Added
Attachment #31206|0 |1
is obsolete|
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
--- Comment #5 from Kostya Serebryany ---
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 hate having so many diffe
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
Alexey Samsonov changed:
What|Removed |Added
CC||samsonov at google dot com
--- Comment
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
--- Comment #3 from Yury Gribov ---
(In reply to Kostya Serebryany from comment #2)
> We should be building libsanitizer with -fno-rtti
Ubsan seems to require it:
/home/ygribov/gcc/gcc-master/libsanitizer/ubsan/ubsan_type_hash.cc:121:63:
error:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
--- Comment #2 from Kostya Serebryany ---
We should be building libsanitizer with -fno-rtti
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106
--- Comment #1 from Yury Gribov ---
Created attachment 31206
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=31206&action=edit
Proposed patch
Does this look sane?
20 matches
Mail list logo