Re: [PATCH, Pointer Bounds Checker, Builtins instrumentation 3/5] Expand instrumented builtin calls

[Ilya Enkovich <enkovich dot gnu at gmail dot com>]

2014-11-17 21:22 GMT+03:00 David Edelsohn <dje.gcc@gmail.com>:
> Ilya,
>
> Thanks for fixing the reference to BNDmode.
>
> However, the patch causes another problem that breaks bootstrap on
> AIX.  All of the builtins are emitted as an enum in debug information
> and the CHKP enums now cause an overflow in the debug data on AIX.
> AIX continues to use stabstrings debugging and does not permit stabs
> continuation lines.
>
> This also is failing while building stage1 GCC -- all currently
> deployed GCC compilers will fail when building GCC trunk.  No change
> to the debugging information produced by GCC trunk will fix this.
>
> Over half of the enum list now contains CHKP.
>
> the first _CHKP builtin is 1156
> END_CHKP_BUILTINS is 2381
> END_BUILTINS is 2388
>
> All of the normal builtins now appear to be duplicated with CHKP versions.
>
> This is a huge amount of bloat in the common parts of GCC for a
> feature that only is available on Intel.  Can you please disable the
> feature that creates duplicate CHKP versions of builtins on non-Intel
> architectures or at least on AIX (_AIX macro defined)?
>
> Thanks for your earlier fixes, but can we please adjust the
> implementation so that it does not break other platforms?
>
> Thanks, David

Hello David,

How comes you emit debug info for functions that do not exist and thus
are never used?  Is problem caused by builtins going after
END_CHKP_BUILTINS? Or some info generated for all builtins ignoring
its existence?

Thanks,
Ilya