From 889ec4b9981b9872e0a6e6be4b7cdeb611299257 Mon Sep 17 00:00:00 2001
From: Teodor Sigaev <teodor@sigaev.ru>
Date: Tue, 5 Sep 2006 18:25:10 +0000
Subject: [PATCH] Fix Intel compiler bug. Per discussion 'GIN FailedAssertions
on Itanium2 with Intel compiler' in pgsql-hackers,
http://archives.postgresql.org/pgsql-hackers/2006-08/msg01914.php
---
src/backend/access/gin/ginutil.c | 19 +++++++++++++++++--
1 file changed, 17 insertions(+), 2 deletions(-)
diff --git a/src/backend/access/gin/ginutil.c b/src/backend/access/gin/ginutil.c
index 9f1fd12b2cd..17eca3d239f 100644
--- a/src/backend/access/gin/ginutil.c
+++ b/src/backend/access/gin/ginutil.c
@@ -8,7 +8,7 @@
* Portions Copyright (c) 1994, Regents of the University of California
*
* IDENTIFICATION
- * $PostgreSQL: pgsql/src/backend/access/gin/ginutil.c,v 1.5 2006/07/14 14:52:16 momjian Exp $
+ * $PostgreSQL: pgsql/src/backend/access/gin/ginutil.c,v 1.6 2006/09/05 18:25:10 teodor Exp $
*-------------------------------------------------------------------------
*/
@@ -122,7 +122,22 @@ compareEntries(GinState *ginstate, Datum a, Datum b) {
}
static FmgrInfo* cmpDatumPtr=NULL;
-static bool needUnique = FALSE;
+
+#if defined(__INTEL_COMPILER) && (defined(__ia64__) || defined(__ia64))
+/*
+ * Intel Compiler on Intel Itanium with -O2 has a bug around
+ * change static variable by user function called from
+ * libc func: it doesn't change. So mark it as volatile.
+ *
+ * It's a pity, but it's impossible to define optimization
+ * level here.
+ */
+#define VOLATILE volatile
+#else
+#define VOLATILE
+#endif
+
+static bool VOLATILE needUnique = FALSE;
static int
cmpEntries(const void * a, const void * b) {
--
GitLab