Skip to content

Commit

Permalink
Add comments for clang version checks
Browse files Browse the repository at this point in the history
  • Loading branch information
wejoncy authored Dec 9, 2024
1 parent 2bc621f commit cf3d389
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions onnxruntime/core/providers/coreml/model/model.mm
Original file line number Diff line number Diff line change
Expand Up @@ -303,6 +303,8 @@ Status GetMLMultiArrayCopyInfo(const MLMultiArray* _Nonnull array,

// since __clang_major__ >= 15, MLComputePlan is introduced in <CoreML/CoreML.h>
// We are actually ensure the MacOS/IOS version and Xcode version is greater than `macOS 14.4, iOS 17.4`.
// The macro API_AVAILABLE should also be fine.

// Otherwise, the compiler will complain `MLComputePlan` is not defined.
// we define __clang_analyzer__ here is for bypass static analysis
void ProfileComputePlan(NSURL* compileUrl, MLModelConfiguration* config) {
Expand Down

0 comments on commit cf3d389

Please sign in to comment.