NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit993bdb9

committed

pg_attribute_no_sanitize_alignment() macro

Modern gcc and clang compilers offer alignment sanitizers, which help to detectpointer misalignment. However, our codebase already contains x86-specificcrc32 computation code, which uses unalignment access. Thankfully, thosecompilers also support the attribute, which disables alignment sanitizers atthe function level. This commit adds pg_attribute_no_sanitize_alignment(),which wraps this attribute, and applies it to pg_comp_crc32c_sse42() function.Discussion:https://postgr.es/m/CAPpHfdsne3%3DT%3DfMNU45PtxdhSL_J2PjLTeS8rwKnJzUR4YNd4w%40mail.gmail.comDiscussion:https://postgr.es/m/475514.1612745257%40sss.pgh.pa.usAuthor: Alexander Korotkov, revised by Tom LaneReviewed-by: Tom Lane

1 parentc8b21b0 commit993bdb9Copy full SHA for 993bdb9

File tree

2 files changed

+13

-0

lines changed

src
- include
  - c.h
- port
  - pg_crc32c_sse42.c

2 files changed

+13

-0

lines changed

`‎src/include/c.h`

Lines changed: 12 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -132,6 +132,18 @@`
`132`	`132`	`#definepg_nodiscard`
`133`	`133`	`#endif`
`134`	`134`
	`135`	`+/*`
	`136`	`+ * Place this macro before functions that should be allowed to make misaligned`
	`137`	`+ * accesses. Think twice before using it on non-x86-specific code!`
	`138`	`+ * Testing can be done with "-fsanitize=alignment -fsanitize-trap=alignment"`
	`139`	`+ * on clang, or "-fsanitize=alignment -fno-sanitize-recover=alignment" on gcc.`
	`140`	`+ */`
	`141`	`+#if__clang_major__ >=7\|\|__GNUC__ >=5`
	`142`	`+#definepg_attribute_no_sanitize_alignment() __attribute__((no_sanitize("alignment")))`
	`143`	`+#else`
	`144`	`+#definepg_attribute_no_sanitize_alignment()`
	`145`	`+#endif`
	`146`	`+`
`135`	`147`	`/*`
`136`	`148`	`* Append PG_USED_FOR_ASSERTS_ONLY to definitions of variables that are only`
`137`	`149`	`* used in assert-enabled builds, to avoid compiler warnings about unused`

`‎src/port/pg_crc32c_sse42.c`

Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -18,6 +18,7 @@`
`18`	`18`
`19`	`19`	`#include"port/pg_crc32c.h"`
`20`	`20`
	`21`	`+pg_attribute_no_sanitize_alignment()`
`21`	`22`	`pg_crc32c`
`22`	`23`	`pg_comp_crc32c_sse42(pg_crc32ccrc,constvoid*data,size_tlen)`
`23`	`24`	`{`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit993bdb9

File tree

2 files changed

2 files changed

`‎src/include/c.h`

`‎src/port/pg_crc32c_sse42.c`

0 commit comments