[GitHub] [tvm] lhutton1 commented on pull request #11091: [AOT] Enable A-Normal Form in the AOT executor

[GitBox <gi...@apache.org>]

lhutton1 commented on PR #11091:
URL: https://github.com/apache/tvm/pull/11091#issuecomment-1108351307

   Thanks for taking a look @mbs-octoml, yeah our plan was to put our foot-in-the-door at this stage as it doesn't require too much modification and then enable ANF at an earlier stage in the future if need be.
   
   I believe your thinking that the visit order is deterministic is correct, so then enabling ANF would just make the evaluation order explicit, like you said. Making the lowering pipeline to adhere to an explicit ordering at this stage can be helpful for passes which rely on an explicit ordering such as liveness analysis, rather than solely relying on the implementation of the visitor to provide such guarantees. @manupa-arm may be able to comment further here :)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org