This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCh][Revisedx2] Fix PR55521 by switching libsanitizer from mach_overrideto mac interpose functions on darwin
On Tue, Dec 04, 2012 at 07:10:27PM +0400, Konstantin Serebryany wrote:
> On Tue, Dec 4, 2012 at 7:03 PM, Jack Howarth <howarth@bromo.med.uc.edu> wrote:
> > On Tue, Dec 04, 2012 at 11:02:01AM +0400, Konstantin Serebryany wrote:
> >> r194120.
> >> I've tested it on Linux, but not on Mac.
> >
> > kcc,
> > Tested at r194135 on x86_64-apple-darwin12. Thanks for the commit.
> > Jack
> > ps Since clang in svn trunk has already switched to the mac function imposition version of asan,
>
> did it?
Thats what Nick Kledzik said...
"The clang version of asan has already moved to using interposing and DYLD_INSERT_LIBRARIES."
I'll reconfirm that with llvm/clang trunk svn tonight (as the bootstrap on darwin there is
currently broken).
Jack
>
> > it might be time to deprecate mach_override out of llvm's compiler-rt trunk so
> > lib/interception/mach_override/, lib/interception/interception_mac.cc and
> > lib/interception/interception_mac.h could be removed.
> >>
> >> On Tue, Dec 4, 2012 at 6:44 AM, Alexander Potapenko <glider@google.com> wrote:
> >> > I've added MISSING_BLOCKS_SUPPORT to LLVM compiler-rt in r169206.
> >> > The rest of your change looks good to me as well.
> >> >
> >> > On Mon, Dec 3, 2012 at 6:33 PM, Alexander Potapenko <glider@google.com> wrote:
> >> >> Jack,
> >> >>
> >> >> Note that MAC_INTERPOSE_FUNCTIONS is always defined in interception.h
> >> >> to either 0 or 1.
> >> >> I'm going to keep "#if MAC_INTERPOSE_FUNCTIONS" (adding "&&
> >> >> !defined(MISSING_BLOCKS_SUPPORT)
> >> >> " where appropriate) in libsanitizer.
> >> >>
> >> >> On Mon, Dec 3, 2012 at 11:17 AM, Mike Stump <mikestump@comcast.net> wrote:
> >> >>> On Dec 3, 2012, at 8:02 AM, Jack Howarth <howarth@bromo.med.uc.edu> wrote:
> >> >>>> The attached patch eliminates PR 55521/sanitizer by switching libasan on darwin
> >> >>>> from using mach_override to mac function interposition
> >> >>>
> >> >>> So, I'm thinking the sanitizer people will just review and approve it, even though it says darwin and is heavily darwin specificâ It's ok by me.
> >> >>
> >> >>
> >> >>
> >> >> --
> >> >> Alexander Potapenko
> >> >> Software Engineer
> >> >> Google Moscow
> >> >
> >> >
> >> >
> >> > --
> >> > Alexander Potapenko
> >> > Software Engineer
> >> > Google Moscow