diff --git a/src/backend/access/brin/brin.c b/src/backend/access/brin/brin.c
index 95ed4ef362..0dddc6fa9c 100644
--- a/src/backend/access/brin/brin.c
+++ b/src/backend/access/brin/brin.c
@@ -594,9 +594,9 @@ bringetbitmap(IndexScanDesc scan, TIDBitmap *tbm)
 					bval = &dtup->bt_columns[attno - 1];
 
 					/*
-					 * If the range has both allnulls and hasnulls set, it means
-					 * there are no rows in the range, so we can skip it (we know
-					 * there's nothing to match).
+					 * If the BRIN tuple indicates that this range is empty,
+					 * we can skip it: there's nothing to match.  We don't
+					 * need to examine the next columns.
 					 */
 					if (BRIN_RANGE_IS_EMPTY(bval))
 					{
diff --git a/src/backend/access/brin/brin_tuple.c b/src/backend/access/brin/brin_tuple.c
index 7355e330f9..b3ba5ac365 100644
--- a/src/backend/access/brin/brin_tuple.c
+++ b/src/backend/access/brin/brin_tuple.c
@@ -607,8 +607,8 @@ brin_deform_tuple(BrinDesc *brdesc, BrinTuple *tuple, BrinMemTuple *dMemtuple)
 
 		/*
 		 * Make sure to overwrite the hasnulls flag, because it was initialized
-		 * to true by brin_memtuple_initialize and we don't want to skip it if
-		 * allnulls=true.
+		 * to true by brin_memtuple_initialize and we don't want to skip [it] if
+		 * allnulls=true. (XXX "it" what?)
 		 */
 		dtup->bt_columns[keyno].bv_hasnulls = hasnulls[keyno];