From 3154e16737ad17b2c63529e3df627bb5eb3bb3be Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Wed, 29 Jun 2016 19:07:19 -0400 Subject: [PATCH] Dodge compiler bug in Visual Studio 2013. VS2013 apparently has a problem with taking the address of a formal parameter in some cases. We do that elsewhere without trouble, but in this case the address is being passed to a subroutine that will probably get inlined, so maybe the combination of those things is what tickles the bug. Anyway, introducing an extra copy of the parameter value is enough to work around it. Per trouble report from Umair Shahid. Report: --- src/backend/optimizer/path/costsize.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/backend/optimizer/path/costsize.c b/src/backend/optimizer/path/costsize.c index 8c1dcccf9b..c4422fe986 100644 --- a/src/backend/optimizer/path/costsize.c +++ b/src/backend/optimizer/path/costsize.c @@ -3914,8 +3914,10 @@ calc_joinrel_size_estimate(PlannerInfo *root, double outer_rows, double inner_rows, SpecialJoinInfo *sjinfo, - List *restrictlist) + List *restrictlist_in) { + /* This apparently-useless variable dodges a compiler bug in VS2013: */ + List *restrictlist = restrictlist_in; JoinType jointype = sjinfo->jointype; Selectivity fkselec; Selectivity jselec; -- 2.40.0