diff mbox

[roland/arm-unwind-header] Move ARM internal unwind.h header to the right sysdeps directory.

Message ID 20140505225633.E56292C39D2@topped-with-meat.com
State New
Headers show

Commit Message

Roland McGrath May 5, 2014, 10:56 p.m. UTC 
This is a trivial file renaming, so I'm not including a real patch, just
the 'git apply' flavored one.

unwind.h contains nothing actually specific to Linux, let alone to NPTL.
It's required by sysdeps/arm/backtrace.c, so it should be in a sysdeps
location that is accessible to all configurations building that file.

The file itself appears to be a quite old copy of the file from GCC (which
has since changed its name in GCC sources), complete with wrong copyright
header (which has an exception and so should be legally adequate, but
probably not what we'd do today).  AFAICT much of what's there is not
actually used anywhere in libc, so it is probably better to prune the file
down rather than to try to keep it in sync with the modern GCC file.  (In
particular the _Unwind_decode_target2 function is troubling, because it
uses #ifdef deeply wrongly for code living in libc.  But that function is
wholly unused.)  But that's cleanup for another day.

OK?


Thanks,
Roland


	* sysdeps/unix/sysv/linux/arm/nptl/unwind.h: File moved to ...
	* sysdeps/arm/unwind.h: ... here.

Comments

Joseph Myers May 6, 2014, 10:08 p.m. UTC | #1 
On Mon, 5 May 2014, Roland McGrath wrote:

> This is a trivial file renaming, so I'm not including a real patch, just
> the 'git apply' flavored one.
> 
> unwind.h contains nothing actually specific to Linux, let alone to NPTL.
> It's required by sysdeps/arm/backtrace.c, so it should be in a sysdeps
> location that is accessible to all configurations building that file.
> 
> The file itself appears to be a quite old copy of the file from GCC (which
> has since changed its name in GCC sources), complete with wrong copyright
> header (which has an exception and so should be legally adequate, but
> probably not what we'd do today).  AFAICT much of what's there is not
> actually used anywhere in libc, so it is probably better to prune the file
> down rather than to try to keep it in sync with the modern GCC file.  (In
> particular the _Unwind_decode_target2 function is troubling, because it
> uses #ifdef deeply wrongly for code living in libc.  But that function is
> wholly unused.)  But that's cleanup for another day.
> 
> OK?

OK.
Roland McGrath May 6, 2014, 10:55 p.m. UTC | #2 
Committed.

Thanks,
Roland
diff mbox

Patch

diff --git a/sysdeps/unix/sysv/linux/arm/nptl/unwind.h b/sysdeps/arm/unwind.h
similarity index 100%
rename from sysdeps/unix/sysv/linux/arm/nptl/unwind.h
rename to sysdeps/arm/unwind.h