-
Notifications
You must be signed in to change notification settings - Fork 154
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
Do not use addr2line by default, backtrace_symbols will do the job #739
Conversation
@@ -132,11 +142,20 @@ static int s_test_memtrace_stacks(struct aws_allocator *allocator, void *ctx) { | |||
/* if this is not a debug build, there may not be symbols, so the test cannot | |||
* verify if a best effort was made */ | |||
#if defined(DEBUG_BUILD) |
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.
Do we still need the define? (It looks like the new approach is meant to handle symbols for a debug build and having no symbols for a non-debug build?)
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.
Yeah, I can't prevent the optimizer from removing those functions, even with noinline.
This commit updates to version 0.4.62 with the following enhancements * Print CMake version when configuring. [0] * Re-adding SOVERSION to CMakeLists.txt [1] * Disabled MSVC 2015's non-constant aggregate initializer warning [2] * Do not use addr2line by default, backtrace_symbols will do the job [3] * Fixed memtrace/stack decoding test so it works more robustly * Removes an unnecessary API hopefully before anyone uses it externally Also block dev-util/cmake-3.19.0 and dev-util/cmake-3.19.1, which introduced a regression causing configure phase to fail. [4] [0] awslabs/aws-c-common#701 [1] awslabs/aws-c-common#702 [2] awslabs/aws-c-common#738 [3] awslabs/aws-c-common#739 [4] https://gitlab.kitware.com/cmake/cmake/-/issues/21529 Closes: https://bugs.gentoo.org/755932 Package-Manager: Portage-3.0.11, Repoman-3.0.2 Signed-off-by: Sven Eden <sven.eden@prydeworx.com>
This commit updates to version 0.4.62 with the following enhancements * Print CMake version when configuring. [0] * Re-adding SOVERSION to CMakeLists.txt [1] * Disabled MSVC 2015's non-constant aggregate initializer warning [2] * Do not use addr2line by default, backtrace_symbols will do the job [3] * Fixed memtrace/stack decoding test so it works more robustly * Removes an unnecessary API hopefully before anyone uses it externally Also block dev-util/cmake-3.19.0 and dev-util/cmake-3.19.1, which introduced a regression causing configure phase to fail. [4] [0] awslabs/aws-c-common#701 [1] awslabs/aws-c-common#702 [2] awslabs/aws-c-common#738 [3] awslabs/aws-c-common#739 [4] https://gitlab.kitware.com/cmake/cmake/-/issues/21529 Closes: https://bugs.gentoo.org/755932 Package-Manager: Portage-3.0.11, Repoman-3.0.2 Signed-off-by: Sven Eden <sven.eden@prydeworx.com> Signed-off-by: Joonas Niilola <juippis@gentoo.org>
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.