Skip to content

Commit

Permalink
[src] Silence GCC 12 warnings about dangling pointers (#45617)
Browse files Browse the repository at this point in the history
* [src] Silence GCC 12 warnings about dangling pointers

* [src] Compile with `-Wno-pragmas` to allow pragrams accepted only by different compilers

* [src] Use pragma only when relevant (GCC >= 12)

Using `-Wno-pragmas` would still cause issues when building with `-Werror`.
  • Loading branch information
giordano authored Jun 25, 2022
1 parent 51c8812 commit a60c76e
Showing 1 changed file with 15 additions and 0 deletions.
15 changes: 15 additions & 0 deletions src/init.c
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,12 @@ void jl_init_stack_limits(int ismaster, void **stack_lo, void **stack_hi)
pthread_attr_getstack(&attr, &stackaddr, &stacksize);
pthread_attr_destroy(&attr);
*stack_lo = (void*)stackaddr;
#pragma GCC diagnostic push
#if defined(_COMPILER_GCC_) && __GNUC__ >= 12
#pragma GCC diagnostic ignored "-Wdangling-pointer"
#endif
*stack_hi = (void*)&stacksize;
#pragma GCC diagnostic pop
return;
# elif defined(_OS_DARWIN_)
extern void *pthread_get_stackaddr_np(pthread_t thread);
Expand Down Expand Up @@ -104,7 +109,12 @@ void jl_init_stack_limits(int ismaster, void **stack_lo, void **stack_hi)
// We intentionally leak a stack address here core.StackAddressEscape
# ifndef __clang_analyzer__
*stack_hi = (void*)&stacksize;
#pragma GCC diagnostic push
#if defined(_COMPILER_GCC_) && __GNUC__ >= 12
#pragma GCC diagnostic ignored "-Wdangling-pointer"
#endif
*stack_lo = (void*)((char*)*stack_hi - stacksize);
#pragma GCC diagnostic pop
# else
*stack_hi = 0;
*stack_lo = 0;
Expand Down Expand Up @@ -722,8 +732,13 @@ JL_DLLEXPORT void julia_init(JL_IMAGE_SEARCH rel)

jl_gc_init();
jl_ptls_t ptls = jl_init_threadtls(0);
#pragma GCC diagnostic push
#if defined(_COMPILER_GCC_) && __GNUC__ >= 12
#pragma GCC diagnostic ignored "-Wdangling-pointer"
#endif
// warning: this changes `jl_current_task`, so be careful not to call that from this function
jl_task_t *ct = jl_init_root_task(ptls, stack_lo, stack_hi);
#pragma GCC diagnostic pop
JL_GC_PROMISE_ROOTED(ct);
_finish_julia_init(rel, ptls, ct);
}
Expand Down

2 comments on commit a60c76e

@nanosoldier
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Executing the daily package evaluation, I will reply here when finished:

@nanosoldier runtests(ALL, isdaily = true)

@nanosoldier
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your package evaluation job has completed - possible new issues were detected. A full report can be found here.

Please sign in to comment.