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

[XLA] Make EmitFullWarpShuffleDownLoopForReduce test the required condition. #52082

Conversation

nouiz
Copy link
Contributor

@nouiz nouiz commented Sep 21, 2021

@cheshire

This check was recently removed by commit 3b9f82e. As it is very hard to detect an error, it is better to keep it.

@google-ml-butler google-ml-butler bot added the size:S CL Change Size: Small label Sep 21, 2021
@google-ml-butler google-ml-butler bot added the awaiting review Pull request awaiting review label Sep 21, 2021
@google-cla google-cla bot added the cla: yes label Sep 21, 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 Sep 22, 2021
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Sep 22, 2021
@gbaned gbaned self-assigned this Sep 22, 2021
@gbaned gbaned added the comp:xla XLA label Sep 22, 2021
@gbaned gbaned added this to Assigned Reviewer in PR Queue via automation Sep 22, 2021
@gbaned gbaned removed the awaiting review Pull request awaiting review label Sep 22, 2021
@sanjoy sanjoy removed their request for review September 22, 2021 20:43
@gbaned
Copy link
Contributor

gbaned commented Sep 23, 2021

@nouiz Can you please resolve conflicts? Thanks!

@gbaned gbaned added stat:awaiting response Status - Awaiting response from author and removed ready to pull PR ready for merge process labels Sep 23, 2021
@nouiz
Copy link
Contributor Author

nouiz commented Sep 23, 2021

rebased.

@nouiz nouiz force-pushed the upstream-EmitFullWarpShuffleDownLoopForReduce branch from 72f16d8 to 0cbbc41 Compare September 23, 2021 13:45
@google-ml-butler google-ml-butler bot added kokoro:force-run Tests on submitted change ready to pull PR ready for merge process labels Sep 23, 2021
PR Queue automation moved this from Assigned Reviewer to Approved by Reviewer Sep 23, 2021
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Sep 23, 2021
@copybara-service copybara-service bot merged commit 8e1c869 into tensorflow:master Sep 24, 2021
@google-ml-butler google-ml-butler bot removed ready to pull PR ready for merge process stat:awaiting response Status - Awaiting response from author labels Sep 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes comp:xla XLA size:S CL Change Size: Small
Projects
PR Queue
  
Approved by Reviewer
Development

Successfully merging this pull request may close these issues.

None yet

4 participants