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

[ROCm] remove GPU_LAUNCH_KERNEL macro #28566

Conversation

jerryyin
Copy link
Member

@jerryyin jerryyin commented May 9, 2019

This PR is a follow-up to the original PR #28343. The reviewer requested to break down the original large PR to a series of smaller ones. According to the plan here, this PR is the sixth one in the whole series.

@chsigg @whchung

@tensorflow-bot tensorflow-bot bot added the size:S CL Change Size: Small label May 9, 2019
@whchung whchung added the kokoro:force-run Tests on submitted change label May 9, 2019
@tensorflow-bot tensorflow-bot bot added the ready to pull PR ready for merge process label May 9, 2019
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label May 9, 2019
@whchung whchung requested a review from chsigg May 9, 2019 20:40
@rthadur rthadur self-assigned this May 9, 2019
@rthadur rthadur added this to Assigned Reviewer in PR Queue via automation May 9, 2019
@rthadur rthadur removed the ready to pull PR ready for merge process label May 9, 2019
@rthadur rthadur requested review from chsigg and removed request for chsigg May 9, 2019 22:23
pull bot pushed a commit to testkevinbonz/tensorflow that referenced this pull request May 10, 2019
…upstream-pr-remove-GPU_LAUNCH_KERNEL

PiperOrigin-RevId: 247626641
@tensorflow-copybara tensorflow-copybara merged commit 7fecf38 into tensorflow:master May 10, 2019
PR Queue automation moved this from Assigned Reviewer to Merged May 10, 2019
@jerryyin jerryyin deleted the google-upstream-pr-remove-GPU_LAUNCH_KERNEL branch December 20, 2019 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes size:S CL Change Size: Small
Projects
PR Queue
  
Merged
Development

Successfully merging this pull request may close these issues.

None yet

6 participants