Retire pvMove in search()

Now we can directly replace it with
the definition resulting in simpler
and possibly faster code because
PvNode is evaluated at compile time.

No functional change.
Since we are inside the RootNode check, PvNode == true, so we don't need to check on that.