[PATCH][RFC] add --enable-zstd-compressed-debug-sections configure option

Pedro Alves pedro@palves.net
Fri Sep 30 11:25:21 GMT 2022


On 2022-09-30 10:48 a.m., Martin Liška wrote:
> Hello.
> 
> The patch can set up default compression algorithm as zstd instead of the
> default zlib-gabi.
> 
> The patch is lightly tested as readelf can't decompress debug sections:
> https://sourceware.org/bugzilla/show_bug.cgi?id=29640
> 
> Thoughts?
> 
> ChangeLog:
> 
> 	* configure.ac: Add --enable-zstd-compressed-debug-sections
> 	configure option.
> 	* configure: Regenerate.

This may become a bit awkward in the future when other better format appears, and you want to
switch to use it by default.  Like, imagine zstd2 is invented.  At that point we'd have to decide
whether to add code to error out if the user specifies both
   "--enable-zstd-compressed-debug-sections --enable-zstd2-compressed-debug-sections",
or always pick the latter option, or some such.  

IMHO, it seems cleaner and more future proof to add instead:

  --enable-default-compressed-debug-sections=zlib|zlib-gnu|zlib-gabi|zstd

WDYT?

Pedro Alves


More information about the Binutils mailing list