Skip to content

Commit

Permalink
Merge pull request #6940 from hughbe/crash-client-hidden
Browse files Browse the repository at this point in the history
Remove CRASH_REPORTER_CLIENT_HIDDEN in favour of equivilent LLVM macro
  • Loading branch information
slavapestov committed Jan 21, 2017
2 parents 28a87e6 + 4e55214 commit 230b502
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
3 changes: 1 addition & 2 deletions include/swift/Runtime/Debug.h
Expand Up @@ -25,7 +25,6 @@

#ifdef SWIFT_HAVE_CRASHREPORTERCLIENT

#define CRASH_REPORTER_CLIENT_HIDDEN __attribute__((visibility("hidden")))
#define CRASHREPORTER_ANNOTATIONS_VERSION 5
#define CRASHREPORTER_ANNOTATIONS_SECTION "__crash_info"

Expand All @@ -41,7 +40,7 @@ struct crashreporter_annotations_t {
};

extern "C" {
CRASH_REPORTER_CLIENT_HIDDEN
LLVM_LIBRARY_VISIBILITY
extern struct crashreporter_annotations_t gCRAnnotations;
}

Expand Down
2 changes: 1 addition & 1 deletion stdlib/public/runtime/Errors.cpp
Expand Up @@ -152,7 +152,7 @@ static void dumpStackTraceEntry(unsigned index, void *framePC) {
// The layout of this struct is CrashReporter ABI, so there are no ABI concerns
// here.
extern "C" {
CRASH_REPORTER_CLIENT_HIDDEN
LLVM_LIBRARY_VISIBILITY
struct crashreporter_annotations_t gCRAnnotations
__attribute__((__section__("__DATA," CRASHREPORTER_ANNOTATIONS_SECTION))) = {
CRASHREPORTER_ANNOTATIONS_VERSION, 0, 0, 0, 0, 0, 0, 0};
Expand Down

0 comments on commit 230b502

Please sign in to comment.