Commit2fe4c73

committed

Make AssertPointerAlignment available to frontend code

We don't need separate definitions for frontend and backend, since thecontained Assert() will take care of the difference. So this alsomakes it simpler overall.Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us>Reviewed-by: Michael Paquier <michael@paquier.xyz>Discussion:https://www.postgresql.org/message-id/f64365b1-d5f9-ef83-41fe-404810f10e5a@enterprisedb.com

1 parentdea8349 commit2fe4c73Copy full SHA for 2fe4c73

File tree

-4

lines changed

-4

lines changed

Lines changed: 2 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -793,14 +793,12 @@ typedef NameData *Name;`
`793`	`793`
`794`	`794`	`#defineAssert(condition)((void)true)`
`795`	`795`	`#defineAssertMacro(condition)((void)true)`
`796`		`-#defineAssertPointerAlignment(ptr,bndr)((void)true)`
`797`	`796`
`798`	`797`	`#elif defined(FRONTEND)`
`799`	`798`
`800`	`799`	`#include<assert.h>`
`801`	`800`	`#defineAssert(p) assert(p)`
`802`	`801`	`#defineAssertMacro(p)((void) assert(p))`
`803`		`-#defineAssertPointerAlignment(ptr,bndr)((void)true)`
`804`	`802`
`805`	`803`	`#else/* USE_ASSERT_CHECKING && !FRONTEND */`
`806`	`804`
`@@ -824,14 +822,14 @@ typedef NameData *Name;`
`824`	`822`	`((void) ((condition) \|\| \`
`825`	`823`	`(ExceptionalCondition(#condition, __FILE__, __LINE__), 0)))`
`826`	`824`
	`825`	`+#endif/* USE_ASSERT_CHECKING && !FRONTEND */`
	`826`	`+`
`827`	`827`	`/*`
`828`	`828`	* Check that `ptr' is `bndr' aligned.
`829`	`829`	`*/`
`830`	`830`	`#defineAssertPointerAlignment(ptr,bndr) \`
`831`	`831`	`Assert(TYPEALIGN(bndr, (uintptr_t)(ptr)) == (uintptr_t)(ptr))`
`832`	`832`
`833`		`-#endif/* USE_ASSERT_CHECKING && !FRONTEND */`
`834`		`-`
`835`	`833`	`/*`
`836`	`834`	`* ExceptionalCondition is compiled into the backend whether or not`
`837`	`835`	`* USE_ASSERT_CHECKING is defined, so as to support use of extensions`

Comments

(0)