[Bug rust/108113] Rust and --enable-link-serialization=1

[dkm at gcc dot gnu.org via Gcc-bugs]

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

Marc Poulhiès  changed:

   What|Removed |Added

   Assignee|unassigned at gcc dot gnu.org  |dkm at gcc dot gnu.org
 Status|UNCONFIRMED |ASSIGNED
   Last reconfirmed||2022-12-14
 Ever confirmed|0   |1

--- Comment #1 from Marc Poulhiès  ---
Thanks Jakub! I'll fix that, thanks for the detailed report

[Bug rust/108113] Rust and --enable-link-serialization=1

[tschwinge at gcc dot gnu.org via Gcc-bugs]

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

Thomas Schwinge  changed:

   What|Removed |Added

 CC||tschwinge at gcc dot gnu.org

--- Comment #2 from Thomas Schwinge  ---
Marc, given Jakub's approval in
, are you
going to push "rust: fix link serialization [PR108113]" to GCC upstream master
branch, too?

---

There's no reason, in opinion, to block such ongoing development/fixes on the
GCC/Rust backlog upstreaming process.

[Bug rust/108113] Rust and --enable-link-serialization=1

[dkm at gcc dot gnu.org via Gcc-bugs]

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

--- Comment #3 from Marc Poulhiès  ---
I can yes, but wasn't really sure and didn't want to interfere with Arthur
ongoing work at updating the gcc's master branch with all the pending changes
from github.

Should I submit the patch for formal approval on gcc-patches@ or can I go ahead
and apply it directly ?

[Bug rust/108113] Rust and --enable-link-serialization=1

[tschwinge at gcc dot gnu.org via Gcc-bugs]

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

Thomas Schwinge  changed:

   What|Removed |Added

 CC||jakub at gcc dot gnu.org

--- Comment #4 from Thomas Schwinge  ---
No need for even more formalism than we already have.  ;-)

Thus: 'git push', and then email , referring to
Jakub's 
approval.

[Bug rust/108113] Rust and --enable-link-serialization=1

[jakub at gcc dot gnu.org via Gcc-bugs]

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

--- Comment #5 from Jakub Jelinek  ---
Yeah.

[Bug rust/108113] Rust and --enable-link-serialization=1

[cvs-commit at gcc dot gnu.org via Gcc-bugs]

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

--- Comment #6 from CVS Commits  ---
The master branch has been updated by Marc Poulhi?s :

https://gcc.gnu.org/g:02c031088ac0bbf716aec52e027d615b7a5a572b

commit r13-4804-g02c031088ac0bbf716aec52e027d615b7a5a572b
Author: Marc Poulhiès 
Date:   Wed Dec 14 21:29:13 2022 +0100

rust: fix link serialization [PR108113]

The Make-lang.in was missing the link serialization support.

PR rust/108113

gcc/rust
* Make-lang.in (rust.serial): New variable.
(rust1$(exeext)): Depend on $(rust.prev). Call LINK_PROGRESS.

Signed-off-by: Marc Poulhiès 

[Bug rust/108113] Rust and --enable-link-serialization=1

[dkm at gcc dot gnu.org via Gcc-bugs]

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

Marc Poulhiès  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|ASSIGNED|RESOLVED

--- Comment #7 from Marc Poulhiès  ---
Fixed in master.