apache / tvm

Open deep learning compiler stack for cpu, gpu and specialized accelerators
https://tvm.apache.org/
Apache License 2.0
11.63k stars 3.45k forks source link

[RFC] Program Analysis Framework in Relay #4449

Closed gussmith23 closed 2 years ago

gussmith23 commented 4 years ago

Please also see #3895, which is @MarisaKirisame's RFC around a specific change to support analyses in Relay.

This RFC pertains to building a centralized, comprehensive program analysis framework in Relay. I see two primary uses of program analyses in Relay: for generating analysis data used by the compiler, to do things such as quantization, and for generating human-readable data used by developers, useful for exploring Relay programs. Whereas Marisa's request pertains more to the first use-case, this request is inspired by the second use-case, and motivated by a desire to build a framework that covers both use-cases. Such frameworks exist -- see LLVM's framework, which is designed to generate data useful to both the compiler and the developer.

I built a small analysis framework for Relay this past summer at Microsoft, useful for building human-readable analyses of Relay programs. A demo of this framework can be found here, on my branch of TVM.

These are features I developed in my simple framework:

These are features which could be considered for a full framework:

These are open questions that I have:

I'm posting this in advance of the TVM conference this year, as I'm doing a lighting talk on program analysis in Relay and am hoping that the talk generates some discussion.

zackcquic commented 3 years ago

Hi @gussmith23:

I just start playing with TVM this year. I think this is very useful to me.

I'm posting this in advance of the TVM conference this year, as I'm doing a lighting talk on program analysis in Relay and am hoping that the talk generates some discussion.

Do you keep the link? I'd like to know more details.

I am wondering if you have plan to make this upstream to TVM? Seems this RFC didn't updated for a long time.

masahi commented 2 years ago

I assume this is no longer active.