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

Add folder pattern for AddOp and MulOp #53553

Merged
merged 1 commit into from Dec 31, 2021

Conversation

qingyunqu
Copy link
Contributor

Add folder pattern:

  • mhlo.add(%0, 0) -> %0
  • mhlo.mul(%0, 1) -> %0

@google-ml-butler google-ml-butler bot added the size:M CL Change Size: Medium label Dec 27, 2021
@google-ml-butler google-ml-butler bot added the awaiting review Pull request awaiting review label Dec 27, 2021
@google-ml-butler google-ml-butler bot added kokoro:force-run Tests on submitted change ready to pull PR ready for merge process labels Dec 27, 2021
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Dec 27, 2021
@gbaned gbaned added this to Assigned Reviewer in PR Queue via automation Dec 28, 2021
Copy link
Member

@jpienaar jpienaar left a comment

Choose a reason for hiding this comment

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

I was going to ask about the equivalent of https://llvm.discourse.group/t/canonicalization-of-x-0-0-in-tosa/4760 , but I see for HLOs we don't consider this.

@google-ml-butler google-ml-butler bot added the kokoro:force-run Tests on submitted change label Dec 29, 2021
PR Queue automation moved this from Assigned Reviewer to Approved by Reviewer Dec 29, 2021
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Dec 29, 2021
@qingyunqu
Copy link
Contributor Author

I was going to ask about the equivalent of https://llvm.discourse.group/t/canonicalization-of-x-0-0-in-tosa/4760 , but I see for HLOs we don't consider this.

There are some models lowered from tf dialect or torch_xla, they are not lowered from tosa. So I think these folders are meaningful.

@google-ml-butler google-ml-butler bot removed the ready to pull PR ready for merge process label Dec 29, 2021
@google-ml-butler google-ml-butler bot added kokoro:force-run Tests on submitted change ready to pull PR ready for merge process labels Dec 29, 2021
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Dec 29, 2021
@gbaned gbaned added ready to pull PR ready for merge process and removed ready to pull PR ready for merge process labels Dec 30, 2021
@google-ml-butler google-ml-butler bot added kokoro:force-run Tests on submitted change ready to pull PR ready for merge process labels Dec 31, 2021
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Dec 31, 2021
copybara-service bot pushed a commit to tensorflow/mlir-hlo that referenced this pull request Dec 31, 2021
Imported from GitHub PR tensorflow/tensorflow#53553

Add folder pattern:
* mhlo.add(%0, 0) -> %0
* mhlo.mul(%0, 1) -> %0
Copybara import of the project:

--
f06d914657484570be0aff4aa159cec4c4e98334 by liuyuanqiang <yuanqiangliu@pku.edu.cn>:

Add folder pattern for AddOp and MulOp

PiperOrigin-RevId: 419128218
@copybara-service copybara-service bot merged commit 221417c into tensorflow:master Dec 31, 2021
@google-ml-butler google-ml-butler bot removed awaiting review Pull request awaiting review ready to pull PR ready for merge process labels Dec 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:M CL Change Size: Medium
Projects
PR Queue
  
Approved by Reviewer
Development

Successfully merging this pull request may close these issues.

None yet

5 participants