From 16d27a43ddfe4543ea7c9d5bfd8c364ab6f05585 Mon Sep 17 00:00:00 2001 From: Scott Roy Date: Thu, 25 Sep 2025 09:55:17 -0700 Subject: [PATCH] Re-introduce some logging after backout (#14526) Summary: Since D80715432 landed, we also landed D81256207 to add more logging. To prevent conflicts, we backout in 3 steps: * In diff 1/3, we backout D81256207, which added more logging * In diff 2/3, we backout D80715432, which is the problematic diff we're trying to revert * In diff 3/3, we introduce some of the logging we backed out in diff 1/3 Reviewed By: mergennachin Differential Revision: D82581441 --- backends/apple/coreml/runtime/delegate/ETCoreMLModelManager.mm | 2 ++ 1 file changed, 2 insertions(+) diff --git a/backends/apple/coreml/runtime/delegate/ETCoreMLModelManager.mm b/backends/apple/coreml/runtime/delegate/ETCoreMLModelManager.mm index f4cfd2146ac..2347936fd34 100644 --- a/backends/apple/coreml/runtime/delegate/ETCoreMLModelManager.mm +++ b/backends/apple/coreml/runtime/delegate/ETCoreMLModelManager.mm @@ -436,11 +436,13 @@ - (nullable NSURL *)compiledModelURLWithIdentifier:(NSString *)identifier switch (modelAssetType.value()) { case ModelAssetType::CompiledModel: { // Model is already compiled. + ETCoreMLLogInfo("The model in the pte file is pre-compiled. Skipping compilation."); return modelURL; } case ModelAssetType::Model: { // Compile the model. + ETCoreMLLogInfo("The model in the pte file is not pre-compiled. Compiling with a 5 min timeout."); NSURL *compiledModelURL = [ETCoreMLModelCompiler compileModelAtURL:modelURL maxWaitTimeInSeconds:(5 * 60) error:error];