public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug driver/106897] New: driver: support -gz=zstd
@ 2022-09-10  0:28 i at maskray dot me
  2022-09-10  0:33 ` [Bug driver/106897] " pinskia at gcc dot gnu.org
                   ` (9 more replies)
  0 siblings, 10 replies; 11+ messages in thread
From: i at maskray dot me @ 2022-09-10  0:28 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106897

            Bug ID: 106897
           Summary: driver: support -gz=zstd
           Product: gcc
           Version: unknown
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: driver
          Assignee: unassigned at gcc dot gnu.org
          Reporter: i at maskray dot me
  Target Milestone: ---

Translate -gz=std to --compress-debug-sections=zstd for as and ld. This
requires that binutils supports zstd, feature request:
https://sourceware.org/bugzilla/show_bug.cgi?id=29397

^ permalink raw reply	[flat|nested] 11+ messages in thread

* [Bug driver/106897] driver: support -gz=zstd
  2022-09-10  0:28 [Bug driver/106897] New: driver: support -gz=zstd i at maskray dot me
@ 2022-09-10  0:33 ` pinskia at gcc dot gnu.org
  2022-09-10  0:36 ` pinskia at gcc dot gnu.org
                   ` (8 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: pinskia at gcc dot gnu.org @ 2022-09-10  0:33 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106897

--- Comment #1 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
Hmm, does gcc already support translating -gz ?

^ permalink raw reply	[flat|nested] 11+ messages in thread

* [Bug driver/106897] driver: support -gz=zstd
  2022-09-10  0:28 [Bug driver/106897] New: driver: support -gz=zstd i at maskray dot me
  2022-09-10  0:33 ` [Bug driver/106897] " pinskia at gcc dot gnu.org
