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

Embed OpenCL tracing files in the repo #71

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

Conversation

mabraham
Copy link
Contributor

Formerly these were obtained from a fixed commit from https://github.com/intel/compute-runtime.git. This has no advantage compared with directly embedding the files. It makes the tools hard to build on a cluster without internet access.

The git hash from which the embedded copies are taken was ca7d47598a63959d42750c62a2981e08ffa392a1 and has been recorded in those files. That was the only modification to the files.

The copies were embedded in a subdirectory matching those from the repo from which they were taken, so no modifications to these files are needed in order to use them. All tools in this repo already add the utils subdirectory as a directory in which the compiler should search.

Also minor related changes and simplifications.

Formerly these were obtained from a fixed commit from
https://github.com/intel/compute-runtime.git. This has no advantage
compared with directly embedding the files. It makes the tools hard to
build on a cluster without internet access.

The git hash from which the embedded copies are taken was
ca7d47598a63959d42750c62a2981e08ffa392a1 and has been recorded in
those files. That was the only modification to the files.

The copies were embedded in a subdirectory matching those from the
repo from which they were taken, so no modifications to these files
are needed in order to use them. All tools in this repo already add
the utils subdirectory as a directory in which the compiler should
search.

Also minor related changes and simplifications.
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.

1 participant