Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[AOT] Name mangling in AOT #8014

Merged
merged 2 commits into from
Jun 28, 2021
Merged

[AOT] Name mangling in AOT #8014

merged 2 commits into from
Jun 28, 2021

Commits on Jun 22, 2021

  1. [AOT] Name mangling in AOT

    Mini-RFC is here: https://discuss.tvm.apache.org/t/mini-rfc-name-mangling-in-aot
    
    With this change we'll mangle the name of global symbols so that we can bundle
    together multiple models in the same application.
    
    The relay.build interface has been left unchanged, which means I am
    resuing mod_name as a prefix for all functions. If mod_name is None then
    a "_tvm" prefix is used.
    
    I had to add two different compilation functions:
    - _CompileEngineLowerWithModuleName to mangle all the operators with the mod_name
    - PartitionGraphWithModName to mangle all the operators produced by BYOC
    
    I could have changed signature of both, but that would have meant a very
    invasive refactoring.
    
    I refactored the aot test utils and added some tests for multiple
    models.
    
    Change-Id: I30e93fa075f660054577ea36cf9268ec0c6eebcb
    Giuseppe Rossini committed Jun 22, 2021
    Configuration menu
    Copy the full SHA
    78e0511 View commit details
    Browse the repository at this point in the history

Commits on Jun 23, 2021

  1. retrigger CI

    Change-Id: I4f11da7fce1327ad89bb25f25209b57077b2c6a3
    Giuseppe Rossini committed Jun 23, 2021
    Configuration menu
    Copy the full SHA
    b9f6790 View commit details
    Browse the repository at this point in the history