RFR: 8149591 - Prepare hotspot for GTest

[Kim Barrett]

> On Feb 10, 2016, at 2:47 PM, Jesper Wilhelmsson <jesper.wilhelmsson at oracle.com> wrote:
> 
> Hi,
> 
> Please review this change to prepare the Hotspot code for the Google unit test framework. From the RFE:
> 
> A few changes are needed in the hotspot code to start using the Google Test framework.
> 
> 1. The new() operator as defined in allocation.cpp can not be used together with GTest. This needs to be moved to a separate file so that we can avoid compiling it when building the GTest enabled JVM.
> 
> 2. In management.cpp there is a local variable called err_msg. This variable is shadowing a global variable in debug.hpp. In the GTest work the global err_msg variable is used in the vmassert macro and this creates a conflict with the local variable in management.cpp.

Where does this happen?  The current vmassert macro doesn't use err_msg.

A better way to address this might be to fix the problematic macro.
For safety, macros whose expansions refer to some namespace-scoped
name should qualify the reference.

> 
> 3. If SuppressFatalErrorMessage is set ALL error messages should be suppressed, even the ones in error_is_suppressed() in debug.cpp.
> 
> This is what is done by this change.
> 
> RFE: https://bugs.openjdk.java.net/browse/JDK-8149591
> Webrev: http://cr.openjdk.java.net/~jwilhelm/8149591/webrev.00/index.html
> 
> Thanks,
> /Jesper