Remove useless if-test.

tglsfdc · tglsfdc · commit 3a2cb5988742 · 2018-03-25T14:54:16.000-04:00
Coverity complained that this check is pointless, and it's right. There is no case where we'd call ExecutorStart with a null plannedstmt, and if we did, it'd have crashed before here. Thinko in commit cc415a5.
diff --git a/src/backend/executor/execMain.c b/src/backend/executor/execMain.c
@@ -249,9 +249,7 @@ standard_ExecutorStart(QueryDesc *queryDesc, int eflags)
 	estate->es_crosscheck_snapshot = RegisterSnapshot(queryDesc->crosscheck_snapshot);
 	estate->es_top_eflags = eflags;
 	estate->es_instrument = queryDesc->instrument_options;
-
-	if (queryDesc->plannedstmt)
-		estate->es_jit_flags = queryDesc->plannedstmt->jitFlags;
+	estate->es_jit_flags = queryDesc->plannedstmt->jitFlags;
 
 	/*
 	 * Set up an AFTER-trigger statement context, unless told not to, or
