This repository was archived by the owner on Sep 10, 2025. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 248
Closed
Changes from all commits
Commits
Show all changes
29 commits
Select commit
Hold shift + click to select a range
b316a9e
add new torchao experimental kernels to torchchat
metascroy 4a0d89c
add doc
metascroy a93d85e
update torchao library name
metascroy 09393f9
typo
metascroy d15816e
add multithreading to ET runner
metascroy a0594f2
update lib names
metascroy d3bf94c
update lib name
metascroy c75133d
update torchao ExecuTorch op lib to static
metascroy 8527ac5
remove old comment
metascroy 2d7a419
add dylib for poor perf repro testing
metascroy 3b20970
change /Users/scroy to /Users/scroy
metascroy 74d221e
remove single-threaded reference in docs
metascroy c319cfc
add github workflow for testing
metascroy 430fb7a
fix torchchat-root in install script
metascroy 1e47737
bug fixes
metascroy eb481bf
ci fixes
metascroy dff891a
fix
metascroy 55a98e4
ci fixes
metascroy 8744a94
fix ci
metascroy d241641
ci fixes
metascroy de9c6de
ci update
metascroy 9fe8070
ci fixes
metascroy da244a0
ci fixes
metascroy e1e997b
ci fixes
metascroy bb5c612
ci fixes
metascroy 5f95405
ci update
metascroy 6145c4c
update et pin
metascroy 2443898
update python version
metascroy d0d87cb
update python version
metascroy File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1 @@ | ||
| 91298923a0076c1b41059efb6dad2876426e4b03 | ||
| c75711cb329cab3df91fb9083a18373f9a568377 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| 3fa38aaf1276e36845a82fb399e5054718a441c4 | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -62,7 +62,6 @@ if(executorch_FOUND) | |
|
|
||
| set(EXECUTORCH_SRC_ROOT ${TORCHCHAT_ROOT}/${ET_BUILD_DIR}/src/executorch) | ||
| set(XNNPACK_ROOT ${EXECUTORCH_SRC_ROOT}/backends/xnnpack) | ||
| list(APPEND _srcs ${XNNPACK_ROOT}/threadpool/cpuinfo_utils.cpp) | ||
| list(APPEND _common_include_directories | ||
| ${XNNPACK_ROOT}/third-party/cpuinfo/include) | ||
|
|
||
|
|
@@ -80,7 +79,9 @@ if(executorch_FOUND) | |
| et_run PRIVATE | ||
| executorch | ||
| extension_module | ||
| extension_tensor | ||
| extension_data_loader | ||
| extension_threadpool | ||
| optimized_kernels | ||
| quantized_kernels | ||
| portable_kernels | ||
|
|
@@ -111,6 +112,15 @@ if(executorch_FOUND) | |
| target_link_libraries(et_run PRIVATE log) | ||
| endif() | ||
|
|
||
| if(LINK_TORCHAO_CUSTOM_OPS) | ||
| # target_link_libraries(et_run PRIVATE "${TORCHCHAT_ROOT}/torchao-build/cmake-out/lib/liblinear_a8wxdq_EXECUTORCH${CMAKE_SHARED_LIBRARY_SUFFIX}") | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Uncomment this line to test with dylib (poor perf vs. static lib) |
||
| target_link_libraries(et_run PRIVATE "$<LINK_LIBRARY:WHOLE_ARCHIVE,${TORCHCHAT_ROOT}/torchao-build/cmake-out/lib/liblinear_a8wxdq_EXECUTORCH.a>") | ||
| target_link_libraries(et_run PRIVATE | ||
| "${TORCHCHAT_ROOT}/torchao-build/cmake-out/lib/libtorchao_ops_linear_EXECUTORCH.a" | ||
| "${TORCHCHAT_ROOT}/torchao-build/cmake-out/lib/libtorchao_kernels_aarch64.a" | ||
| ) | ||
| endif() | ||
|
|
||
| # Adding target_link_options_shared_lib as commented out below leads to this: | ||
| # | ||
| # CMake Error at Utils.cmake:22 (target_link_options): | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO: update to commit hash that contains D62394341 after it lands.