remove unnecessary and problematic _Noreturn from crt/ldso startup
authorRich Felker <dalias@aerifal.cx>
Tue, 25 Jun 2019 22:50:05 +0000 (18:50 -0400)
committerRich Felker <dalias@aerifal.cx>
Tue, 25 Jun 2019 23:05:40 +0000 (19:05 -0400)
commit54b7564b72c1edcc79fca083047521fb56eaea00
tree4756518df5e03dc40701b157fec55e7f54e414e7
parent95dfa3dd12108f42b23a1083e7b32266246a3590
remove unnecessary and problematic _Noreturn from crt/ldso startup

after commit a48ccc159a5fa061a18419296100ee48a1cd6cc9 removed the use
of _Noreturn on the stage3_func type (which only worked due to it
being defined to the "GNU C" attribute in C99 mode), GCC could no
longer assume that the ends of __dls2 and __dls2b are unreachable, and
produced a warning that a function marked _Noreturn returns.

also, since commit 4390383b32250a941ec616e8bff6f568a801b1c0, the
_Noreturn declaration for __libc_start_main in crt1/rcrt1 has been not
only inconsistent with the definition, but wrong. formally,
__libc_start_main does return, via a (hopefully) tail call to a helper
function after the barrier. incorrect usage of _Noreturn in the
declaration was probably formal UB.

the _Noreturn specifiers were not useful in any of these places, so
remove them all. now, the only remaining usage of _Noreturn is in
public interfaces where _Noreturn is part of their contract.
crt/crt1.c
crt/rcrt1.c
ldso/dynlink.c