That code is part of #15158, see the final commit of this PR for the correct diff.

I think using tap here makes more sense, `initialize_type_map` is defined in 4 places, while this is only defined once. I don't think `initialize_type_map` should have any meaningful return value.