If you are using an AD636, then you shouldn't need the scaling network, at least for the RMS output. The RMS output is never negative. An AD636 can produce buffered unipolar output, as AD636 contains internal buffer amplifier (in=pin 7 and out=pin 8) perfectly capable of driving the ADC directly. Only caveat there is that the buffer is an emitter follower. Thus it can't sink current. See datasheet fig. 5 and fig. 13. The dB output however, requires level shifting. It also seems to have a quite large temperature coefficient.
Nasty thing about ADCs is that their input impedance is often nonlinear. When the sampling switch closes, the apparent input impedance may drop considerably. Although seemingly low-bandwidth circuit, you might end up with quite fast buffer anyway to reduce the settling time. You might need a buffer if the signal goes also somewhere else. Otherwise, the ADC sampling can cause noise (glitches) on the signal. Figure 21-3 in the PIC datasheet shows the ADC input structure. PIC ADC has several kilo-ohms of internal series resistance (Rss+Ric).
There are some caveats using the AD636 but it can give greater bandwidth than just plain PIC ADC. I'm not so sure if AD636 will improve the accuracy, as the ADC is in the loop anyway. Direct digital computation is better at low frequencies and has better temperature stability. Perhaps just using the MCU ADC could work up to some kHz, if sampling rate would be at least 20 kHz. The cost is of course greatly increased computation requirement, but digital hardware is amazingly cheap nowadays.
Regards,
Janne