Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/15139
Note: Links to docs will display an error until the docs builds have been completed. ✅ No FailuresAs of commit 0f37370 with merge base c9339e2 ( This comment was automatically generated by Dr. CI and updates every 15 minutes. |
This PR needs a
|
6a66c8f to
56403b9
Compare
56403b9 to
0f37370
Compare
|
thanks for the comments @mergennachin! Updated. |
### Summary Fix bug where if we have two constants with different names but the same data, the second one is not saved and results in missing tensor. (This happened in quant flows). ### Test plan ``` python -m unittest executorch.exir.emit.test.test_emit.TestEmit.test_constant_tagged_tensor_dedup ```
Summary
Fix bug where if we have two constants with different names but the same data, the second one is not saved and results in missing tensor. (This happened in quant flows).
Test plan