Skip to content

Makefile: fix dependencies, use auto variables#983

Merged
ggerganov merged 1 commit intoggml-org:masterfrom
sw:make-dep
Apr 14, 2023
Merged

Makefile: fix dependencies, use auto variables#983
ggerganov merged 1 commit intoggml-org:masterfrom
sw:make-dep

Conversation

@sw
Copy link
Copy Markdown
Contributor

@sw sw commented Apr 14, 2023

llama.o was missing a dependency on ggml.h, which caused inconsistent builds for me when switching between branches.

This also introduces automatic variables (crib: the arrow in $< points left to the first prerequisite, the arrow in $^ points up to all the prerequisites).

The benchmark target is a bit weird in that it doesn't refer to a file, but I didn't want to change this too much.

@sw sw requested review from prusnak and unbounded April 14, 2023 19:25
@ggerganov ggerganov merged commit 93265e9 into ggml-org:master Apr 14, 2023
@sw sw deleted the make-dep branch April 14, 2023 19:43
jeroen-mostert pushed a commit to jeroen-mostert/llama.cpp that referenced this pull request Aug 30, 2024
Seunghhon pushed a commit to Seunghhon/llama.cpp that referenced this pull request Apr 26, 2026
phuongncn pushed a commit to phuongncn/llama.cpp-gx10-dgx-sparks-deepseekv4 that referenced this pull request Apr 28, 2026
phuongncn pushed a commit to phuongncn/llama.cpp-gx10-dgx-sparks-deepseekv4 that referenced this pull request Apr 28, 2026
Co-authored-by: Iwan Kawrakow <iwan.kawrakow@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants