-
Notifications
You must be signed in to change notification settings - Fork 15
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
Update readme. Add build instructions #69
Conversation
…ijie/add-cpu-runner
Depends on #66 |
README.md
Outdated
We have now installed LLVM at `llvm-project/llvm-install`. | ||
|
||
Clone the graph-compiler repo: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since this is the readme file of this repo, we can skip the clone step.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fine overall. I suggest adding the default build option with fetching llvm installation via gh
. @leshikus, please take a look.
Co-authored-by: ciyong <[email protected]>
Co-authored-by: Petr Kurapov <[email protected]>
Co-authored-by: Petr Kurapov <[email protected]>
…ijie/update_doc
…piler into yijie/update_doc
Please rebase. |
I believe a shorter instruction to run Why readme does not say what the graph compiler is for? I still don't know |
Updates: recommend users to use the all-in-one script. Added a brief description for the project. |
|
Co-authored-by: ciyong <[email protected]>
I am not sure the system requirements of using compile.sh. Let's update it later! |
No description provided.