/*
 * Copyright (c) Facebook, Inc. and its affiliates.
 *
 * This source code is licensed under the MIT license found in the
 * LICENSE file in the root directory of this source tree.
 */

#include "JSITracing.h"

namespace facebook {
namespace jsi {
void addNativeTracingHooks(Runtime &rt) {
  // Unimplemented
}
} // namespace jsi
} // namespace facebook

/*
 * Copyright (c) Facebook, Inc. and its affiliates.
 *
 * This source code is licensed under the MIT license found in the
 * LICENSE file in the root directory of this source tree.
 */

#include "JSITracing.h"

namespace facebook {
namespace jsi {
void addNativeTracingHooks(Runtime &rt) {
  assert(false && "unimplemented");
}
} // namespace jsi
} // namespace facebook

--- 
+++ 
@@ -10,7 +10,7 @@
 namespace facebook {
 namespace jsi {
 void addNativeTracingHooks(Runtime &rt) {
-  assert(false && "unimplemented");
+  // Unimplemented
 }
 } // namespace jsi
 } // namespace facebook
Remove unconditional assert from addNativeTracingHooks. Do nothing instead.

Summary:
This line was and apparently has always been triggered from the
HermesExecutorFactory. We just didn't notice because we didn't build with
assertions.

Changelog: [Internal] Don't crash on load when assertions and Hermes are enabled

Reviewed By: dulinriley

Differential Revision: D20163138

fbshipit-source-id: af672bb51eeb1833f3e27ad9a00921731c345fc5