@ 2022-09-10  0:36 ` pinskia at gcc dot gnu.org
  2022-09-10  0:40 ` pinskia at gcc dot gnu.org
                   ` (7 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: pinskia at gcc dot gnu.org @ 2022-09-10  0:36 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106897

--- Comment #2 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
(In reply to Andrew Pinski from comment #1)
> Hmm, does gcc already support translating -gz ?

The answer to my own question is yes.
Adding zstd should be a simple patch I suspect.

^ permalink raw reply	[flat|nested] 11+ messages in thread

* [Bug driver/106897] driver: support -gz=zstd
  2022-09-10  0:28 [Bug driver/106897] New: driver: support -gz=zstd i at maskray dot me
  2022-09-10  0:33 ` [Bug driver/106897] " pinskia at gcc dot gnu.org
  2022-09-10  0:36 ` pinskia at gcc dot gnu.org
@ 2022-09-10  0:40 ` pinskia at gcc dot gnu.org
  2022-09-10  3:09 ` pinskia at gcc dot gnu.org
                   ` (6 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: pinskia at gcc dot gnu.org @ 2022-09-10  0:40 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106897

--- Comment #3 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
The hardest part is the configure change I think
gcc_GAS_CHECK_FEATURE([compressed debug sections],

^ permalink raw reply	[flat|nested] 11+ messages in thread

* [Bug driver/106897] driver: support -gz=zstd
  2022-09-10  0:28 [Bug driver/106897] New: driver: support -gz=zstd i at maskray dot me
                   ` (2 preceding siblings ...)
  2022-09-10  0:40 ` pinskia at gcc dot gnu.org
@ 2022-09-10  3:09 ` pinskia at gcc dot gnu.org
  2022-09-10  7:58 ` i at maskray dot me
                   ` (5 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: pinskia at gcc dot gnu.org @ 2022-09-10  3:09 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106897

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
           Severity|normal                      |enhancement
   Last reconfirmed|                            |2022-09-10
     Ever confirmed|0                           |1

^ permalink raw reply	[flat|nested] 11+ messages in thread

* [Bug driver/106897] driver: support -gz=zstd
  2022-09-10  0:28 [Bug driver/106897] New: driver: support -gz=zstd i at maskray dot me
                   ` (3 preceding siblings ...)
  2022-09-10  3:09 ` pinskia at gcc dot gnu.org
@ 2022-09-10  7:58 ` i at maskray dot me
  2022-09-12 20:01 ` marxin at gcc dot gnu.org
                   ` (4 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: i at maskray dot me @ 2022-09-10  7:58 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106897

--- Comment #4 from Fangrui Song <i at maskray dot me> ---
Yes, the change will be straightforward, basically the files touched by  the
pending https://gcc.gnu.org/pipermail/gcc-patches/2022-July/597586.html
("[PATCH] Remove legacy -gz=zlib-gnu").

I sent it because I knew that we would need a new compression format, and some
cleanup would make the logic more maintainable.

^ permalink raw reply	[flat|nested] 11+ messages in thread

* [Bug driver/106897] driver: support -gz=zstd
  2022-09-10  0:28 [Bug driver/106897] New: driver: support -gz=zstd i at maskray dot me
                   ` (4 preceding siblings ...)
  2022-09-10  7:58 ` i at maskray dot me
@ 2022-09-12 20:01 ` marxin at gcc dot gnu.org
  2022-09-22 12:55 ` marxin at gcc dot gnu.org
                   ` (3 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: marxin at gcc dot gnu.org @ 2022-09-12 20:01 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106897

Martin Liška <marxin at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
           Assignee|unassigned at gcc dot gnu.org      |marxin at gcc dot gnu.org

--- Comment #5 from Martin Liška <marxin at gcc dot gnu.org> ---
I can work on that as I added zstd compression for LTO bytecode some time ago.

^ permalink raw reply	[flat|nested] 11+ messages in thread

* [Bug driver/106897] driver: support -gz=zstd
  2022-09-10  0:28 [Bug driver/106897] New: driver: support -gz=zstd i at maskray dot me
                   ` (5 preceding siblings ...)
  2022-09-12 20:01 ` marxin at gcc dot gnu.org
@ 2022-09-22 12:55 ` marxin at gcc dot gnu.org
  2022-09-29  9:27 ` cvs-commit at gcc dot gnu.org
                   ` (2 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: marxin at gcc dot gnu.org @ 2022-09-22 12:55 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106897

Martin Liška <marxin at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |patch

--- Comment #6 from Martin Liška <marxin at gcc dot gnu.org> ---
Patch candidate:
https://gcc.gnu.org/pipermail/gcc-patches/2022-September/602005.html

^ permalink raw reply	[flat|nested] 11+ messages in thread

* [Bug driver/106897] driver: support -gz=zstd
  2022-09-10  0:28 [Bug driver/106897] New: driver: support -gz=zstd i at maskray dot me
                   ` (6 preceding siblings ...)
  2022-09-22 12:55 ` marxin at gcc dot gnu.org
@ 2022-09-29  9:27 ` cvs-commit at gcc dot gnu.org
  2022-09-29  9:28 ` marxin at gcc dot gnu.org
  2022-11-28 22:40 ` pinskia at gcc dot gnu.org
  9 siblings, 0 replies; 11+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2022-09-29  9:27 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106897

--- Comment #7 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Martin Liska <marxin@gcc.gnu.org>:

https://gcc.gnu.org/g:b7723e135334df95597c8c744276b9de5a88214a

commit r13-2931-gb7723e135334df95597c8c744276b9de5a88214a
Author: Martin Liska <mliska@suse.cz>
Date:   Thu Sep 22 14:30:44 2022 +0200

    support -gz=zstd for both linker and assembler

            PR driver/106897

    gcc/ChangeLog:

            * common.opt: Add -gz=zstd value.
            * configure.ac: Detect --compress-debug-sections=zstd
            for both linker and assembler.
            * configure: Regenerate.
            * gcc.cc (LINK_COMPRESS_DEBUG_SPEC): Handle -gz=zstd.
            (ASM_COMPRESS_DEBUG_SPEC): Likewise.

^ permalink raw reply	[flat|nested] 11+ messages in thread

* [Bug driver/106897] driver: support -gz=zstd
  2022-09-10  0:28 [Bug driver/106897] New: driver: support -gz=zstd i at maskray dot me
                   ` (7 preceding siblings ...)
  2022-09-29  9:27 ` cvs-commit at gcc dot gnu.org
@ 2022-09-29  9:28 ` marxin at gcc dot gnu.org
  2022-11-28 22:40 ` pinskia at gcc dot gnu.org
  9 siblings, 0 replies; 11+ messages in thread
From: marxin at gcc dot gnu.org @ 2022-09-29  9:28 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106897

Martin Liška <marxin at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|ASSIGNED                    |RESOLVED

--- Comment #8 from Martin Liška <marxin at gcc dot gnu.org> ---
Implemented for GCC 13.

^ permalink raw reply	[flat|nested] 11+ messages in thread

* [Bug driver/106897] driver: support -gz=zstd
  2022-09-10  0:28 [Bug driver/106897] New: driver: support -gz=zstd i at maskray dot me
                   ` (8 preceding siblings ...)
  2022-09-29  9:28 ` marxin at gcc dot gnu.org
@ 2022-11-28 22:40 ` pinskia at gcc dot gnu.org
  9 siblings, 0 replies; 11+ messages in thread
From: pinskia at gcc dot gnu.org @ 2022-11-28 22:40 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106897

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |13.0

^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~2022-11-28 22:40 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-10  0:28 [Bug driver/106897] New: driver: support -gz=zstd i at maskray dot me
2022-09-10  0:33 ` [Bug driver/106897] " pinskia at gcc dot gnu.org
2022-09-10  0:36 ` pinskia at gcc dot gnu.org
2022-09-10  0:40 ` pinskia at gcc dot gnu.org
2022-09-10  3:09 ` pinskia at gcc dot gnu.org
2022-09-10  7:58 ` i at maskray dot me
2022-09-12 20:01 ` marxin at gcc dot gnu.org
2022-09-22 12:55 ` marxin at gcc dot gnu.org
2022-09-29  9:27 ` cvs-commit at gcc dot gnu.org
2022-09-29  9:28 ` marxin at gcc dot gnu.org
2022-11-28 22:40 ` pinskia at gcc dot gnu.org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).