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(-) -- Sent via pgsql-committers mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-committers
