Skip to content

Conversation

@srkreddy1238
Copy link
Contributor

This brings in basic support for Adreno target on Windows platforms and the tools we generally use for Adreno

Basic support while building Adreno on Windows
@tqchen
Copy link
Member

tqchen commented Nov 14, 2024

would be great to confirm the compatibility of this since zlib is also used as a dep for LLM which we link against, wonder if zlib here would go against that

@srkreddy1238
Copy link
Contributor Author

The zlib here is used only while building rtvm cli for windows (dependency of cnpy). LLM don't have an issue with this.

@tqchen
Copy link
Member

tqchen commented Nov 14, 2024

get it, let us put zlib license here https://github.com/apache/tvm/tree/main/licenses

@srkreddy1238 srkreddy1238 force-pushed the windows_adreno branch 2 times, most recently from 9e4f56e to 8c36cd5 Compare December 6, 2024 05:05
@srkreddy1238 srkreddy1238 force-pushed the windows_adreno branch 2 times, most recently from 0d7e811 to 8188e12 Compare January 4, 2025 04:10
@srkreddy1238
Copy link
Contributor Author

@tvm-bot rerun

8 similar comments
@srkreddy1238
Copy link
Contributor Author

@tvm-bot rerun

@srkreddy1238
Copy link
Contributor Author

@tvm-bot rerun

@srkreddy1238
Copy link
Contributor Author

@tvm-bot rerun

@srkreddy1238
Copy link
Contributor Author

@tvm-bot rerun

@srkreddy1238
Copy link
Contributor Author

@tvm-bot rerun

@srkreddy1238
Copy link
Contributor Author

@tvm-bot rerun

@srkreddy1238
Copy link
Contributor Author

@tvm-bot rerun

@srkreddy1238
Copy link
Contributor Author

@tvm-bot rerun

@tqchen
Copy link
Member

tqchen commented Jan 25, 2025

interesting the ci failed in this way, error was in python 3.8, latest one was 3.9 in the CI it seems. Not sure if it is due to CI image staleness. perhaps we can try close and reopen another PR, to see if it picks up the latest main changes in python version

@srkreddy1238
Copy link
Contributor Author

Ref. #17608 reopen on new CI.

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