There is an efficeincy improvement when going from field windings to magnets, but it's not because of any magic property of the magnets.
It's simply that field windings are made of wire and wire has resistance. Thus the current that goes through the field winding contributes heat. Making a beefer field winding can improve it's efficeincy, but there's only so much size / weight one can tolerate.
Does that mean that brushless magnet motors are the best? Well, they are are if you can spare no expense and keep them operating in a specific range (torque vs speed). I've designed a goodly many BLDC motor controls (got one behind me), and I love their efficeincy and the ease at which you can control them.
But, when it comes to EV's they have a few serious disadvantages. An obvious one is expense. It takes a fairly large BLDC motor and six really large switching transistors to make them go.
Next, their torque tops out and they have an absolute speed limit for a given battery voltage. Thus, at low speed starts, the series wound motor can kick major behind. It can also contribute when the speed is high without a hard speed limit. Finally, the series wound motor only takes one large switching transistor, this makes for a much cheaper controller.
There is one advantage of the BLDC that I don't know whether anyone is taking: it can be easily designed for regeneration. With a change to the firing pattern, the six transistor controller can do regenerative braking. This is commonly done in industry to break the motor, though the energy is just dumped into a resistor (heat), because it's a matter of some expense to get it back as useful AC power.