pgsql: Remove unnecessary local variables to work around an icc optimiz - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Remove unnecessary local variables to work around an icc optimiz
Date
Msg-id E1W1Jtl-0000rr-70@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Remove unnecessary local variables to work around an icc optimization bug.

Buildfarm member dunlin has been crashing since commit 8b49a60, but other
machines seem fine with that code.  It turns out that removing the local
variables in ordered_set_startup() that are copies of fields in "qstate"
dodges the problem.  This might cost a few cycles on register-rich
machines, but it's probably a wash on others, and in any case this code
isn't performance-critical.  Thanks to Jeremy Drake for off-list
investigation.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/faab7a957d31389f4abfd83784f622c91d076f49

Modified Files
--------------
src/backend/utils/adt/orderedsetaggs.c |   40 ++++++++++++--------------------
1 file changed, 15 insertions(+), 25 deletions(-)


pgsql-committers by date:

Previous
From: Michael Meskes
Date:
Subject: pgsql: Changed regression test to ecpg test suite for alignment problem
Next
From: Alvaro Herrera
Date:
Subject: pgsql: Accept pg_upgraded tuples during multixact freezing