Tighten the interface to the optimized code map

src/objects.h

Index: src/objects.h
diff --git a/src/objects.h b/src/objects.h
index 8960f92b7f9d0cd9b68339b9bf878f309ecc908e..bfd1ad122b299ab9b439fd87f4e1f43465b7ec80 100644
--- a/src/objects.h
+++ b/src/objects.h
@@ -6509,14 +6509,25 @@ class SharedFunctionInfo: public HeapObject {
                                               Handle<Code> code);
 
   // Add a new entry to the optimized code map for context-dependent code.
-  // |code| is either a code object or an undefined value. In the latter case
-  // the entry just maps |native_context, osr_ast_id| pair to |literals| array.
-  static void AddToOptimizedCodeMap(Handle<SharedFunctionInfo> shared,
-                                    Handle<Context> native_context,
-                                    Handle<HeapObject> code,
-                                    Handle<LiteralsArray> literals,
-                                    BailoutId osr_ast_id);
+  inline static void AddToOptimizedCodeMap(Handle<SharedFunctionInfo> shared,
+                                           Handle<Context> native_context,
+                                           Handle<Code> code,
+                                           Handle<LiteralsArray> literals,
+                                           BailoutId osr_ast_id);
 
+  inline static void AddLiteralsToOptimizedCodeMap(

[Michael Starzinger, 2015/12/10 14:57:28]

nit: Can we have a short comment explaining this method? Just so that the
difference to the above one is documented.

[mvstanton, 2015/12/10 15:21:49]

Done.

+      Handle<SharedFunctionInfo> shared, Handle<Context> native_context,
+      Handle<LiteralsArray> literals, BailoutId osr_ast_id);

[Michael Starzinger, 2015/12/10 14:57:28]

This should not need a {BailoutId}, because without code this value doesn't make
much sense. Can we remove that argument from this interface?

[mvstanton, 2015/12/10 15:21:48]

doh! Absolutely, thanks!

+
+ private:
+  // If code is undefined, then existing code won't be overwritten.
+  static void AddToOptimizedCodeMapInternal(Handle<SharedFunctionInfo> shared,
+                                            Handle<Context> native_context,
+                                            Handle<HeapObject> code,
+                                            Handle<LiteralsArray> literals,
+                                            BailoutId osr_ast_id);
+
+ public:
   // Set up the link between shared function info and the script. The shared
   // function info is added to the list on the script.
   static void SetScript(Handle<SharedFunctionInfo> shared,