Make CATCH macro assert not fatal

The CATCH macro in exceptions.hpp has a ShouldNotReachHere that will fail in PRODUCT mode. This should only fail in debug mode.
The CATCH macro isn't used very often and I want to resist changing the name or deleting it. Where it's used, it seems useful and there isn't a better short catchy (get it?) name that we've been able to come up with.