espressif/esp-idf
1
0
Fork 1
mirror of https://github.com/espressif/esp-idf.git synced 2025-08-01 19:54:32 +02:00
Code Issues Packages Projects Releases Wiki Activity

heap: __builtin_return_address argument has to be a literal value

Browse Source 
Clang does not accept const int as an argument of __builtin_return_address.

Ref LLVM-14
This commit is contained in:
Ivan Grokhotkov
2018-12-06 15:12:17 +08:00
parent 94f2dd27d4
commit 3150920cb4
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
components/heap/heap_trace.c
View File

@@ -271,11 +271,14 @@ inline static uint32_t get_ccount(void)
return ccount; return ccount;
} }
// Caller is 2 stack frames deeper than we care about
#define STACK_OFFSET 2
#define TEST_STACK(N) do { \ #define TEST_STACK(N) do { \
if (STACK_DEPTH == N) { \ if (STACK_DEPTH == N) { \
return; \ return; \
} \ } \
callers[N] = __builtin_return_address(N+offset); \ callers[N] = __builtin_return_address(N+STACK_OFFSET); \
if (!esp_ptr_executable(callers[N])) { \ if (!esp_ptr_executable(callers[N])) { \
return; \ return; \
} \ } \
@@ -288,7 +291,6 @@ inline static uint32_t get_ccount(void)
*/ */
static IRAM_ATTR __attribute__((noinline)) void get_call_stack(void **callers) static IRAM_ATTR __attribute__((noinline)) void get_call_stack(void **callers)
{ {
const int offset = 2; // Caller is 2 stack frames deeper than we care about
bzero(callers, sizeof(void *) * STACK_DEPTH); bzero(callers, sizeof(void *) * STACK_DEPTH);
TEST_STACK(0); TEST_STACK(0);
TEST_STACK(1); TEST_STACK(1);