It's generally not safe to compare floating-point values for equality. I mean using `mcv_sel == 0.0` to detect that there are no extended MCV matches. IMO, we should use a bool variable (similar to how var_eq_const() does it) to track whether any MCV item matched, specifically in the loop where we check extended MCV matches in mcv_clauselist_selectivity().
