Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

common, examples, ggml : fix MSYS2 GCC compiler errors and warnings when building with LLAMA_CURL=ON and GGML_OPENCL=ON #10992

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

peter277
Copy link
Contributor

@peter277 peter277 commented Dec 27, 2024

In common/common.cpp:

  • Convert usage of stat() function call to check if file exists to standard library function std::filesystem::exists (error unable to match to correct function signature)
  • Additional conditions to check if PATH_MAX is already defined in WIN32 environment (warning it is already defined in MSYS2)

In examples/run/run.cpp:

  • Add io.h header inclusion (error cannot find function _get_osfhandle)
  • Change initialisers for OVERLAPPED to empty struct (warning about uninitialised members)
  • Add initialiser for hFile (warning it may be uninitialised)
  • Add cast for curl_off_t percentage value to long int in generate_progress_prefix function (warning that curl_off_t is long long int)

In ggml/src/ggml-opencl/ggml-opencl.cpp:

  • Initialise certain declared cl_mem variables to nullptr for greater safety (warning about B_d variable possibly used unassigned)

@peter277 peter277 force-pushed the master branch 2 times, most recently from 73346ce to a645be4 Compare December 27, 2024 01:32
…hen building with LLAMA_CURL=ON and GGML_OPENCL=ON

In common/common.cpp:
* Convert usage of stat() function call to check if file exists to standard library function std::filesystem::exists (error unable to match to correct function signature)
* Additional conditions to check if PATH_MAX is already defined in WIN32 environment (warning it is already defined in MSYS2)

In examples/run/run.cpp:
* Add io.h header inclusion (error cannot find function _get_osfhandle)
* Change initialisers for OVERLAPPED to empty struct (warning about uninitialised members)
* Add initialiser for hFile (warning it may be uninitialised)
* Add cast for curl_off_t percentage value to long int in generate_progress_prefix function (warning that curl_off_t is long long int)

In ggml/src/ggml-opencl/ggml-opencl.cpp:
* Initialise certain declared cl_mem variables to nullptr for greater safety (warning about B_d variable possibly used unassigned)
@github-actions github-actions bot added the ggml changes relating to the ggml tensor library for machine learning label Dec 27, 2024
@peter277 peter277 changed the title common, examples : fix MSYS2 GCC compiler errors and warnings when building with LLAMA_CURL=ON common, examples : fix MSYS2 GCC compiler errors and warnings when building with LLAMA_CURL=ON and GGML_OPENCL=ON Dec 27, 2024
@peter277 peter277 changed the title common, examples : fix MSYS2 GCC compiler errors and warnings when building with LLAMA_CURL=ON and GGML_OPENCL=ON common, examples, ggml : fix MSYS2 GCC compiler errors and warnings when building with LLAMA_CURL=ON and GGML_OPENCL=ON Dec 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
examples ggml changes relating to the ggml tensor library for machine learning
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant