Revert of Visit the Optimized Code Map on first call rather than closure creation.

test/cctest/cctest.status

 # Copyright 2011 the V8 project authors. All rights reserved.
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions are
 # met:
 #
 #     * Redistributions of source code must retain the above copyright
 #       notice, this list of conditions and the following disclaimer.
 #     * Redistributions in binary form must reproduce the above
 #       copyright notice, this list of conditions and the following
 #       disclaimer in the documentation and/or other materials provided
@@ skipping 510 matching lines @@
 
   # TODO(mythria,4680): Lack of code-ageing and/or lack of compilation cache
   # in interpreter.
   'test-heap/CompilationCacheCachingBehavior': [FAIL],
 
   # TODO(mstarzinger,4680): Fails due to the turbo-asm pipeline only being taken
   # in compiler.cc GetLazyCode for uncompiled code, and no similar path for eager
   # code.
   'test-api/TurboAsmDisablesNeuter': [FAIL],
 
-  # TODO(mvstanton,4900): CHECK(!g_function->is_compiled());
-  'test-heap/TestUseOfIncrementalBarrierOnCompileLazy': [FAIL],
-
   # TODO(rmcilroy,4837): We don't set a LoadContextSlot for a function as
   # immutable in the BytecodeGraphBuilder, therefore no inlining happens.
   'test-run-inlining/InlineLoopGuardedTwice': [FAIL],
   'test-run-inlining/InlineSurplusArgumentsDeopt': [FAIL],
   'test-run-inlining/InlineTwice': [FAIL],
   'test-run-inlining/InlineSurplusArgumentsObject': [FAIL],
   'test-run-inlining/InlineTwiceDependentDiamond': [FAIL],
   'test-run-inlining/InlineWithArguments': [FAIL],
   'test-run-inlining/InlineLoopUnguardedTwice': [FAIL],
   'test-run-inlining/InlineOmitArgumentsObject': [FAIL],
   'test-run-inlining/InlineLoopUnguardedOnce': [FAIL],
   'test-run-inlining/InlineOmitArgumentsDeopt': [FAIL],
   'test-run-inlining/InlineTwiceDependentDiamondDifferent': [FAIL],
   'test-run-inlining/SimpleInliningContext': [FAIL],
   'test-run-inlining/InlineMutuallyRecursive': [FAIL],
   'test-run-inlining/InlineLoopGuardedEmpty': [FAIL],
   'test-run-inlining/InlineLoopGuardedOnce': [FAIL],
   'test-run-inlining/InlineOmitArguments': [FAIL],
   'test-run-inlining/SimpleInlining': [FAIL],
   'test-run-inlining/InlineLoopUnguardedEmpty': [FAIL],
   'test-run-inlining/InlineNestedBuiltin': [FAIL],
   'test-run-inlining/InlineSurplusArguments': [FAIL],
   'test-run-inlining/InlineBuiltin': [FAIL],
   'test-run-inlining/InlineTwiceDependent': [FAIL],
   'test-run-inlining/SimpleInliningContextDeopt': [FAIL],
 }],  # ignition == True
 
 ]