ExprMatchExpression's optimizer in ExprMatchExpression::getOptimizer() function first calls $expr rewriter to rewrite its child expression and then calls optimize() on a new expression which includes the rewritten expression.

The $expr rewriter calls optimize() on the rewritten expression as well.

One of the two calls seems to be redundant, please consider if one the calls can be safely removed.