| Code-Review | +1 |
Binary-Size: Size increase is unavoidable (see above).blank line before this line, between the description contents and the footers
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
blank line before this line, between the description contents and the footers
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Roll TFLite/LiteRT to Next Green Version
Android binary size increase caused by xnnpack roll.
Version Changes:
XNNPACK: 4b7c368f5e48d9292c5834593c2f83e66b55ce83 to d9ff5bf36955f97c657013272acd3a46d696b301
tflite: 66fe77c30816f9c0a503fb9f2f12fdb56c6eee76 to a12cd7d36bb3dbae1dd7a4b572643084bc62f6e1
litert: ee121300b37b49cd4f9942c0f1256936aa9e611a to d419a5d6b81ad7f6db0a469dac5b82d47f3ac980
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Code-Review | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Code-Review | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Code-Review | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Code-Review | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
"LITERT_COMPILE_LIBRARY",Here is the Gemini explained why it failed on Windows but not Linux, "on Linux, shared objects ( .so ) resolve symbols globally at runtime regardless of LITERT_COMPILE_LIBRARY. On Windows ( LITERT_WINDOWS_OS ), LITERT_CAPI_EXPORT expands to __declspec(dllexport) when LITERT_COMPILE_LIBRARY is defined, and __declspec(dllimport) when it is not defined."
Can you try remove "LITERT_COMPILE_LIBRARY" from config("litert_config") in third_party/litert/BUILD.gn and apply it via a private config ( configs += [":litert_private_config"]) only on targets that build the LiteRT library implementation (litert_c , litert_runtime , etc.), so external DLLs properly import (__declspec(dllimport)) across Windows DLL boundaries?
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Here is the Gemini explained why it failed on Windows but not Linux, "on Linux, shared objects ( .so ) resolve symbols globally at runtime regardless of LITERT_COMPILE_LIBRARY. On Windows ( LITERT_WINDOWS_OS ), LITERT_CAPI_EXPORT expands to __declspec(dllexport) when LITERT_COMPILE_LIBRARY is defined, and __declspec(dllimport) when it is not defined."
Can you try remove "LITERT_COMPILE_LIBRARY" from config("litert_config") in third_party/litert/BUILD.gn and apply it via a private config ( configs += [":litert_private_config"]) only on targets that build the LiteRT library implementation (litert_c , litert_runtime , etc.), so external DLLs properly import (__declspec(dllimport)) across Windows DLL boundaries?
Thanks for running this on your machine! Really appreciate it. I've gone ahead and implemented what you suggested. Fingers crossed for the CQ.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
"LITERT_COMPILE_LIBRARY",Jaewon LeeHere is the Gemini explained why it failed on Windows but not Linux, "on Linux, shared objects ( .so ) resolve symbols globally at runtime regardless of LITERT_COMPILE_LIBRARY. On Windows ( LITERT_WINDOWS_OS ), LITERT_CAPI_EXPORT expands to __declspec(dllexport) when LITERT_COMPILE_LIBRARY is defined, and __declspec(dllimport) when it is not defined."
Can you try remove "LITERT_COMPILE_LIBRARY" from config("litert_config") in third_party/litert/BUILD.gn and apply it via a private config ( configs += [":litert_private_config"]) only on targets that build the LiteRT library implementation (litert_c , litert_runtime , etc.), so external DLLs properly import (__declspec(dllimport)) across Windows DLL boundaries?
Thanks for running this on your machine! Really appreciate it. I've gone ahead and implemented what you suggested. Fingers crossed for the CQ.
Actually it may still have issue, I sent out cl/947995524 and cc'd you on that, after that you can just define LITERT_STATIC instead of LITERT_COMPILE_LIBRARY to workaround this issue.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
"LITERT_COMPILE_LIBRARY",Jaewon LeeHere is the Gemini explained why it failed on Windows but not Linux, "on Linux, shared objects ( .so ) resolve symbols globally at runtime regardless of LITERT_COMPILE_LIBRARY. On Windows ( LITERT_WINDOWS_OS ), LITERT_CAPI_EXPORT expands to __declspec(dllexport) when LITERT_COMPILE_LIBRARY is defined, and __declspec(dllimport) when it is not defined."
Can you try remove "LITERT_COMPILE_LIBRARY" from config("litert_config") in third_party/litert/BUILD.gn and apply it via a private config ( configs += [":litert_private_config"]) only on targets that build the LiteRT library implementation (litert_c , litert_runtime , etc.), so external DLLs properly import (__declspec(dllimport)) across Windows DLL boundaries?
Lynne JiangThanks for running this on your machine! Really appreciate it. I've gone ahead and implemented what you suggested. Fingers crossed for the CQ.
Actually it may still have issue, I sent out cl/947995524 and cc'd you on that, after that you can just define LITERT_STATIC instead of LITERT_COMPILE_LIBRARY to workaround this issue.
FYI cl/948456770 was submitted, can you update to the latest version contains the cl and then defines LITERT_STATIC in the public `litert_config` to solve the issue.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
"LITERT_COMPILE_LIBRARY",Jaewon LeeHere is the Gemini explained why it failed on Windows but not Linux, "on Linux, shared objects ( .so ) resolve symbols globally at runtime regardless of LITERT_COMPILE_LIBRARY. On Windows ( LITERT_WINDOWS_OS ), LITERT_CAPI_EXPORT expands to __declspec(dllexport) when LITERT_COMPILE_LIBRARY is defined, and __declspec(dllimport) when it is not defined."
Can you try remove "LITERT_COMPILE_LIBRARY" from config("litert_config") in third_party/litert/BUILD.gn and apply it via a private config ( configs += [":litert_private_config"]) only on targets that build the LiteRT library implementation (litert_c , litert_runtime , etc.), so external DLLs properly import (__declspec(dllimport)) across Windows DLL boundaries?
Lynne JiangThanks for running this on your machine! Really appreciate it. I've gone ahead and implemented what you suggested. Fingers crossed for the CQ.
Lynne JiangActually it may still have issue, I sent out cl/947995524 and cc'd you on that, after that you can just define LITERT_STATIC instead of LITERT_COMPILE_LIBRARY to workaround this issue.
FYI cl/948456770 was submitted, can you update to the latest version contains the cl and then defines LITERT_STATIC in the public `litert_config` to solve the issue.
Thank you Lynne for your fast work on this! I've gone ahead and created a new roll here: https://crrev.com/c/8102719/1..2
Going to mark this CL as abandoned now.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Var('chromium_git') + '/external/github.com/google-ai-edge/LiteRT.git' + '@' + '3bb8b6529011d69dc17613152ae4003c2b5ed6f6',This version doesn't include the fix yet https://github.com/google-ai-edge/LiteRT/commit/e38ed2ff061f3c61f512e9d4e8a738301f90aba6.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Var('chromium_git') + '/external/github.com/google-ai-edge/LiteRT.git' + '@' + '3bb8b6529011d69dc17613152ae4003c2b5ed6f6',This version doesn't include the fix yet https://github.com/google-ai-edge/LiteRT/commit/e38ed2ff061f3c61f512e9d4e8a738301f90aba6.
I have a new roll set up with the fixes here: https://crrev.com/c/8105154 now; abandoning this CL and thank you for the catch!
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
The test failures are caused by the ABI discrepancies when internal and external litert code misaligned, more sprcifically cl/943394391 needs to be included in both internal and external litert.
defines = [ "LITERT_COMPILE_LIBRARY" ]You don't need to define LITERT_COMPILE_LIBRARY any more if we defined LITERT_STATIC, same as below.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
You don't need to define LITERT_COMPILE_LIBRARY any more if we defined LITERT_STATIC, same as below.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
The test failures are caused by the ABI discrepancies when internal and external litert code misaligned, more sprcifically cl/943394391 needs to be included in both internal and external litert.
Resolved offline: we found that the ABI discrepancies were caused by the ODML roll, which has been behind by several weeks now. Significant progress has been made on the roll, which now depends on some of the changes we've made in the litert BUILD.gn file here.
We decided to move forward by temporarily disabling the WebNN tests to get this TF Lite roll in first; then shortly after we will attempt to get the ODML roll in, solving the ABI discrepancies. Afterwards, we will re-enable the WebNN tests.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Roll TFLite/LiteRT to Next Green Version
Version Changes:
XNNPACK: d9ff5bf36955f97c657013272acd3a46d696b301 to dff8cb5db8c102d0f65140838f11de8bc9668c86
tflite: a12cd7d36bb3dbae1dd7a4b572643084bc62f6e1 to 6eded1ce7a1aa23f6ff3e22f163c43061bc14c9f
litert: d419a5d6b81ad7f6db0a469dac5b82d47f3ac980 to 61f432493259e1eddbb72a24426b15beea9395fd
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Code-Review | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Roll TFLite/LiteRT to Next Green Version
Version Changes:
TFLite: 6eded1ce7a1aa23f6ff3e22f163c43061bc14c9f to 7bedf2e94aff6ca62150d74fd40aa63cb73c6583
LiteRT: 44c71d8b9300fb9ec8f646261f5ce3eaf1e76a47 to 1ae23176dd5d3a43e35961d8236d5adda7f8c0a1
XNNPACK: dff8cb5db8c102d0f65140838f11de8bc9668c86 to 09c4a9137f3a19b053dc45724a62f4f73ec7746a
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Code-Review | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Roll TFLite/LiteRT to Next Green Version
Version Changes:
XNNPACK: 09c4a9137f3a19b053dc45724a62f4f73ec7746a to 711f2b6e4c71fd17ca7b8001ebc3744c11aa8cf1
tflite: 7bedf2e94aff6ca62150d74fd40aa63cb73c6583 to cf01bcbd0605c40e8e4e060d23e6bfea7bc9a93e
litert: 1ae23176dd5d3a43e35961d8236d5adda7f8c0a1 to 9eedac0510c52e2014d54a2d1112d96c00d0adc2
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |