Make forcedCompilation registration thread safe

Author: Christian Wimmer

substratevm/src/com.oracle.svm.hosted/src/com/oracle/svm/hosted/code/CompilationInfoSupport.java

@@ -24,11 +24,10 @@
  */
 package com.oracle.svm.hosted.code;
 
-import java.util.HashMap;
-import java.util.HashSet;
 import java.util.List;
 import java.util.Map;
 import java.util.Set;
+import java.util.concurrent.ConcurrentHashMap;
 import java.util.stream.Collectors;
 
 import org.graalvm.compiler.nodeinfo.Verbosity;
@@ -122,10 +121,10 @@ public DeoptSourceFrameInfo mergeStateInfo(FrameState state) {
 
     protected boolean sealed;
 
-    private final Set<AnalysisMethod> forcedCompilations = new HashSet<>();
-    private final Set<AnalysisMethod> frameInformationRequired = new HashSet<>();
-    private final Map<AnalysisMethod, Map<Long, DeoptSourceFrameInfo>> deoptEntries = new HashMap<>();
-    private final Set<AnalysisMethod> deoptInliningExcludes = new HashSet<>();
+    private final Set<AnalysisMethod> forcedCompilations = ConcurrentHashMap.newKeySet();
+    private final Set<AnalysisMethod> frameInformationRequired = ConcurrentHashMap.newKeySet();
+    private final Map<AnalysisMethod, Map<Long, DeoptSourceFrameInfo>> deoptEntries = new ConcurrentHashMap<>();
+    private final Set<AnalysisMethod> deoptInliningExcludes = ConcurrentHashMap.newKeySet();
 
     public static CompilationInfoSupport singleton() {
         return ImageSingletons.lookup(CompilationInfoSupport.class);
@@ -150,7 +149,7 @@ public void registerFrameInformationRequired(AnalysisMethod method) {
          * deoptimization target. No bci needs to be registered, it is enough to have a non-null
          * value in the map.
          */
-        deoptEntries.computeIfAbsent(method, m -> new HashMap<>());
+        deoptEntries.computeIfAbsent(method, m -> new ConcurrentHashMap<>());
     }
 
     public boolean isFrameInformationRequired(ResolvedJavaMethod method) {
@@ -163,7 +162,7 @@ public void registerDeoptEntry(FrameState state) {
         assert state.bci >= 0;
         long encodedBci = FrameInfoEncoder.encodeBci(state.bci, state.duringCall(), state.rethrowException());
 
-        Map<Long, DeoptSourceFrameInfo> sourceFrameInfoMap = deoptEntries.computeIfAbsent(toAnalysisMethod(state.getMethod()), m -> new HashMap<>());
+        Map<Long, DeoptSourceFrameInfo> sourceFrameInfoMap = deoptEntries.computeIfAbsent(toAnalysisMethod(state.getMethod()), m -> new ConcurrentHashMap<>());
         sourceFrameInfoMap.compute(encodedBci, (k, v) -> v == null ? DeoptSourceFrameInfo.create(state) : v.mergeStateInfo(state));
     }