If you had a lot more terrain shaders in there, then yes, the number of times they are calculated substantially affects the render time (for the same reason that Compute Terrain affects render time). Moreover, someone might follow this whole setup by a Compute Terrain, so you want it to be efficient. Your Displacement to Scalar should be replaced with Get Altitude and its input disconnected. I believe it will produce the same output, but more efficiently.