public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug other/55353] New: [asan] the flag for asan should match the one used in clang
@ 2012-11-16 16:57 konstantin.s.serebryany at gmail dot com
2012-11-19 5:20 ` [Bug other/55353] " konstantin.s.serebryany at gmail dot com
` (4 more replies)
0 siblings, 5 replies; 6+ messages in thread
From: konstantin.s.serebryany at gmail dot com @ 2012-11-16 16:57 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55353
Bug #: 55353
Summary: [asan] the flag for asan should match the one used in
clang
Classification: Unclassified
Product: gcc
Version: unknown
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: other
AssignedTo: unassigned@gcc.gnu.org
ReportedBy: konstantin.s.serebryany@gmail.com
CC: dseketel@redhat.com, dvyukov@google.com,
jakub@redhat.com
Currently, gcc trunk uses -faddress-sanitizer
Clang has recently changed the flag to -fsanitize=address
(there was a data race between clang change and gcc review-commit cycle :)
We need to match the flags.
Same applies to tsan (seems like it is not committed to gcc yet)
^ permalink raw reply [flat|nested] 6+ messages in thread* [Bug other/55353] [asan] the flag for asan should match the one used in clang
2012-11-16 16:57 [Bug other/55353] New: [asan] the flag for asan should match the one used in clang konstantin.s.serebryany at gmail dot com
@ 2012-11-19 5:20 ` konstantin.s.serebryany at gmail dot com
2012-11-19 5:55 ` wmi at google dot com
` (3 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: konstantin.s.serebryany at gmail dot com @ 2012-11-19 5:20 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55353
Konstantin Serebryany <konstantin.s.serebryany at gmail dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |hjl.tools at gmail dot com,
| |wmi at gcc dot gnu.org
--- Comment #1 from Konstantin Serebryany <konstantin.s.serebryany at gmail dot com> 2012-11-19 05:20:24 UTC ---
Wei, this needs to happen ASAP, otherwise there will be too many places with
the old flag.
^ permalink raw reply [flat|nested] 6+ messages in thread* [Bug other/55353] [asan] the flag for asan should match the one used in clang
2012-11-16 16:57 [Bug other/55353] New: [asan] the flag for asan should match the one used in clang konstantin.s.serebryany at gmail dot com
2012-11-19 5:20 ` [Bug other/55353] " konstantin.s.serebryany at gmail dot com
@ 2012-11-19 5:55 ` wmi at google dot com
2012-11-19 7:45 ` jakub at gcc dot gnu.org
` (2 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: wmi at google dot com @ 2012-11-19 5:55 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55353
--- Comment #2 from wmi at google dot com 2012-11-19 05:54:44 UTC ---
Hi Kostya,
Ok, I will extract the change from the tsan patch and send out a
separate patch about it.
Regards,
Wei.
On Sun, Nov 18, 2012 at 9:20 PM, konstantin.s.serebryany at gmail dot
com <gcc-bugzilla@gcc.gnu.org> wrote:
>
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55353
>
> Konstantin Serebryany <konstantin.s.serebryany at gmail dot com> changed:
>
> What |Removed |Added
> ----------------------------------------------------------------------------
> CC| |hjl.tools at gmail dot com,
> | |wmi at gcc dot gnu.org
>
> --- Comment #1 from Konstantin Serebryany <konstantin.s.serebryany at gmail dot com> 2012-11-19 05:20:24 UTC ---
> Wei, this needs to happen ASAP, otherwise there will be too many places with
> the old flag.
>
> --
> Configure bugmail: http://gcc.gnu.org/bugzilla/userprefs.cgi?tab=email
> ------- You are receiving this mail because: -------
> You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 6+ messages in thread* [Bug other/55353] [asan] the flag for asan should match the one used in clang
2012-11-16 16:57 [Bug other/55353] New: [asan] the flag for asan should match the one used in clang konstantin.s.serebryany at gmail dot com
2012-11-19 5:20 ` [Bug other/55353] " konstantin.s.serebryany at gmail dot com
2012-11-19 5:55 ` wmi at google dot com
@ 2012-11-19 7:45 ` jakub at gcc dot gnu.org
2013-11-13 9:24 ` [Bug sanitizer/55353] " y.gribov at samsung dot com
2013-11-13 9:29 ` kcc at gcc dot gnu.org
4 siblings, 0 replies; 6+ messages in thread
From: jakub at gcc dot gnu.org @ 2012-11-19 7:45 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55353
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |jakub at gcc dot gnu.org
--- Comment #3 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-11-19 07:45:31 UTC ---
Yes, please. But make sure to update all the spots where -faddress-sanitizer
is used in the tree (e.g. gcc/testsuite/lib/asan-dg.exp, gcc/doc/*, etc.).
^ permalink raw reply [flat|nested] 6+ messages in thread* [Bug sanitizer/55353] [asan] the flag for asan should match the one used in clang
2012-11-16 16:57 [Bug other/55353] New: [asan] the flag for asan should match the one used in clang konstantin.s.serebryany at gmail dot com
` (2 preceding siblings ...)
2012-11-19 7:45 ` jakub at gcc dot gnu.org
@ 2013-11-13 9:24 ` y.gribov at samsung dot com
2013-11-13 9:29 ` kcc at gcc dot gnu.org
4 siblings, 0 replies; 6+ messages in thread
From: y.gribov at samsung dot com @ 2013-11-13 9:24 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55353
Yury Gribov <y.gribov at samsung dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |y.gribov at samsung dot com
--- Comment #4 from Yury Gribov <y.gribov at samsung dot com> ---
Could someone close this? Was fixed in r193702
^ permalink raw reply [flat|nested] 6+ messages in thread* [Bug sanitizer/55353] [asan] the flag for asan should match the one used in clang
2012-11-16 16:57 [Bug other/55353] New: [asan] the flag for asan should match the one used in clang konstantin.s.serebryany at gmail dot com
` (3 preceding siblings ...)
2013-11-13 9:24 ` [Bug sanitizer/55353] " y.gribov at samsung dot com
@ 2013-11-13 9:29 ` kcc at gcc dot gnu.org
4 siblings, 0 replies; 6+ messages in thread
From: kcc at gcc dot gnu.org @ 2013-11-13 9:29 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55353
Kostya Serebryany <kcc at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |RESOLVED
Resolution|--- |FIXED
--- Comment #5 from Kostya Serebryany <kcc at gcc dot gnu.org> ---
fixed indeed.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-11-13 9:29 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-11-16 16:57 [Bug other/55353] New: [asan] the flag for asan should match the one used in clang konstantin.s.serebryany at gmail dot com
2012-11-19 5:20 ` [Bug other/55353] " konstantin.s.serebryany at gmail dot com
2012-11-19 5:55 ` wmi at google dot com
2012-11-19 7:45 ` jakub at gcc dot gnu.org
2013-11-13 9:24 ` [Bug sanitizer/55353] " y.gribov at samsung dot com
2013-11-13 9:29 ` kcc at gcc dot gnu.org
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).