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.

Sign up for GitHub

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

Jump to bottom

Update README.md to reflect LTC landing in main branch #1000

Merged
merged 4 commits into from
Jun 30, 2022
Merged

Update README.md to reflect LTC landing in main branch #1000

merged 4 commits into from
Jun 30, 2022

Conversation

henrytwo
Copy link
Member

@henrytwo henrytwo commented Jun 30, 2022 •
edited
Loading

This PR updates README.md now that the LTC staging branch will be merged into main.

edit: also happy PR #1000 wooo 🎉 🎉

@henrytwo henrytwo requested a review from silvasean June 30, 2022 19:31
@henrytwo henrytwo self-assigned this Jun 30, 2022
@henrytwo henrytwo requested a review from antoniojkim June 30, 2022 19:32
@antoniojkim antoniojkim force-pushed the torch_mlir_ltc_backend branch 2 times, most recently from 14f5d50 to 26c32ad Compare June 30, 2022 19:41
@henrytwo henrytwo changed the title Update README.md Update README.md to reflect LTC entering main branch Jun 30, 2022
@henrytwo henrytwo changed the title Update README.md to reflect LTC entering main branch Update README.md to reflect LTC landing in main branch Jun 30, 2022
antoniojkim
antoniojkim reviewed Jun 30, 2022
View reviewed changes
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
henrytwo and others added 3 commits June 30, 2022 15:54
@henrytwo @antoniojkim
Update README.md
e6a23e6 
Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>
@henrytwo @antoniojkim
Update README.md
c0de898 
Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>
@henrytwo
Update README.md
e81a564
silvasean
silvasean approved these changes Jun 30, 2022
View reviewed changes
Copy link
Contributor

@silvasean silvasean left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great! Could you also add a page in docs/ explaining the architecture and structure of the LTC support in Torch-MLIR in more technical detail (code layout, expeected extension points, etc.)

(separate PR OK)

@henrytwo henrytwo merged commit 3099b0e into torch_mlir_ltc_backend Jun 30, 2022
@henrytwo henrytwo deleted the henrytwo-patch-1 branch June 30, 2022 20:34
antoniojkim added a commit that referenced this pull request Jul 5, 2022
@henrytwo @antoniojkim
Update README.md to reflect LTC landing in main branch (#1000)
fa1615b 
* Update README.md

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>
antoniojkim added a commit that referenced this pull request Jul 7, 2022
@henrytwo @antoniojkim
Update README.md to reflect LTC landing in main branch (#1000)
e2c7dfd 
* Update README.md

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>
henrytwo added a commit that referenced this pull request Jul 8, 2022
@henrytwo @antoniojkim
Update README.md to reflect LTC landing in main branch (#1000)
e6c7f76 
* Update README.md

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>
henrytwo added a commit that referenced this pull request Jul 8, 2022
@henrytwo @antoniojkim
Update README.md to reflect LTC landing in main branch (#1000)
4ac56c3 
* Update README.md

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>
henrytwo added a commit that referenced this pull request Jul 12, 2022
@henrytwo @antoniojkim
Update README.md to reflect LTC landing in main branch (#1000)
f09b3ab 
* Update README.md

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>
antoniojkim added a commit that referenced this pull request Jul 15, 2022
@henrytwo @antoniojkim
Update README.md to reflect LTC landing in main branch (#1000)
3c3c301 
* Update README.md

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>
antoniojkim added a commit that referenced this pull request Jul 19, 2022
@henrytwo @antoniojkim
Update README.md to reflect LTC landing in main branch (#1000)
ddff4c4 
* Update README.md

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>
antoniojkim added a commit that referenced this pull request Jul 22, 2022
@henrytwo @antoniojkim
Update README.md to reflect LTC landing in main branch (#1000)
a399791 
* Update README.md

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>
henrytwo added a commit that referenced this pull request Jul 29, 2022
@henrytwo @antoniojkim
Update README.md to reflect LTC landing in main branch (#1000)
549034d 
* Update README.md

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>
henrytwo added a commit that referenced this pull request Jul 29, 2022
@henrytwo @antoniojkim
Update README.md to reflect LTC landing in main branch (#1000)
eb4b4d7 
* Update README.md

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>
henrytwo added a commit that referenced this pull request Jul 30, 2022
@henrytwo @antoniojkim
Update README.md to reflect LTC landing in main branch (#1000)
a3244b6 
* Update README.md

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>

* Update README.md

Co-authored-by: Jae Hoon (Antonio) Kim <[email protected]>
@tanyokwok tanyokwok mentioned this pull request Sep 21, 2022
Closed
qedawkins pushed a commit to nod-ai/torch-mlir that referenced this pull request Oct 3, 2022
@imaihal
Added MemRefsNormalizable trants to krnl ops whose input or output ha…
978eae3 
…s memref type (llvm#1000)

* Added base class for krnl dialect ops.

Signed-off-by: Haruki Imai <[email protected]>

* Revert "Added base class for krnl dialect ops."

This reverts commit ff5f51e15a6b88193e681d7dad43d07876aae542.

Signed-off-by: Haruki Imai <[email protected]>

* Added MemRefsNormalizable traits to kernl ops

Added MemRefsNormalizable traits to kernl ops whose input or output can
be a memref.

Signed-off-by: Haruki Imai <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@antoniojkim antoniojkim antoniojkim left review comments

@silvasean silvasean silvasean approved these changes

Assignees

@henrytwo henrytwo

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@henrytwo @antoniojkim @silvasean