Use checkStatus to handle the return value of llvh::isArray

`runtime_` is modified on error and restored in `checkStatus`, So it
should be safe to remove the constness and call `checkStatus` here.

API/hermes/hermes.cpp

@@ -1997,9 +1997,7 @@ void HermesRuntimeImpl::setPropertyValue(
 
 bool HermesRuntimeImpl::isArray(const jsi::Object &obj) const {
   auto res = llvh::isArray(runtime_, llvh::dyn_vmcast<vm::JSObject>(phv(obj)));
-  if (LLVM_UNLIKELY(res == llvh::ExecutionStatus::EXCEPTION)) {
-    return false;
-  }
+  const_cast<HermesRuntimeImpl *>(this)->checkStatus(res.getStatus());
   return *res;
 }