You should probably make anything curved a staticmesh.
The angles cut through your geometry (because the engine is weird) and it can cause problems later on. But perhaps more importantly: the way BSP receives light it makes corners very obvious so it's not going to look round.
StaticMeshes can smooth out the lighting via smoothing groups.
Complex shapes made with the 2d Shape Editor often have extra edges and vertices as well that wouldn't be needed to complete the shape but are there anyway. I would be careful with those because they have a tendency to cause cuts.
It's probably best to stick to cubes, where it's reasonable. E.g. when making a floor plan of an office floor, use individual cubes instead of one uber-complex 2d Shape Editor brush. Performance may be roughly the same but it's safer in the long run, in my experience.