Skip to content

Commit 580d013

Browse files
Update on "[ExecuTorch][#10447] Extend PyBundledModule with extension.BundledModule"
#10447 # Context This issue is a step of #9638. In #9638, we want to have `extension.Module` as the single source of implementation in `pybindings`, which means that `pybindings.PyModule` should use `extension.Module` rather than its own `pybindings.Module`. # Proposal Now that we have `extension.BundledModule` ready, we want to test it out by having our existing `PyBundledModule` to extend it, and let `verify_result_with_bundled_expected_output` to use it, so that we can test out the whole thing with https://github.com/pytorch/executorch/blob/fb45e19055a92d2a91a4d4b7008e135232cbb14b/devtools/bundled_program/test/test_end2end.py Differential Revision: [D73564127](https://our.internmc.facebook.com/intern/diff/D73564127/) [ghstack-poisoned]
2 parents c047287 + 6ab0e91 commit 580d013

File tree

1 file changed

+4
-6
lines changed

1 file changed

+4
-6
lines changed

extension/module/bundled_module.h

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -54,13 +54,11 @@ class BundledModule : public Module {
5454
}
5555

5656
/**
57-
* Constructs an instance with the bundled program buffer pointer.
58-
*
59-
* This constructor reads the program from bundled program buffer to load the
60-
* module with data loader. The bundled program pointer is preserved so that
61-
* the portion outside of program is accessible.
57+
* Constructs an instance by loading a bundled program from a file with
58+
* specified memory locking behavior.
6259
*
63-
* @param[in] file_path The path to the ExecuTorch program file to load.
60+
* @param[in] file_path The path to the ExecuTorch bundled program file to
61+
* load.
6462
* @param[in] memory_allocator A MemoryAllocator used for memory management.
6563
* @param[in] temp_allocator A MemoryAllocator to use when allocating
6664
* temporary data during kernel or delegate execution.

0 commit comments

Comments
 (0)