On 3/30/2026 6:16 AM, Filip Kastl wrote:
Hi.

This is the second version of this patch.  In this version:
- I've also modified gcc.cc instead of just toplev.cc (so both places in GCC's
   codebase where 'stack_limit_increase ()' gets called are now patched
- I correctly request more memory when GCC *is* built with ASAN.  I switched
   that around in v1.

Please also consider this to be a ping.

Btw, if this isn't a stage 4 material, do tell me and I'll postpone pinging to
stage 1 :).

Cheers,
Filip Kastl

-- 8< --


64MB stack is not enough for running
gcc/testsuite/gcc.c-torture/compile/limits-exprparen.c with an
ASAN-instrumented GCC.  Ask for more stack if GCC was compiled with ASAN
instrumentation.

        PR sanitizer/124206

gcc/ChangeLog:

        * gcc.cc (driver::global_initializations): Ask for 128MB stack
        instead of just 64MB when __SANITIZE_ADDRESS__ is defined.
        * toplev.cc (toplev::main): Ditto.
I'm not excited by new #ifdefs.  I don't guess we have any runtime way to check for asan being enabled?  Assuming the answer to that is no, we don't have a way to check that, then this is OK for gcc-17.


jeff

Reply via email to