New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
FreeBSD build fix proposal. #1352
base: main
Are you sure you want to change the base?
Conversation
@@ -26,6 +26,10 @@ | |||
#endif | |||
#endif // FOLLY_MOBILE | |||
|
|||
#if defined(__FreeBSD__) | |||
#define FOLLY_ASSUME_NO_TCMALLOC 1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is supposed to be an override.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it was to make it handier for the FreeBSD package
folly/memory/Malloc.h
Outdated
@@ -145,6 +141,10 @@ inline bool getTCMallocNumericProperty(const char* name, size_t* out) noexcept { | |||
return true; | |||
} | |||
#else | |||
inline bool getTCMallocNumericProperty(const char* name, size_t* out) noexcept { | |||
return MallocExtension_Internal_GetNumericProperty(name, strlen(name), out); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This weak symbol should always be present - folly declares it -, even if it does not link to anything.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not the case unfortunately but anyway only called within this context.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it a problem to declare these weak symbols also on freebsd?
no tcmalloc support on FreeBSD.
28d2f90
to
98db52d
Compare
no tcmalloc support on FreeBSD.