This is an archive of the discontinued LLVM Phabricator instance.

Differential D68046

Fix Go ppc64le build
ClosedPublic

Authored by randall77 on Sep 25 2019, 12:49 PM.

Details

Reviewers
dvyukov
vitalybuka
Summary

This #define is in the non-Go ppc64le build but not in the Go build.

Diff Detail

Event Timeline

randall77 created this revision.Sep 25 2019, 12:49 PM
randall77 added a comment.Oct 8 2019, 9:23 AM

Ping.

dvyukov added a reviewer: vitalybuka.Oct 15 2019, 1:23 AM
dvyukov added a project: Restricted Project.
dvyukov accepted this revision.Oct 15 2019, 1:38 AM

Committed as r374868.
I assumed that Sanitizers project and some reviewers should be been added automatically based on changed paths. But somehow it did not happen.
I would suggest to add more people as reviewers in future, but I guess you don't know whom to add...

This revision is now accepted and ready to land.Oct 15 2019, 1:38 AM
dvyukov closed this revision.Oct 15 2019, 1:38 AM

Revision Contents

PathSize
lib/
tsan/
rtl/
2 lines

Diff 221821

lib/tsan/rtl/tsan_platform.h

Context not available.
static const uptr kAppMemEnd = 0x00e000000000ull; static const uptr kAppMemEnd = 0x00e000000000ull;
}; };
#define TSAN_RUNTIME_VMA 1
#elif SANITIZER_GO && defined(__aarch64__) #elif SANITIZER_GO && defined(__aarch64__)
/* Go on linux/aarch64 (48-bit VMA) /* Go on linux/aarch64 (48-bit VMA)
Context not available.