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)
Wei, this needs to happen ASAP, otherwise there will be too many places with the old flag.
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.
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.).
Could someone close this? Was fixed in r193702
fixed indeed.