https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68260
Jakub Jelinek changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68260
--- Comment #12 from Jakub Jelinek ---
Author: jakub
Date: Tue May 30 07:22:25 2017
New Revision: 248606
URL: https://gcc.gnu.org/viewcvs?rev=248606=gcc=rev
Log:
Backported from mainline
2016-09-14 Jakub Jelinek
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68260
--- Comment #11 from Jakub Jelinek ---
Fixed for 6.3+ so far.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68260
--- Comment #10 from Jakub Jelinek ---
Author: jakub
Date: Fri Sep 16 09:37:50 2016
New Revision: 240182
URL: https://gcc.gnu.org/viewcvs?rev=240182=gcc=rev
Log:
Backported from mainline
2016-09-14 Jakub Jelinek
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68260
--- Comment #9 from Jakub Jelinek ---
Author: jakub
Date: Wed Sep 14 09:01:49 2016
New Revision: 240129
URL: https://gcc.gnu.org/viewcvs?rev=240129=gcc=rev
Log:
PR sanitizer/68260
* tsan.c: Include target.h.
(enum
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68260
Andrew Pinski changed:
What|Removed |Added
CC||rogero at howzatt dot
demon.co.uk
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68260
--- Comment #7 from Dmitry Vyukov ---
I looked at the patch, but I am unqualified to review it. The test looks good
to me. +Yuri
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68260
--- Comment #6 from Jakub Jelinek ---
Created attachment 39573
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=39573=edit
gcc7-pr68260.patch
Untested fix.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68260
Jakub Jelinek changed:
What|Removed |Added
Status|UNCONFIRMED |ASSIGNED
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68260
--- Comment #4 from Dmitry Vyukov ---
Good point. I wonder if using -O2 fixes this.
We tend to always use tsan with -O2 for performance reasons. Tsan already
considerably slows down execution, and additional unnecessary memory accesses
and
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68260
--- Comment #3 from Andrew Pinski ---
(In reply to Dmitry Vyukov from comment #1)
> gcc does not instrument atomic operations:
>
> 004a73d0 <_ZN4spin6unlockEv>:
>
> void unlock() { flag.clear(std::memory_order_release); }
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68260
--- Comment #2 from Andrew Pinski ---
#include
struct spin {
void lock() {
while (flag.test_and_set(std::memory_order_acquire))
;
}
void unlock() { flag.clear(std::memory_order_release); }
std::atomic_flag flag =
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68260
Dmitry Vyukov changed:
What|Removed |Added
CC||dvyukov at google dot com
--- Comment
13 matches
Mail list logo
