Disable side-effect guards under read only code cache

Side-effect guards can require runtime assumptions that have to patch
the code, which is not allowed. Therefore, disable the generation of
side-effect guards until such time as the runtime assumptions can patch
a write-able data area, rather than the code cache.

Signed-off-by: Irwin D'Souza <dsouzai.gh@gmail.com>

compiler/optimizer/VPHandlers.cpp

@@ -9193,7 +9193,7 @@ static TR::Node *constrainIfcmpeqne(OMR::ValuePropagation *vp, TR::Node *node, b
                      }
                   }
 
-               if (clazzToBeInitialized &&
+               if (!vp->comp()->getGenerateReadOnlyCode() && clazzToBeInitialized &&
                    (((clazzNameLen == 35) && !strncmp(clazzToBeInitialized, "Lcom/ibm/ejs/ras/TraceEnabledToken;", clazzNameLen)) ||
                     ((clazzNameLen == 41) && !strncmp(clazzToBeInitialized, "Lcom/ibm/websphere/ras/TraceEnabledToken;", clazzNameLen)) ||
                     ((clazzNameLen == 40) && !strncmp(clazzToBeInitialized, "Ljava/lang/String$StringCompressionFlag;", clazzNameLen))) &&