http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063
--- Comment #24 from Jakub Jelinek jakub at gcc dot gnu.org ---
Author: jakub
Date: Tue Dec 3 16:01:13 2013
New Revision: 205639
URL: http://gcc.gnu.org/viewcvs?rev=205639root=gccview=rev
Log:
PR sanitizer/59063
* lib/asan-dg.exp: Don't
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063
--- Comment #21 from ygribov at gcc dot gnu.org ---
Author: ygribov
Date: Fri Nov 29 12:30:36 2013
New Revision: 205524
URL: http://gcc.gnu.org/viewcvs?rev=205524root=gccview=rev
Log:
gcc/
2013-11-29 Jakub Jelinek ja...@redhat.com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063
--- Comment #22 from Yury Gribov y.gribov at samsung dot com ---
Joost, could you check and close bug if trunk works for you?
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063
Joost VandeVondele Joost.VandeVondele at mat dot ethz.ch changed:
What|Removed |Added
Status|NEW
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063
--- Comment #20 from Kostya Serebryany kcc at gcc dot gnu.org ---
This is fixed in llvm trunk, right?
I am planing another merge to gcc in ~1 week.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063
Richard Biener rguenth at gcc dot gnu.org changed:
What|Removed |Added
Priority|P3 |P1
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063
--- Comment #18 from Yury Gribov y.gribov at samsung dot com ---
(In reply to Evgeniy Stepanov from comment #17)
Sorry, I forgot to mention this. Should be done in r194372.
Thanks! A pity this isn't going to help in gcc case that much because of
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063
Joost VandeVondele Joost.VandeVondele at mat dot ethz.ch changed:
What|Removed |Added
URL|
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063
--- Comment #16 from Yury Gribov y.gribov at samsung dot com ---
(In reply to Evgeniy Stepanov from comment #15)
No need, I'm fixing it now.
Sorry for bothering but is this in compiler-rt trunk yet?
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063
--- Comment #17 from Evgeniy Stepanov eugeni.stepanov at gmail dot com ---
(In reply to Yury Gribov from comment #16)
(In reply to Evgeniy Stepanov from comment #15)
No need, I'm fixing it now.
Sorry for bothering but is this in compiler-rt
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063
Yury Gribov y.gribov at samsung dot com changed:
What|Removed |Added
Attachment #31192|0 |1
is
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063
--- Comment #13 from Yury Gribov y.gribov at samsung dot com ---
(In reply to Evgeniy Stepanov from comment #11)
(In reply to Yury Gribov from comment #4)
Evgeniy, what about a warning in GetRealFunctionAddress if real symbol is
NULL?
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063
--- Comment #14 from Yury Gribov y.gribov at samsung dot com ---
(In reply to Yury Gribov from comment #12)
Makes sense. Attaching new patch.
Forgot to mention - tested on x64_64.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063
--- Comment #15 from Evgeniy Stepanov eugeni.stepanov at gmail dot com ---
(In reply to Yury Gribov from comment #13)
(In reply to Evgeniy Stepanov from comment #11)
(In reply to Yury Gribov from comment #4)
Evgeniy, what about a warning in
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063
Richard Biener rguenth at gcc dot gnu.org changed:
What|Removed |Added
Target Milestone|--- |4.9.0
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063
Yury Gribov y.gribov at samsung dot com changed:
What|Removed |Added
CC||y.gribov at
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063
--- Comment #2 from Evgeniy Stepanov eugeni.stepanov at gmail dot com ---
Weak symbols. Ouch.
We should link with libasan with librt. In Clang, we add -lrt to the linker
flags when we see -fsanitize=(address|thread|memory).
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063
--- Comment #3 from Andrew Pinski pinskia at gcc dot gnu.org ---
I think in newer versions of glibc, clock_gettime is in libc.so now too.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063
--- Comment #4 from Yury Gribov y.gribov at samsung dot com ---
(In reply to Andrew Pinski from comment #3)
I think in newer versions of glibc, clock_gettime is in libc.so now too.
I think Andrew is right - clock_gettime is in libc nowadays.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063
--- Comment #5 from Yury Gribov y.gribov at samsung dot com ---
(In reply to Yury Gribov from comment #4)
I think Andrew is right - clock_gettime is in libc nowadays. Should we
wontfix this bug? A trivial WAR for users would be to add -lrt to
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063
Andrew Pinski pinskia at gcc dot gnu.org changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063
--- Comment #7 from Yury Gribov y.gribov at samsung dot com ---
Created attachment 31192
-- http://gcc.gnu.org/bugzilla/attachment.cgi?id=31192action=edit
Proposed patch
Does this patch make sense?
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063
--- Comment #8 from Jakub Jelinek jakub at gcc dot gnu.org ---
(In reply to Yury Gribov from comment #7)
Created attachment 31192 [details]
Proposed patch
Does this patch make sense?
No. Just add it to STATIC_LIBASAN_LIBS (dunno about tsan,
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063
--- Comment #9 from Yury Gribov y.gribov at samsung dot com ---
(In reply to Jakub Jelinek from comment #8)
No. Just add it to STATIC_LIBASAN_LIBS and make sure libasan is linked
against librt.
Hm, I was under impression that these only get
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063
--- Comment #10 from Jakub Jelinek jakub at gcc dot gnu.org ---
(In reply to Yury Gribov from comment #9)
(In reply to Jakub Jelinek from comment #8)
No. Just add it to STATIC_LIBASAN_LIBS and make sure libasan is linked
against librt.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063
--- Comment #11 from Evgeniy Stepanov eugeni.stepanov at gmail dot com ---
(In reply to Yury Gribov from comment #4)
(In reply to Andrew Pinski from comment #3)
I think in newer versions of glibc, clock_gettime is in libc.so now too.
I think
26 matches
Mail list logo
