The second option is better because it doesn't mutate any objects. To improve readability I'd extract a variable like this:
var x = player.Vector.X - player.Speed * (float)gameTime.ElapsedGameTime.TotalSeconds;
player.Vector = new Vector2(x, player.Vector.Y);