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

Update gpu_fusible.h #25741

Merged
merged 2 commits into from
Feb 18, 2019
Merged

Conversation

siju-samuel
Copy link
Contributor

conumser -> consumer

conumser -> consumer
@pragyaak pragyaak self-assigned this Feb 14, 2019
@pragyaak pragyaak added the size:XS CL Change Size: Extra Small label Feb 14, 2019
@pragyaak pragyaak added this to Assigned Reviewer in PR Queue via automation Feb 14, 2019
@pragyaak pragyaak added the awaiting review Pull request awaiting review label Feb 14, 2019
PR Queue automation moved this from Assigned Reviewer to Approved by Reviewer Feb 14, 2019
@pragyaak pragyaak added kokoro:force-run Tests on submitted change and removed awaiting review Pull request awaiting review labels Feb 14, 2019
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Feb 14, 2019
@pragyaak pragyaak added the ready to pull PR ready for merge process label Feb 15, 2019
PR Queue automation moved this from Approved by Reviewer to Reviewer Requested Changes Feb 16, 2019
@@ -48,7 +48,7 @@ bool IsInputFusibleReduction(const HloInstruction& instr);

// Whether instruction shapes are compatible for multi-output fusion, i.e.
// whether the emitters support lowering the resulting fusion.
// This function works for both, sibling and producer-conumser multi-output
// This function works for both, sibling and producer-consumser multi-output
Copy link
Member

Choose a reason for hiding this comment

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

consumser is still a typo ;)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@d0k thanks.

PR Queue automation moved this from Reviewer Requested Changes to Approved by Reviewer Feb 16, 2019
@tensorflow-copybara tensorflow-copybara merged commit 4fc2fef into tensorflow:master Feb 18, 2019
PR Queue automation moved this from Approved by Reviewer to Merged Feb 18, 2019
tensorflow-copybara pushed a commit that referenced this pull request Feb 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes ready to pull PR ready for merge process size:XS CL Change Size: Extra Small
Projects
PR Queue
  
Merged
Development

Successfully merging this pull request may close these issues.

None yet

7 participants