obstack.h __PTR_ALIGN vs. ubsan
authorAlan Modra <amodra@gmail.com>
Wed, 1 Sep 2021 23:35:05 +0000 (09:05 +0930)
committerAlan Modra <amodra@gmail.com>
Thu, 2 Sep 2021 04:18:21 +0000 (13:48 +0930)
commit76aa354424729aafdabdf4929aa9751c6fdee081
tree6f4158b08147ef89e98315304cf5bc4d2f31c013
parentc79332d2d13936dfc029a935a9b9823ca0d575c3
obstack.h __PTR_ALIGN vs. ubsan

Current ubsan complains on every use of __PTR_ALIGN (when ptrdiff_t is
as large as a pointer), due to making calculations relative to a NULL
pointer.  This patch avoids the problem by extracting out and
simplifying __BPTR_ALIGN for the usual case.  I've continued to use
ptrdiff_t here, where it might be better to throw away __BPTR_ALIGN
entirely and just assume uintptr_t exists.

* obstack.h (__PTR_ALIGN): Expand and simplify __BPTR_ALIGN
rather than calculating relative to a NULL pointer.
include/obstack.h