[llvm-bugs] [Bug 26174] New: Don't override abort() and __assert_rtn() in ENABLE_CRASH_OVERRIDES when running on Lion+

via llvm-bugs llvm-bugs at lists.llvm.org
Sat Jan 16 10:18:32 PST 2016


https://llvm.org/bugs/show_bug.cgi?id=26174

            Bug ID: 26174
           Summary: Don't override abort() and __assert_rtn() in
                    ENABLE_CRASH_OVERRIDES when running on Lion+
           Product: libraries
           Version: trunk
          Hardware: PC
                OS: All
            Status: NEW
          Severity: normal
          Priority: P
         Component: Support Libraries
          Assignee: unassignedbugs at nondot.org
          Reporter: jeremyhu at apple.com
                CC: llvm-bugs at lists.llvm.org
    Classification: Unclassified

I updated abort() to call pthread_kill(pthread_self(), SIGABRT) instead of
raise(SIGABRT) in Lion, so the comment justifying these overrides is no longer
relevant.

raise() is still implemented in terms of kill() on current versions of OS X,
but I don't believe that concerns us since the main reason for these overrides
was abort() and assert().

As such, I think we can just remove that code in Signal.inc

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20160116/b25ccd61/attachment.html>


More information about the llvm-bugs mailing list